search

department-of-veterans-affairs / va-mobile-app

"If VA were a company, it would have a flagship mobile app."
https://department-of-veterans-affairs.github.io/va-mobile-app/
17 stars 2 forks source link

BUG - sev-2 - All - Messages: can't send messages from draft (only new) #8689

Closed TKDickson closed 5 months ago

TKDickson commented 5 months ago

What happened?

When attempting to open a draft and send the messagee, I can't. The UI in the app is just a generic error snackbar, but the API responses are giving an error back that it doesn't have a recipient (it does, so there's something else afoot).

See the charles proxy section for the reponses I got for +226, with the second draft in the list of drafts (Subject: Test: Test Inquiry, 1pm timestamp).

Have reproduced on the 5.21 develop build as well as the HSP branch.

Specs:

Steps to Reproduce

Desired behavior

Acceptance Criteria

Bug Severity - BE SURE TO ADD THE SEVERITY LABEL

See [Bug Tracking](https://department-of-veterans-affairs.github.io/va-mobile-app/docs/QA#issue-severity) for details on severity levels

Linked to Story

Screen shot(s) and additional information

Full JSON response for services related to issue (expand/collapse) `mobile/v0/messaging/health/messages/3620826` HTTP/1.1 200 OK Date: Tue, 21 May 2024 20:07:52 GMT Content-Type: application/json; charset=utf-8 Etag: W/"7b0cd6334470726f3274935ff5dd14f9" Referrer-Policy: strict-origin-when-cross-origin Vary: Accept, Origin X-Content-Type-Options: nosniff X-Custom-Response-Header: eks-staging X-Frame-Options: SAMEORIGIN X-Git-Sha: a12ca244559ed07b071c8ed46c917586b7d54c55 X-Github-Repository: https://github.com/department-of-veterans-affairs/vets-api X-Permitted-Cross-Domain-Policies: none X-Request-Id: 52f4b9f3-ac08-4d51-964f-fa9fa7609a92 X-Runtime: 0.438598 X-Xss-Protection: 0 Strict-Transport-Security: max-age=31536000; includeSubDomains; preload X-Frame-Options: SAMEORIGIN Cache-Control: no-cache, no-store Pragma: no-cache Content-Encoding: gzip Transfer-Encoding: chunked Connection: keep-alive {"data":{"id":"3620826","type":"messages","attributes":{"messageId":3620826,"category":"TEST_RESULTS","subject":"Test Inquiry","body":"Test","attachment":false,"sentDate":null,"senderId":873433,"senderName":"WEAVER, JOHNNIE LEONARD","recipientId":1766073,"recipientName":"VA Flagship mobile applications interface 1_DAYT29","readReceipt":null,"triageGroupName":"VA Flagship mobile applications interface 1_DAYT29","proxySenderName":null},"relationships":{"attachments":{"data":[]}},"links":{"self":"https://staging-api.va.gov/mobile/v0/messaging/health/messages/3620826"}},"meta":{"userInTriageTeam?":true}} POST to mobile `v0/messaging/health/messages` { "body": "Test", "draft_id": 3620826, "category": "TEST_RESULTS" } response back from `mobile v0/messaging/health/messages` HTTP/1.1 422 Unprocessable Entity Date: Tue, 21 May 2024 20:07:57 GMT Content-Type: application/json; charset=utf-8 Content-Length: 170 Referrer-Policy: strict-origin-when-cross-origin Vary: Accept, Origin X-Content-Type-Options: nosniff X-Custom-Response-Header: eks-staging X-Frame-Options: SAMEORIGIN X-Git-Sha: a12ca244559ed07b071c8ed46c917586b7d54c55 X-Github-Repository: https://github.com/department-of-veterans-affairs/vets-api X-Permitted-Cross-Domain-Policies: none X-Request-Id: 98ea0fbe-bcd5-4cee-a158-4fce381bd964 X-Runtime: 0.029036 X-Xss-Protection: 0 Strict-Transport-Security: max-age=31536000; includeSubDomains; preload X-Frame-Options: SAMEORIGIN Connection: keep-alive {"errors":[{"title":"Recipient can't be blank","detail":"recipient-id - can't be blank","code":"100","source":{"pointer":"data/attributes/recipient-id"},"status":"422"}]}

Ticket Checklist

TKDickson commented 5 months ago

I'm also seeing this in the 2.28 RC build

DJUltraTom commented 5 months ago

the on demand build fix looks good. found: 8691 while verifying, likely unrelated.

DJUltraTom commented 5 months ago

spot checked on RC build, looks good! closing