nextjs-wordpress-starter icon indicating copy to clipboard operation
nextjs-wordpress-starter copied to clipboard

Forms

Open gregrickaby opened this issue 2 years ago • 1 comments

Closes #879

Description

This PR was recreated from #925 off the new main branch. All credit to @ravewebdev

  • Adds the new Input wrapper component with some basic validation, styling, and stories
  • Refactors site forms (post comment, login, register) using Formik directly and the new Input component
  • Started refactoring GF component to use Formik directly with new Input component – this still needs more work & testing, though basic form submission is currently working Note:
  • Post-submission field handling may need tweaking
  • formValidationSchema needs to be tweaked/overhauled so as not to override field-level validation unnecessarily (ideally, all/most of that abstraction can be removed entirely)

Screenshot

screenshot

screenshot

screenshot

Verification

How will a stakeholder test this?

  1. gh pr checkout 947
  2. npm i --legacy-peer-deps
  3. Visit any blog post with a comment

gregrickaby avatar Feb 23 '22 14:02 gregrickaby

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/webdevstudios/nextjs-wordpress-starter/4DXTZgpDeocHqLxxK5G2P6SK67Kj
✅ Preview: https://nextjs-wordpress-starter-git-feature-879-formik-2-webdevstudios.vercel.app

vercel[bot] avatar Feb 23 '22 14:02 vercel[bot]