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

feat: support error list to show at the top or bottom #3196

Merged

Conversation

jacqueswho
Copy link
Contributor

@jacqueswho jacqueswho commented Oct 12, 2022

Reasons for making this change

If this is related to existing tickets, include links to them as well. Use the syntax fixes #634

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

@jacqueswho jacqueswho linked an issue Oct 12, 2022 that may be closed by this pull request
@jacqueswho jacqueswho requested review from heath-freenome and nickgros and removed request for heath-freenome October 13, 2022 11:42
packages/core/src/components/Form.tsx Outdated Show resolved Hide resolved
docs/5.x upgrade guide.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
packages/core/test/Form_test.js Outdated Show resolved Hide resolved
packages/core/src/components/Form.tsx Outdated Show resolved Hide resolved
packages/core/src/components/Form.tsx Outdated Show resolved Hide resolved
packages/core/src/components/Form.tsx Outdated Show resolved Hide resolved
packages/core/src/components/Form.tsx Outdated Show resolved Hide resolved
packages/playground/src/app.jsx Outdated Show resolved Hide resolved
chore: cs check

fix: default to top

chore: cs-format

Update app.jsx

Update Form_test.js

Update CHANGELOG.md

Co-authored-by: Nick Grosenbacher <[email protected]>

Update docs/5.x upgrade guide.md

Co-authored-by: Nick Grosenbacher <[email protected]>

chore: changes

chore
@jacqueswho jacqueswho force-pushed the 634-support-showing-the-error-list-at-the-bottom-of-the-form branch from 309fee8 to 2bc3167 Compare October 17, 2022 18:50
Copy link
Member

@heath-freenome heath-freenome left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Still one minor little change, otherwise ready for merge

packages/playground/src/app.jsx Outdated Show resolved Hide resolved
heath-freenome
heath-freenome previously approved these changes Oct 17, 2022
@heath-freenome heath-freenome dismissed their stale review October 18, 2022 00:00

Because I want tests updated

Copy link
Member

@heath-freenome heath-freenome left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add some tests, see my comments above

CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
@jacqueswho jacqueswho merged commit 93c4971 into main Oct 26, 2022
@jacqueswho jacqueswho deleted the 634-support-showing-the-error-list-at-the-bottom-of-the-form branch October 26, 2022 06:24
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.

Support showing the error list at the bottom of the form
3 participants