Skip to content
This repository is currently being migrated. It's locked while the migration is in progress.

[1115] Update swagger docs for Veterans Status Card API response#26348

Merged
kbog25 merged 15 commits intomasterfrom
1115-update-swagger
Feb 9, 2026
Merged

[1115] Update swagger docs for Veterans Status Card API response#26348
kbog25 merged 15 commits intomasterfrom
1115-update-swagger

Conversation

@kbog25
Copy link
Copy Markdown
Contributor

@kbog25 kbog25 commented Feb 4, 2026

Keep your PR as a Draft until it's ready for Platform review. A PR is ready for Platform review when it has a teammate approval and tests, linting, and settings checks pass CI. See these tips on how to avoid common delays in getting your PR merged.

Summary

  • This work is behind a feature toggle (flipper): NO
  • (Summarize the changes that have been made to the platform) Update swagger docs for Veterans Status Card API response
  • (If bug, how to reproduce)
  • (What is the solution, why is this the solution?)
  • (Which team do you work for, does your team own the maintenance of this component?) Mobile Feature Support
  • (If introducing a flipper, what is the success criteria being targeted?)

Related issue(s)

  • Link to ticket created in va.gov-team repo OR screenshot of Jira ticket if your team uses Jira 1115
  • Link to previous change of the code/bug (if applicable)
  • Link to epic if not included in ticket

Testing done

  • New code is covered by unit tests
  • Describe what the old behavior was prior to the change
  • Describe the steps required to verify your changes are working as expected. Exclusively stating 'Specs run' is NOT acceptable as appropriate testing
  • If this work is behind a flipper:
    • Tests need to be written for both the flipper on and flipper off scenarios. Docs.
    • What is the testing plan for rolling out the feature?

Screenshots

Note: Optional

What areas of the site does it impact?

(Describe what parts of the site are impacted andifcode touched other areas)

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
  • Documentation has been updated (link to documentation)
  • No sensitive information (i.e. PII/credentials/internal URLs/etc.) is captured in logging, hardcoded, or specs
  • Feature/bug has a monitor built into Datadog (if applicable)
  • If app impacted requires authentication, did you login to a local build and verify all authenticated routes work as expected
  • I added a screenshot of the developed feature

Requested Feedback

(OPTIONAL)What should the reviewers know in addition to the above. Is there anything specific you wish the reviewer to assist with. Do you have any concerns with this PR, why?

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Feb 4, 2026

✅ Static OpenAPI documentation has been automatically generated and committed from openapi.yaml.

Copy link
Copy Markdown
Contributor

@mtharmer-wc mtharmer-wc left a comment

Choose a reason for hiding this comment

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

Overall it looks great, just a few minor tweaks. Also, if you want, you can apply the same examples from VeteranStatusCard.yml to veteran_status_card.rb, using either:

property :full_name, example: 'John T Doe Jr'

or

property :full_name do
  key :example, 'John T Doe Jr`

It's up to you, but we do have some time and it might be a good idea to match the examples between different API docs.

Comment thread app/swagger/swagger/requests/veteran_status_cards.rb Outdated
Comment thread app/swagger/swagger/requests/veteran_status_cards.rb
Comment thread app/swagger/swagger/requests/veteran_status_cards.rb
Comment thread modules/mobile/docs/schemas/VeteranStatusCard.yml
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Feb 4, 2026

✅ Static OpenAPI documentation has been automatically generated and committed from openapi.yaml.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Feb 4, 2026

✅ Static OpenAPI documentation has been automatically generated and committed from openapi.yaml.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Feb 5, 2026

✅ Static OpenAPI documentation has been automatically generated and committed from openapi.yaml.

Copy link
Copy Markdown
Contributor

@mtharmer-wc mtharmer-wc left a comment

Choose a reason for hiding this comment

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

LGTM! 👍

@kbog25 kbog25 marked this pull request as ready for review February 5, 2026 20:16
@kbog25 kbog25 requested review from a team as code owners February 5, 2026 20:16
@kbog25 kbog25 requested a review from mmiddaugh February 5, 2026 20:16
@kbog25 kbog25 merged commit 7f1d10d into master Feb 9, 2026
43 checks passed
@kbog25 kbog25 deleted the 1115-update-swagger branch February 9, 2026 17:45
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.

4 participants