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

fix(core): skip cross dataset validation on mutations #3846

Merged

Conversation

sjelfull
Copy link
Member

@sjelfull sjelfull commented Nov 8, 2022

Description

This change disables cross dataset validation on mutations in the studio. This is mainly done to prevent the studio crashing because the mutation fails when 1) There is a cross dataset in the document being mutated 2) The document referenced has been deleted in the target dataset.

What to review

  • That a document with a cross dataset reference where the document has been deleted won't crash the studio anymore

Notes for release

  • Fix a edge case where a cross dataset reference to a deleted document would prevent you changing the document with the reference

@sjelfull sjelfull added the v3 label Nov 8, 2022
@sjelfull sjelfull requested a review from bjoerge November 8, 2022 12:50
@sjelfull sjelfull self-assigned this Nov 8, 2022
@vercel
Copy link

vercel bot commented Nov 8, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated
test-studio ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Nov 9, 2022 at 10:14AM (UTC)
1 Ignored Deployment
Name Status Preview Comments Updated
studio-workshop ⬜️ Ignored (Inspect) Nov 9, 2022 at 10:14AM (UTC)

Signed-off-by: Fred Carlsen <fred@sjelfull.no>
@sjelfull sjelfull force-pushed the bug/sc-26727/studio-crashes-every-time-you-edit-cdr-document branch from b9285dd to 3707e18 Compare November 9, 2022 10:08
@sjelfull sjelfull merged commit 966f66a into v3 Nov 9, 2022
@sjelfull sjelfull deleted the bug/sc-26727/studio-crashes-every-time-you-edit-cdr-document branch November 9, 2022 11:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants