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

JsonEditor Validation Broken in 47.1.0 #1037

Open
crimson-cata1yst opened this issue Apr 20, 2024 · 2 comments · May be fixed by #1038
Open

JsonEditor Validation Broken in 47.1.0 #1037

crimson-cata1yst opened this issue Apr 20, 2024 · 2 comments · May be fixed by #1038

Comments

@crimson-cata1yst
Copy link

crimson-cata1yst commented Apr 20, 2024

Due to the upgrade to Ajv 8.x.x with the Angular 17 upgrades, some breaking changes to how Ajv classes and error validation works was not accounted for and has left the JsonEditor component validation in a broken state

Ajv 8.0.0 Changelog with breaking changes
Ajv Migration from 6.x to 8.x

I've narrowed down the main culprit to dataPath now called 'instancePath' in Ajv. Though after fixing it locally, there now there are errors about $meta not existing for the strict schema when working through the bug; so there are some other underlying things that must also need fixed.

Meta issue error:

image

Steps to Reproduce: Build and run the latest ngx-ui demo, navigate to the JsonEditor and try to trigger validation errors. Validation never fires and the inputs never highlight. Open web debugger and see underlying Ajv errors.

@crimson-cata1yst
Copy link
Author

Resolved in linked PR

@crimson-cata1yst
Copy link
Author

@marjan-georgiev @Hypercubed @haeminn How can I get this attached PR reviewed? I would rather not publish a forked version of this repo for my team to use but I'm going to soon have to. :(

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 a pull request may close this issue.

1 participant