Skip to content
This repository has been archived by the owner on Feb 27, 2024. It is now read-only.

Fixing 500 error on GF Form Submission #606

Merged
merged 6 commits into from
Aug 31, 2021

Conversation

sloyer
Copy link
Contributor

@sloyer sloyer commented Aug 25, 2021

Closes #569

Description

Currently if a user selects an email field it will 500 error out up submission.
This fixes that issue allowing a user to select the email field and having the submission go through and confirm submission.

Verification

How will a stakeholder test this?

  1. Create a GF form with a email field
  2. Fill out form (email)
  3. Submit.

@sloyer sloyer requested a review from gregrickaby as a code owner August 25, 2021 00:47
@vercel
Copy link

vercel bot commented Aug 25, 2021

@sloyer is attempting to deploy a commit to the WebDevStudios Team on Vercel.

A member of the Team first needs to authorize it.

@vercel
Copy link

vercel bot commented Aug 27, 2021

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/GKVCCSoh5RAnWxvbJnREGGRiyNGS
✅ Preview: https://nextjs-wordpress-starter-git-fork-alopexid-3c98c4-webdevstudios.vercel.app

Copy link
Contributor

@gregrickaby gregrickaby left a comment

Choose a reason for hiding this comment

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

@sloyer Thank you for the PR! There are some linting/formatting fixes that need to be taken care of before I can merge this in. Thanks!

@AlopexID
Copy link

@sloyer Thank you for the PR! There are some linting/formatting fixes that need to be taken care of before I can merge this in. Thanks!

Ran a lint fix on it, hope its good to go now, cheers!

@gregrickaby gregrickaby merged commit 37dfb71 into WebDevStudios:main Aug 31, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Form Fields do not come through Gravity Forms
3 participants