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.

Sign up for GitHub

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

Jump to bottom

Allow submit for form 21p-0847 #24520

Merged
merged 1 commit into from
Jun 12, 2023
Merged

Allow submit for form 21p-0847 #24520

merged 1 commit into from
Jun 12, 2023

Conversation

gmrabian
Copy link
Contributor

Need vets-api PR adding form to be merged first

Summary

  • Remove mock submit and allow full end-to-end submission
  • Add shared transformForSubmit to config so that we add the form_number to the payload

Testing done

  • Tested locally with vets-api running

What areas of the site does it impact?

21P-0847

@gmrabian gmrabian marked this pull request as ready for review June 12, 2023 17:52
@gmrabian gmrabian requested a review from a team as a code owner June 12, 2023 17:52
@gmrabian gmrabian merged commit 7aefdde into main Jun 12, 2023
@gmrabian gmrabian deleted the simple-forms/allow-submit-0847 branch June 12, 2023 18:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rhasselle-oddball rhasselle-oddball rhasselle-oddball approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@gmrabian @rhasselle-oddball