-
-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
🚧 Inject the default schemas properties when loading it #1357
Conversation
Current coverage is
|
There may still be an edge case that is not covered. |
Which one would that be? |
Querying the GET /schemas/ on system classes is empty, it should return the default schema no? |
There are tests for that which are passing so I guess it's working. Schemas API has it's own way of merging the defaults schemas. |
fine for me! |
this maximizes the compatibility with previously set CLP before migration where the Schema wouldn't have all the default keys set