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(base): skip cross dataset validation on mutations #3856

Merged

Conversation

sjelfull
Copy link
Member

@sjelfull sjelfull commented Nov 9, 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

This is a port from v3 of PR #3846

Signed-off-by: Fred Carlsen <fred@sjelfull.no>
@sjelfull sjelfull requested a review from bjoerge November 9, 2022 14:06
@sjelfull sjelfull self-assigned this Nov 9, 2022
@vercel
Copy link

vercel bot commented Nov 9, 2022

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

Name Status Preview Comments Updated
studio-workshop ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Nov 9, 2022 at 2:10PM (UTC)
test-studio ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Nov 9, 2022 at 2:10PM (UTC)

@sjelfull sjelfull merged commit d746aae into next Nov 14, 2022
@sjelfull sjelfull deleted the bug/sc-26727/studio-crashes-every-time-you-edit-cdr-document branch November 14, 2022 08:53
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