Remember read/unread status of tweets
Reported by thynctank | March 21st, 2010 @ 06:31 AM | in Version 0.10
Upon quitting and restarting Spaz, I'm greeted every time with the new notification window, and all tweets in the timeline show as unread (i.e., not collapsed, in the Leopaz theme)
Comments and changes to this ticket
-
-
Mark Armendariz April 24th, 2010 @ 12:30 AM
I know this one's already assigned, but it seems Ed already has about 20 other things with higher priority assigned - I figure it's easier to review a diff and merge.
I'm not sure if this is the approach you'd want to take - if maybe it goes against any optimizations or whatnot, but here's a simple solution. I basically added a db query called
Spaz.DB.isRead(entryId), which returns boolean. Then, when parsing the timeline, it calls that query and sets a "read" parameter on the tweet (data) object, and finally in the template, it sets the html class to "new" or "read".And that's enough spaz for me tonight. It's been fun!
Mark (enobrev)
-
Ed Finkler April 24th, 2010 @ 12:48 PM
MARK!
This is great!
I think for now, this gets us there. Ultimately we're using a
schemaless lawnchair-based setup to store message data, but this may
be quite adequate for now. Small steps.One potential issue here is that messages aren't namespaced. If a user
switches between Twitter and StatusNet, for example, there could be
conflicts.for now I think I'm gonna patch this, though. Big thanks for stepping up!
-
Ed Finkler May 8th, 2010 @ 11:47 AM
This one is kinda working okay, but the synchronous connection is slowing us down a bit. we need to refactor the approach some before 0.10
-
alab May 9th, 2010 @ 03:30 AM
In git spaz-desktop-air master does not remember marked as read on RT, nor DM tweets.
-
-
Ed Finkler May 9th, 2010 @ 07:31 PM
Yes, the saving of read state is mucked up, and honestly it should
probably be almost entirely redone. not sure right now whether to
patch up what we have to redo it all. Probably the former. -
Ed Finkler May 10th, 2010 @ 01:10 PM
- State changed from “new” to “open”
at very least, the read state table needs to have an index on it to speed up reads considerably.
-
Ed Finkler May 10th, 2010 @ 01:11 PM
- Tag set to “db, sqlite”
-
-
-
Ed Finkler December 3rd, 2010 @ 10:11 AM
- State changed from “open” to “resolved”
- Milestone order changed from “0” to “0”
Please Sign in or create a free account to add a new ticket.
With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.
Create new ticket
Create your profile
Help contribute to this project by taking a few moments to create your personal profile. Create your profile ยป
The Spaz Desktop project
Ed Finkler
thynctank