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

Rename forms api module to simple forms api #13035

Merged
merged 0 commits into from Jun 20, 2023
Merged

Conversation

gmrabian
Copy link
Contributor

@gmrabian gmrabian commented Jun 20, 2023

The Lighthouse team has an API named Forms API so they asked us to rename ours. Since our vets-website folder is called simple-forms we've chosen to rename from forms_api to simple_forms_api

For reference: Original PR adding forms_api module

Summary

Related issue(s)

Testing done

  • Ran all associated tests locally and they passed
Screenshot 2023-06-20 at 11 08 50 AM
  • Used postman to test multiple requests locally and they worked as expected

  • Since there is no logic change the expected functionality is that vets-api starts up as expected without issues

  • Updated frontend forms that submit to this endpoint and tested locally that they submit correctly

Screenshots

Forms 26-4555, 21-4142, and 21-10210 submitting locally without issues to the new endpoint
Screenshot 2023-06-20 at 11 44 05 AM
Screenshot 2023-06-20 at 11 45 29 AM
Screenshot 2023-06-20 at 11 47 16 AM

What areas of the site does it impact?

A variety of forms that are non-prod

Associated PR to update those submission endpoints

Acceptance criteria

  • I fixed|updated|added unit tests and integration tests for each feature (if applicable).
  • No error nor warning in the console.
  • Events are being sent to the appropriate logging solution
  • No sensitive information (i.e. PII/credentials/internal URLs/etc.) is captured in logging, hardcoded, or specs

@gmrabian
Copy link
Contributor Author

Looks like the failing tests are the same ones failing in master. Searching simple_forms_api yielded no results in the error log and those tests are ok locally.

@gmrabian gmrabian requested a review from ndsprinkle June 20, 2023 16:36
@va-vsp-bot va-vsp-bot requested a deployment to rename_forms_api_module/main/main June 20, 2023 17:09 In progress
Copy link
Contributor

@ryan-mcneil ryan-mcneil left a comment

Choose a reason for hiding this comment

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

👍

@gmrabian gmrabian merged commit 9dce776 into master Jun 20, 2023
9 checks passed
@gmrabian gmrabian deleted the rename_forms_api_module branch June 20, 2023 18:02
ryan-mcneil pushed a commit that referenced this pull request Dec 11, 2023
* arbitrary line change

* rename forms_api to simple_forms_api

* cleanup changes

* undo db schema change

* linting

* linting
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants