Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

RemoveRecord when part of has many relationship should not clobber the whole relationship in IndexedDB source #608

Merged
merged 1 commit into from Mar 17, 2019

Conversation

tchak
Copy link
Member

@tchak tchak commented Mar 15, 2019

IndexedDB source clobber parent record relationship when one of the members of the given relationship is removed. I added a record-cache and local-storage tests to show that the problem is scoped to IndexedDB source.

@tchak tchak force-pushed the remove-record-clobber-persisted-source branch from 624eb72 to 311c6a7 Compare March 17, 2019 13:09
@tchak tchak changed the title [FAILING] #push - removeRecord when part of has many relationship RemoveRecord when part of has many relationship should not clobber the whole relationship Mar 17, 2019
@tchak tchak changed the title RemoveRecord when part of has many relationship should not clobber the whole relationship RemoveRecord when part of has many relationship should not clobber the whole relationship in IndexedDB source Mar 17, 2019
@tchak tchak force-pushed the remove-record-clobber-persisted-source branch from 311c6a7 to a484268 Compare March 17, 2019 13:18
@tchak tchak force-pushed the remove-record-clobber-persisted-source branch from a484268 to b190e34 Compare March 17, 2019 14:15
@dgeb dgeb merged commit 8f98f49 into orbitjs:master Mar 17, 2019
@dgeb
Copy link
Member

dgeb commented Mar 17, 2019

@tchak fantastic - thanks for the quick fix!

@dgeb dgeb added the bug label Mar 17, 2019
@tchak tchak deleted the remove-record-clobber-persisted-source branch May 30, 2019 11:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants