ubuntu-touch-coreapps team mailing list archive

[Joey Chan <qqworini@xxxxxxxxx>]

Hi Roman,

I suggest every member should involve the discussion, not just we two. But
I don't know other members' status, I feel we are under some kind of
"chaos"...

I am thinking about the database design to fit  all functions' demand...
Honestly, my mind is so chaos that I don't know how to continue coding and
how to make teamwork for this  :(


2013/3/12 Roman Shchekin <mrqtros@xxxxxxxxx>

> Joey, I already have same solution in my code! =)
> Just look at thinXmlHttpRequestWrapper ;)
>
> When we will discuss later development? What about Thursday?)
>
>
>
> 12.03.13 11:16 Joey Chan написал(а):
>
> Hi Roman,
>
>
> I have approved your branch, and I saw something in your code:
>
>
> /* I really want remove this model. It is loading whole XML only for
> getting feed information. And I haven't found any way to save loaded XML
> to database, what we are want to do for offline reading (I suppose)...
> UPDATE Now this helper model only parses data, downloaded by
> thinXmlHttpRequest item. It works.
>
> */
>
>
> I already have a solution for your issue:
> 1. download the xml content using xmlhttprequest, like: var xmlContent
> = doc.responseText;
> 2. save the xml content into database;
> 3. create xmllistmodel without "source" property, pls use "xml" instead,
> like:
> XmlListModel {
> id: model
> //source: url // don't use
>
>
> xml: xmlcontent // xmlcontent comes from database
> query: "/rss/channel"
>
>
> XmlRole { name: "title"; query: "title/string()" }
>
> }
>
> // for detail, pls read the Qt documentation :)
>
>
>
>
> 2013/3/12 Roman Shchekin <mrqtros@xxxxxxxxx>
>
>
> > Hi, Joey!
> >
> > First of all - how correctly we can write your last name?) Previously it
> > was "Chen", now "Chan"...
> >
> > And now about our project. You only need to review my merge proposal,
> >
> >
> https://code.launchpad.net/~mrqtros/ubuntu-rssreader-app/feedlist/+merge/152463
> > ,
> > and set status to "Approved". Thanks!
> >
> >
> > 2013/3/12 Joey Chan <mrqtros@xxxxxxxxx>
> >
> > > Hi Roman,
> > >
> > > Need some help from me ? What can I do for you ?
> > >
> > > :)
> > >
> > >
> > > 2013/3/12 Roman Shchekin <mrqtros@xxxxxxxxx>
> > >
> > > > Thanks, guys!
> > > > Martin, I already resubmitted :) It is my second attempt.
> > > > And I am really interested to get Jenkins account.
> > > >
> > > > But for now someone can approve my request, like Joey Chen, he
> already
> > > > tried my branch.
> > > >
> > > >
> > > > 2013/3/11 David Planella <mrqtros@xxxxxxxxx>
> > > >
> > > > > I'd recommend #1. Roman, can you or someone from the team change
> the
> > > > > global status of the merge proposal to "Approved"?
> > > > >
> > > > > This will let Jenkins take care of landing the branch in trunk. I
> > would
> > > > > suggest someone else from the RSS team do it, since, while possible
> > in
> > > > > terms of permissions, it is not really good practice to review and
> > > > approve
> > > > > one's own merge proposals, as it defeats the purpose of the review
> > > > process.
> > > > > --
> > > > >
> > > > >
> > > >
> > >
> >
> https://code.launchpad.net/~mrqtros/ubuntu-rssreader-app/feedlist/+merge/152463
> > > > > You are the owner of lp:~mrqtros/ubuntu-rssreader-app/feedlist.
> > > > >
> > > >
> > > > --
> > > >
> > > >
> > >
> >
> https://code.launchpad.net/~mrqtros/ubuntu-rssreader-app/feedlist/+merge/152463
> > > > Your team Ubuntu RSS Feed Reader Developers is subscribed to branch
> > > > lp:ubuntu-rssreader-app.
> > > >
> > >
> > > --
> > >
> > >
> >
> https://code.launchpad.net/~mrqtros/ubuntu-rssreader-app/feedlist/+merge/152463
> > > You are the owner of lp:~mrqtros/ubuntu-rssreader-app/feedlist.
> > >
> >
> > --
> >
> >
> https://code.launchpad.net/~mrqtros/ubuntu-rssreader-app/feedlist/+merge/152463
> > Your team Ubuntu RSS Feed Reader Developers is subscribed to branch
> > lp:ubuntu-rssreader-app.
> >
>
> --
>
> https://code.launchpad.net/~mrqtros/ubuntu-rssreader-app/feedlist/+merge/152463
>
> You are the owner of lp:~mrqtros/ubuntu-rssreader-app/feedlist.
>
>
>
>
>
> https://code.launchpad.net/~mrqtros/ubuntu-rssreader-app/feedlist/+merge/152463
> Your team Ubuntu RSS Feed Reader Developers is subscribed to branch
> lp:ubuntu-rssreader-app.
>