Skip to content
This repository has been archived by the owner on Jul 31, 2020. It is now read-only.

Reflect updated record when resolving UPDATE in order to propagate UPDATE record to sync chain #280

Closed
wants to merge 1 commit into from

Conversation

darkdh
Copy link
Member

@darkdh darkdh commented Feb 4, 2019

fix #231

Copy link
Member

@SergeyZhukovsky SergeyZhukovsky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Won't we reapply records on client side that were not really changed with that fix?

@darkdh
Copy link
Member Author

darkdh commented Feb 4, 2019

It turns out to be the bug in brave-core and I will link the brave-core PR later.

@darkdh
Copy link
Member Author

darkdh commented Feb 4, 2019

brave/brave-core#1572 and it is sometimes affected by
#281

@darkdh darkdh deleted the record-update branch May 13, 2019 18:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

objects order update are ignored by sync library
2 participants