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

Properly pass formContext to SchemaField #3040

Merged

Conversation

heath-freenome
Copy link
Member

@heath-freenome heath-freenome commented Aug 23, 2022

Reasons for making this change

Fixes #2394, #2274

  • Updated ArrayField, MultiSchemaField, ObjectField and SchemaField to pass formContext down the hierarchy properly
  • Updated tests to validate formContext is properly passed to SchemaField
  • Updated the CHANGELOG.md file with the fix mentioned

Checklist

  • I'm updating documentation
  • I'm adding or updating code
    • I've added and/or updated tests. I've run npm run test:update to update snapshots, if needed.
    • I've updated docs if needed
    • I've updated the changelog with a description of the PR
  • I'm adding a new feature
    • I've updated the playground with an example use of the feature

- Updated `ArrayField`, `MultiSchemaField`, `ObjectField` and `SchemaField` to pass `formContext` down the hierarchy properly
- Updated tests to validate `formContext` is properly passed to `SchemaField`
- Updated the `CHANGELOG.md` file with the fix mentioned
@heath-freenome heath-freenome force-pushed the pass-formContext-properly branch from aee8f12 to 8654a61 Compare August 24, 2022 15:07
@heath-freenome heath-freenome merged commit 15c011d into rjsf-team:rjsf-v5 Aug 25, 2022
@heath-freenome heath-freenome deleted the pass-formContext-properly branch August 25, 2022 16:54
heath-freenome added a commit to heath-freenome/react-jsonschema-form that referenced this pull request Aug 27, 2022
* Properly pass formContext to SchemaField
- Updated `ArrayField`, `MultiSchemaField`, `ObjectField` and `SchemaField` to pass `formContext` down the hierarchy properly
- Updated tests to validate `formContext` is properly passed to `SchemaField`
- Updated the `CHANGELOG.md` file with the fix mentioned

* Update packages/core/test/allOf_test.js
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.

2 participants