#31 ✓resolved
thynctank

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

  • Ed Finkler

    Ed Finkler April 14th, 2010 @ 08:17 PM

    • Milestone set to Version 0.10
  • Mark Armendariz

    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

    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

    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

    alab May 9th, 2010 @ 03:30 AM

    In git spaz-desktop-air master does not remember marked as read on RT, nor DM tweets.

  • alab

    alab May 9th, 2010 @ 03:32 AM

    Neither does Mark on the "Tweet's you've sent" View.

  • Ed Finkler

    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

    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

    Ed Finkler May 10th, 2010 @ 01:11 PM

    • Tag set to db, sqlite
  • Ed Finkler
  • Ed Finkler
  • Ed Finkler

    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.

New-ticket 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

Shared Ticket Bins

People watching this ticket

Attachments

Tags

Pages