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

correct antd SelectWidget filter option check. fix PR #3377 #3919

Merged
merged 2 commits into from
Oct 28, 2023

Conversation

hextion
Copy link
Contributor

@hextion hextion commented Oct 24, 2023

Hello everyone

Filtering made in #3377 isn't working because Select.Option doesn't have label prop passed into.
So I fixed that

@hextion
Copy link
Contributor Author

hextion commented Oct 24, 2023

@nickgros review please

@heath-freenome
Copy link
Member

heath-freenome commented Oct 27, 2023

@hextion Can you please rebase and update the CHANGELOG.md with a description of your fix:

# 5.13.3

## @rjsf/antd

- <your fix description>

@heath-freenome
Copy link
Member

@hextion Can you please rebase and update the CHANGELOG.md with a description of your fix:

# 5.13.3

## @rjsf/antd

- <your fix description>

Nevermind, I'm pushing a fix today and will update the changelog

@heath-freenome heath-freenome merged commit c7ae432 into rjsf-team:main Oct 28, 2023
4 checks passed
heath-freenome added a commit to heath-freenome/react-jsonschema-form that referenced this pull request Oct 28, 2023
… items

Fixes rjsf-team#3924 by defaulting to an empty object for the `itemSchema` when undefined
- In `@rjsf/core` updated `ArrayField`'s `renderFixedArray()` to default the `itemSchema` to an empty object if it is falsy
  - Updated the tests for `ArrayField` to verify the fix
- Updated `CHANGELOG.md` accordingly
  - Also added entry for the fix for rjsf-team#3919
heath-freenome added a commit to heath-freenome/react-jsonschema-form that referenced this pull request Oct 28, 2023
… items

Fixes rjsf-team#3924 by defaulting to an empty object for the `itemSchema` when undefined
Fixes rjsf-team#3927 by fixing `getSnapshotBeforeUpdate()` added for the fix rjsf-team#1794
- In `@rjsf/core` updated `ArrayField`'s `renderFixedArray()` to default the `itemSchema` to an empty object if it is falsy
  - Updated the tests for `ArrayField` to verify the fix
  - Updated the `getSnapshotBeforeUpdate()` to diff the old and new props and to set the `shouldUpdate` prop if the old and new state differs
- Updated `CHANGELOG.md` accordingly
  - Also added entry for the fix for rjsf-team#3919
heath-freenome added a commit to heath-freenome/react-jsonschema-form that referenced this pull request Oct 28, 2023
… items

Fixes rjsf-team#3924 by defaulting to an empty object for the `itemSchema` when undefined
Fixes rjsf-team#3927 by fixing `getSnapshotBeforeUpdate()` added for the fix rjsf-team#1794
- In `@rjsf/core` updated `ArrayField`'s `renderFixedArray()` to default the `itemSchema` to an empty object if it is falsy
  - Updated the tests for `ArrayField` to verify the fix
  - Updated the `getSnapshotBeforeUpdate()` to diff the old and new props and to set the `shouldUpdate` prop if the old and new state differs
- Updated `CHANGELOG.md` accordingly
  - Also added entry for the fix for rjsf-team#3919
heath-freenome added a commit to heath-freenome/react-jsonschema-form that referenced this pull request Oct 28, 2023
… items

Fixes rjsf-team#3924 by defaulting to an empty object for the `itemSchema` when undefined
Fixes rjsf-team#3927 by fixing `getSnapshotBeforeUpdate()` added for the fix rjsf-team#1794
- In `@rjsf/core` updated `ArrayField`'s `renderFixedArray()` to default the `itemSchema` to an empty object if it is falsy
  - Updated the tests for `ArrayField` to verify the fix
  - Updated the `getSnapshotBeforeUpdate()` to diff the old and new props and to set the `shouldUpdate` prop if the old and new state differs
- Updated `CHANGELOG.md` accordingly
  - Also added entry for the fix for rjsf-team#3919
@hextion
Copy link
Contributor Author

hextion commented Oct 29, 2023

@heath-freenome sorry I was out of town so I couldn't do stuff. thank you for accepting PR, really appreciate it

@hextion hextion deleted the fix-antd-selectwidget-filter branch October 29, 2023 14:04
heath-freenome added a commit that referenced this pull request Oct 30, 2023
…3929)

Fixes #3924 by defaulting to an empty object for the `itemSchema` when undefined
Fixes #3927 by fixing `getSnapshotBeforeUpdate()` added for the fix #1794
- In `@rjsf/core` updated `ArrayField`'s `renderFixedArray()` to default the `itemSchema` to an empty object if it is falsy
  - Updated the tests for `ArrayField` to verify the fix
  - Updated the `getSnapshotBeforeUpdate()` to diff the old and new props and to set the `shouldUpdate` prop if the old and new state differs
- Updated `CHANGELOG.md` accordingly
  - Also added entry for the fix for #3919
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