Homec4science

Conpherence - fix JS and add code to repair old bad data

Authored by Bob Trahan <bob.trahan@gmail.com> on Apr 8 2013, 22:41.

Description

Conpherence - fix JS and add code to repair old bad data

Summary:
the JS is fragile with respect to the tokenizer coming in from the people widget. make sure to always try to load this up. Note this generally needs to get cleaned up where the server should only send down the *exact* bits the client needs. This is all TODO as part of getting this on mobile perfectly. Also note this fragility exists still in that you can break conpherence by clicking quickly before the initial tokenizer load loads.

For old bad data, at some point we weren't updating participation as well as we do today in the editor class. the result is with the migration and code change some conversation participants have bad "last seen message" counts. the simplest case is the test user talking to themselves -- threads before the editor code fixes / changes will have the entire thread as unread for these folks. The other buggy case I saw was where the "last reply" to a thread wasn't being count. These issues showed up for threads February and older which is old.

Test Plan: edited conpherence meta data and no JS bugs. pontificated and no JS bugs. added a person and no JS bugs.

Reviewers: chad, epriestley

Reviewed By: epriestley

CC: aran, Korvin

Differential Revision: https://secure.phabricator.com/D5622

Details

Committed
Bob Trahan <bob.trahan@gmail.com>Apr 8 2013, 22:41
Pushed
aubortJan 31 2017, 17:16
Parents
rPH73c49775c12b: Displaying links instead of dumping raw remarkup in Phriction mails
Branches
Unknown
Tags
Unknown

Event Timeline

Bob Trahan <bob.trahan@gmail.com> committed rPHe07077ae4e75: Conpherence - fix JS and add code to repair old bad data (authored by Bob Trahan <bob.trahan@gmail.com>).Apr 8 2013, 22:41