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

Remove AJV usage from combinator mappers #2371

Open
lucas-koehler opened this issue Aug 28, 2024 · 0 comments
Open

Remove AJV usage from combinator mappers #2371

lucas-koehler opened this issue Aug 28, 2024 · 0 comments
Assignees
Labels
Milestone

Comments

@lucas-koehler
Copy link
Contributor

lucas-koehler commented Aug 28, 2024

Direct AJV usage in renderers (i.e. oneof renderer to determine applicable sub schema) should be removed completely as this is a hacky attempt to select the suitable subschema.
In general, we want to get rid of mandatory ajv usage in general.

For combinator renderers, AJV is invoked in mapStateToCombinatorRendererProps. This should be re-written to use a heuristic instead:

  • If there is an id/type/kind attribute: use to select initial entry. See if we can match this against a const in the schema. Otherwise try with first string and/or number property.
  • id property should be configurable via a custom JSON Schema attribute (we already use a custom i18n attribute). The new attribute should start with x-jsf-. I.e. x-jsf-type-property
  • allOf: Investigate whether calling mapStateToCombinatorRendererProps in mapStateToAllOfProps is necessary
@lucas-koehler lucas-koehler self-assigned this Aug 28, 2024
@sdirix sdirix added this to the 4.0 milestone Sep 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants