Skip to content
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

[$500] iOS - Login - Paste menu does not point to the correct field on magic code page #32241

Closed
2 of 6 tasks
lanitochka17 opened this issue Nov 30, 2023 · 16 comments
Closed
2 of 6 tasks
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 Engineering Help Wanted Apply this label when an issue is open to proposals by contributors

Comments

@lanitochka17
Copy link

lanitochka17 commented Nov 30, 2023

If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!


Version Number: 1.4.5.7
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:

Action Performed:

  1. Open New Expensify app
  2. Enter any email to try to log in
  3. Copy the magic code
  4. Try to paste the magic code on any field that is not the first one

Expected Result:

The paste menu should point to the field the user is trying to paste the code

Actual Result:

The paste menu does not point to the correct field, is always pointing to the first field of the magic code

Workaround:

Unknown

Platforms:

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

  • Android: Native
  • Android: mWeb Chrome
  • iOS: Native
  • iOS: mWeb Safari
  • MacOS: Chrome / Safari
  • MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

Bug6295303_1701302822057.Bjxf4284_1_.mp4

View all open jobs on GitHub

@lanitochka17 lanitochka17 added the DeployBlockerCash This issue or pull request should block deployment label Nov 30, 2023
Copy link
Contributor

👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open `StagingDeployCash` deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

  1. Identify the pull request that introduced this issue and revert it.
  2. Find someone who can quickly fix the issue.
  3. Fix the issue yourself.

Copy link

melvin-bot bot commented Nov 30, 2023

Triggered auto assignment to @NikkiWines (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

@s-alves10
Copy link
Contributor

s-alves10 commented Nov 30, 2023

This looks like a regression of the PR #28711

@aimane-chnaif
Copy link
Contributor

you meant #28711?

@s-alves10

This comment was marked as duplicate.

@aimane-chnaif
Copy link
Contributor

It's already 2nd version of magic code input refactor.
It's better to fix quickly than revert.
cc: @wojtus7

@kosmydel
Copy link
Contributor

kosmydel commented Nov 30, 2023

Hey, I'm from SWM and I'm currently responsible for the magic code blur issue.

It looks like it was a conscious decision to leave it like this. It was pointed out here and approved here and here.

So in my opinion, it is not a deploy blocker.

cc @dangrous, @Santhosh-Sellavel

@mountiny mountiny added Daily KSv2 and removed DeployBlockerCash This issue or pull request should block deployment Hourly KSv2 labels Nov 30, 2023
@mountiny
Copy link
Contributor

Yeah I dont think this is a blocker, tested this and you can just fine paste it in if you focus on the first number

Assigning Santosh and Kuba in the meantime if we decide this needs to be fixed or not

@Santhosh-Sellavel
Copy link
Collaborator

This is a known issue. cc: @dangrous

@dangrous
Copy link
Contributor

Yeah I think we're okay with this. If we CAN find a solution that would be neat, but I think it's not a bug, just a shift from before.

@melvin-bot melvin-bot bot added the Overdue label Dec 4, 2023
@NikkiWines NikkiWines changed the title iOS - Login - Paste menu does not point to the correct field on magic code page [500$] iOS - Login - Paste menu does not point to the correct field on magic code page Dec 4, 2023
@NikkiWines NikkiWines added the Bug Something is broken. Auto assigns a BugZero manager. label Dec 4, 2023
Copy link

melvin-bot bot commented Dec 4, 2023

Triggered auto assignment to @joekaufmanexpensify (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

@melvin-bot melvin-bot bot removed the Overdue label Dec 4, 2023
Copy link

melvin-bot bot commented Dec 4, 2023

Bug0 Triage Checklist (Main S/O)

  • This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
  • This bug is not a duplicate report (check E/App issues and #expensify-bugs)
    • If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead
  • This bug is reproducible using the reproduction steps in the OP. S/O
    • If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
    • If the reproduction steps aren't clear and you determine the correct steps, please update the OP.
  • This issue is filled out as thoroughly and clearly as possible
    • Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.
  • I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync

@NikkiWines NikkiWines changed the title [500$] iOS - Login - Paste menu does not point to the correct field on magic code page [$500] iOS - Login - Paste menu does not point to the correct field on magic code page Dec 4, 2023

This comment was marked as off-topic.

@NikkiWines NikkiWines added the Help Wanted Apply this label when an issue is open to proposals by contributors label Dec 4, 2023
@NikkiWines
Copy link
Contributor

Unassigning @kosmydel since we determined this wasn't a regression from the previously mentioned PR and we're opening up this issue for proposals

@joekaufmanexpensify
Copy link
Contributor

I kinda think we should do nothing about this and close the issue. We auto-focus to the first field of the magic code after the user submits their email and moves to the magic code page.

I don't think it will be common for someone to actively then choose to click on a different one of the 6 fields and try to paste the code there. If they do, pasting the code beginning with where they focused doesn't seem wrong to me.

In an ideal world, it'd be cool if pasting the code to any one of the 6 fields auto adjusts the code into the correct place. But that doesn't seem like a bug fix IMO, it seems like a new feature. And not one that really feels like a priority. Personally, I think it makes sense to leave as is, and reevaluate if any actual customers run into an issue with this.

@joekaufmanexpensify
Copy link
Contributor

Closing. LMK if anyone disagrees!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 Engineering Help Wanted Apply this label when an issue is open to proposals by contributors
Projects
None yet
Development

No branches or pull requests

9 participants