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 3- Android - Claims file upload warning message missing text on android #6445

Closed DJUltraTom closed 1 year ago

DJUltraTom commented 1 year ago

What happened?

On android the warning box before you upload a file or photo supporting a claim is missing text.

Specs:

Steps to Reproduce

navigate to "Benefits > Claims > Claims History > (select a claim with an upload available) > File request select the blue (open) file request and go through the flow to either take a picture or attach a file. Select the document type and check the required check box Tap submit file observe the "are you ready to submit your file?" messaging

Desired behavior

Information should be complete

Acceptance Criteria

TBD, but likely match the messaging that is present on iOS

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

Screenshot_20230802-172746.png

Full JSON response for services related to issue (expand/collapse)

Ticket Checklist

theodur commented 1 year ago

I reproduced the steps, but I'm not seeing any missing text or discrepancies between iOS and Android. Can you confirm whether you're still seeing this issue on the latest build @DJUltraTom?

Here's what I'm seeing on both platforms:   

jennb33 commented 1 year ago

@DJUltraTom this ticket needs your attention, as @rbontrager and @theodur can not reproduce the error. Moving this ticket to Sprint 80, for when you are back in office.

DJUltraTom commented 1 year ago

Able to reproduce still on my google pixel, screenshot of my text settings below Private Zenhub Image

theodur commented 1 year ago

I was able to reproduce the issue and opened a PR as a workaround. This is actually a bug in the react-native-action-sheet library. Seems like others are also experiencing the text cut off when bold text is enabled. There's no word on when this is expected to be fixed, but the workaround in my PR should be good enough for now

bischoffa commented 1 year ago

Adding Sprint 81 as the ticket was not closed out in 80. Carryover work.

DJUltraTom commented 1 year ago

verified issue no longer persists! Qa approves!

DJUltraTom commented 1 year ago

Validated looking good on the nightly QA build, closing as deployed