Sync bug: contextIndex overwritten

[raineorshine]

Sometimes a new subthought will disappear unexpectedly.

When the subthought is added manually, all of its descendants are populated, indicating that the thoughts are being saved initially but the contextIndex entry is being overwritten.

• How are deletions differentiated from past states?
  • Until we have write receipts or history, they aren’t. We have to use timestamps.
• Where and how are timestamps utilized?
  • Ignored in loadLocalState
    • Should be okay since only initialState exists and the remote state is deferred.
  • updateState (requires migration to add timestamp to contextIndex)
• Does it get overwritten by the local store or remote?
  • Must be in the remote. There is nothing for local store to overwrite.
• Test A: Two browser tabs logged in.
  • Unable to reproduce
• Test B: Try in various combinations.
  1. Two browser tabs logged in
  2. Go offline in tab X
  3. Make edits in tab X
  4. Reconnect in tab Y

[raineorshine]

See if this is fixed by #230