Review Request: Don't create another timeline on every timeout refresh in Remember The Milk plasmoid

Alexey Chernov 4ernov at gmail.com
Sat Apr 23 21:13:58 CEST 2011


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/101212/
-----------------------------------------------------------

Review request for Plasma.


Summary
-------

Currently RTM plasmoid creates new timeline on every refresh triggered by timeout (usually 5 minutes) of any action with tasks. Timeline is something like user session and every user action can be undone while still staying in the same timeline. I think, creating a new timeline on every refresh isn't so wise, especially as it triggeres the massive update of all the tasks and lists.


Diffs
-----

  dataengines/rememberthemilk/taskssource.cpp 06d354f9a6facfe8dc1ba42683078ab02ff39900 

Diff: http://git.reviewboard.kde.org/r/101212/diff


Testing
-------

The comment to the line of code says: "Allow us to get new information that people entered on the website or in another api". I tested the version without the line quite deeply performing task edits in both RTM web service and plasmoid and they were always synchronized with each other correctly (for plasmoid - as current timeout expired). So I see no reason in this.


Thanks,

Alexey

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/plasma-devel/attachments/20110423/0059b1ac/attachment-0001.htm 


More information about the Plasma-devel mailing list