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: Optional schema #14385

Merged
merged 1 commit into from
Jan 9, 2025
Merged

fix: Optional schema #14385

merged 1 commit into from
Jan 9, 2025

Conversation

lassopicasso
Copy link
Contributor

@lassopicasso lassopicasso commented Jan 9, 2025

Description

Made the Schema property optional to accommodate state-less apps that may not have this configured. Having a required Schema would cause crashes in the Studio when absent.

Ref. https://github.com/Altinn/altinn-support-private/issues/3872

Related Issue(s)

  • #{issue number}

Verification

  • Your code builds clean without any errors or warnings
  • Manual testing done (required)
  • Relevant automated test added (if you find this hard, leave it and we'll help out)

Documentation

  • User documentation is updated with a separate linked PR in altinn-studio-docs. (if applicable)

@github-actions github-actions bot added solution/studio/designer Issues related to the Altinn Studio Designer solution. backend labels Jan 9, 2025
@lassopicasso lassopicasso changed the title Optional schema fix: Optional schema Jan 9, 2025
@lassopicasso lassopicasso marked this pull request as ready for review January 9, 2025 12:22
@lassopicasso lassopicasso merged commit e2ab849 into main Jan 9, 2025
17 of 18 checks passed
@lassopicasso lassopicasso deleted the make-schema-optional branch January 9, 2025 12:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

3 participants