-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
fix(#3233): fix rendering 0 when there are no errors. #3234
Conversation
@suyanhanx Can you please update your PR to fix the tests as well as update the |
@suyanhanx We are still waiting for a couple of changes so that we can merge your fix |
@heath-freenome Sorry to keep you waiting for so long. I've updated the snapshot and changelog. |
@suyanhanx How did you update the snapshot as yours seem to have broken the build. You can simply use |
@heath-freenome Well, I did this manually. Now I have used the command you suggested to update snapshots. Thanks for your suggestion. |
…team#3234) * fix(rjsf-team#3233): fix rendering 0 when there are no errors. * fix: update snapshot and changelog * fix: update snapshot Co-authored-by: Heath C <51679588+heath-freenome@users.noreply.github.com>
…team#3234) * fix(rjsf-team#3233): fix rendering 0 when there are no errors. * fix: update snapshot and changelog * fix: update snapshot Co-authored-by: Heath C <51679588+heath-freenome@users.noreply.github.com>
Reasons for making this change
fixes #3233
Checklist
npm run test:update
to update snapshots, if needed.