App icon indicating copy to clipboard operation
App copied to clipboard

Android Hybrid - Login - Cursor displayed at the beggining of Email when trying to re login

Open lanitochka17 opened this issue 1 year ago • 3 comments

If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!


Version Number: 9.0.56-2 Reproducible in staging?: Y Reproducible in production?: Y If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: Y If this was caught during regression testing, add the test name, ID and link from TestRail: N/A Issue reported by: Applause - Internal Team

Action Performed:

  1. Open the Expensify app
  2. Log out of the account
  3. Tap on "Continue with Email"
  4. Verify that the cursor in Email input, is displayed at the end of the last used Email

Expected Result:

When trying to re log in, the cursor in Email input should be displayed at the end of the last used Email

Actual Result:

After logging out and trying to re login, the cursor in Email input is displayed at the beggining of the last used Email

Workaround:

Unknown

Platforms:

Which of our officially supported platforms is this issue occurring on?

  • [ ] Android: Standalone
  • [x] Android: HybridApp
  • [ ] Android: mWeb Chrome
  • [ ] iOS: Standalone
  • [ ] iOS: HybridApp
  • [ ] iOS: mWeb Safari
  • [ ] MacOS: Chrome / Safari
  • [ ] MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

https://github.com/user-attachments/assets/09572b28-f84a-40a3-8ef4-d5383e1d815f

View all open jobs on GitHub

lanitochka17 avatar Nov 01 '24 16:11 lanitochka17

Triggered auto assignment to @abekkala (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.

melvin-bot[bot] avatar Nov 01 '24 16:11 melvin-bot[bot]

@abekkala FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

lanitochka17 avatar Nov 01 '24 16:11 lanitochka17

Not quite sure if it’s related, but I worked on a similar issue https://github.com/Expensify/App/issues/50055.

QichenZhu avatar Nov 02 '24 03:11 QichenZhu

@abekkala Whoops! This issue is 2 days overdue. Let's get this updated quick!

melvin-bot[bot] avatar Nov 04 '24 18:11 melvin-bot[bot]

Looks like there is discusson in #50055 on next steps

abekkala avatar Nov 06 '24 19:11 abekkala

@abekkala Eep! 4 days overdue now. Issues have feelings too...

melvin-bot[bot] avatar Nov 12 '24 09:11 melvin-bot[bot]

@QichenZhu would the fix for https://github.com/Expensify/App/issues/50055 resolve this issue?

abekkala avatar Nov 12 '24 20:11 abekkala

would the fix for https://github.com/Expensify/App/issues/50055 resolve this issue?

@abekkala Sorry, I can't answer this question as I don't have access to the HybridApp repo.

QichenZhu avatar Nov 13 '24 02:11 QichenZhu

Job added to Upwork: https://www.upwork.com/jobs/~021857152364358853345

melvin-bot[bot] avatar Nov 14 '24 20:11 melvin-bot[bot]

Triggered auto assignment to Contributor-plus team member for initial proposal review - @mollfpr (External)

melvin-bot[bot] avatar Nov 14 '24 20:11 melvin-bot[bot]

@mollfpr I think we should hold this one on https://github.com/Expensify/App/issues/50055

abekkala avatar Nov 14 '24 20:11 abekkala

@abekkala, @mollfpr Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

melvin-bot[bot] avatar Nov 18 '24 09:11 melvin-bot[bot]

The #50055 will close soon and the PR for the RN patch is ready for review. We can test again after the PR merged.

mollfpr avatar Nov 19 '24 04:11 mollfpr

@abekkala, @mollfpr Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

melvin-bot[bot] avatar Nov 22 '24 09:11 melvin-bot[bot]

@abekkala, @mollfpr 6 days overdue. This is scarier than being forced to listen to Vogon poetry!

melvin-bot[bot] avatar Nov 26 '24 09:11 melvin-bot[bot]

The RN patch is merged and hit production. @abekkala We can ask QA to test it again.

mollfpr avatar Nov 26 '24 13:11 mollfpr

@lanitochka17 can this be retested please?

abekkala avatar Nov 26 '24 21:11 abekkala

Issue still reproducible Cursor is still displayed at the beginning of the Email when trying to log in again.

https://github.com/user-attachments/assets/c0a14524-9e83-4382-bffa-d5e94fb105e4

lanitochka17 avatar Nov 27 '24 14:11 lanitochka17

This seems low value. I think we should close it.

muttmuure avatar Nov 29 '24 17:11 muttmuure