-
Notifications
You must be signed in to change notification settings - Fork 62
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
API-39748-add-index-to-error-collection-messages #18333
API-39748-add-index-to-error-collection-messages #18333
Conversation
'returned from the /service-branches endpoint of the Benefits ' \ | ||
'Reference Data API.' | ||
'Reference Data API.' \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Obviously not going to hold this up, total nit, but I think the \
is not needed here? Did not appear to change the error return for any reason when removing it, again, obviously, total nit.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The error returns look great on this.
* master: case9 logic fixed (#18362) API-39748-add-index-to-error-collection-messages (#18333) Use random value in Simple Forms PDF stamping file paths (#18361) Add `find_dependents_by_ptcpnt_id` to local bgs (#18303) remove request_ping defs (#18353) API-38827-update-validations-intl-postal-code (#18323) [Document Upload Failure] Email veteran on Form 0781/Form 0781a upload retry exhaustion (#18206) add feature flags for meb and 5490 (#18328) Handle 500 responses from VA Profile v3 Service for v0/profile/contacts endpoint (#18192) Resolve MethodLength Rubocop Disables - Part 3 (#18331) Resolve MethodLength Rubocop Disables - Part BGS::Form674 (#18326)
Summary
Related issue(s)
Testing done
Request_body:
What areas of the site does it impact?
Acceptance criteria