Lighthouse has a new layout. Prefer the old one? Return to the old layout, and switch back any time from the link at the top of each page.

Remember read/unread status of tweets

#31

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)

Reported by thynctank · March 21st, 2010 @ 06:31 AM

State: resolved
Milestone: Version 0.10
Assigned to: Ed Finkler Ed Finkler

Activity

  1. Ed Finkler
    Ed Finkler
    • Milestone set to Version 0.10

    April 14th, 2010 @ 08:17 PM

  2. Mark Armendariz
    Mark Armendariz

    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)

    April 24th, 2010 @ 12:30 AM

  3. Ed Finkler
    Ed Finkler

    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!

    April 24th, 2010 @ 12:48 PM

  4. Ed Finkler
    Ed Finkler

    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

    May 8th, 2010 @ 11:47 AM

  5. alab
    alab

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

    May 9th, 2010 @ 03:30 AM

  6. alab
    alab

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

    May 9th, 2010 @ 03:32 AM

  7. Ed Finkler
    Ed Finkler

    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.

    May 9th, 2010 @ 07:31 PM

  8. Ed Finkler
    Ed Finkler
    • 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.

    May 10th, 2010 @ 01:10 PM

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

    May 10th, 2010 @ 01:11 PM

  10. Ed Finkler
  11. Ed Finkler
  12. Ed Finkler
    Ed Finkler
    • State changed from open to resolved
    • Milestone order changed from 0 to 0

    December 3rd, 2010 @ 10:11 AM

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.