Skip to content

Pass explicit undefined defaults to optional jsonforms#2597

Merged
lucas-koehler merged 1 commit into
eclipsesource:masterfrom
kchobantonov:fix/vue-optional-jsonforms-injection-warning
Jul 3, 2026
Merged

Pass explicit undefined defaults to optional jsonforms#2597
lucas-koehler merged 1 commit into
eclipsesource:masterfrom
kchobantonov:fix/vue-optional-jsonforms-injection-warning

Conversation

@kchobantonov

Copy link
Copy Markdown
Contributor

Pass explicit undefined defaults to optional jsonforms and dispatch injections so Vue does not warn when helper components are rendered outside a JsonForms provider.

…njections so Vue does not warn when helper components are rendered outside a JsonForms provider.
@netlify

netlify Bot commented Jun 16, 2026

Copy link
Copy Markdown

Deploy Preview for jsonforms-examples ready!

Name Link
🔨 Latest commit 3c772d2
🔍 Latest deploy log https://app.netlify.com/projects/jsonforms-examples/deploys/6a31d7d30bee0f0008069291
😎 Deploy Preview https://deploy-preview-2597--jsonforms-examples.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@kchobantonov

Copy link
Copy Markdown
Contributor Author

@sdirix please review

@coveralls

Copy link
Copy Markdown

Coverage Status

coverage: 83.776%. remained the same — kchobantonov:fix/vue-optional-jsonforms-injection-warning into eclipsesource:master

@lucas-koehler lucas-koehler left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for the contribution ❤️ The change makes sense to me and looks good :)

@lucas-koehler lucas-koehler merged commit 6db5bc8 into eclipsesource:master Jul 3, 2026
8 checks passed
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.

3 participants