-
Notifications
You must be signed in to change notification settings - Fork 113
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
Comments
5 tasks
Resolved in linked PR |
@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
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:
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.
The text was updated successfully, but these errors were encountered: