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

[Fabric Rollout] iOS - Second keyboard opens when you click on the cursor in Request money #14372

Closed
1 task
dylanexpensify opened this issue Jan 17, 2023 · 21 comments
Assignees
Labels
Monthly KSv2 NewFeature Something to build that is a new item.

Comments

@dylanexpensify
Copy link
Contributor

Action Performed:

  1. Launch App
  2. Enter email. password
  3. Click the FAB button and select Request Money
  4. Verify that the big number pad (BNP) page appears next
  5. Tab on the cursor next to currency

Expected Result:

There should be only one keyboard

Actual Result:

Second keyboard opens when you click on the cursor in Request money

Workaround:

Unknown

Platforms:

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

  • IOS / Native

Version Number: 1.2.49.0

Reproducible in staging?: Yes

Reproducible in production?: No

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

Notes/Photos/Videos: Any additional supporting documentation

Bug5888070_cursor_request_0701.mp4

Issue reported by: Applause - Internal Team

@dylanexpensify dylanexpensify added Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels Jan 17, 2023
@melvin-bot
Copy link

melvin-bot bot commented Jan 17, 2023

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

@melvin-bot
Copy link

melvin-bot bot commented Jan 17, 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

@dylanexpensify
Copy link
Contributor Author

Coming from here. Context here!

@roryabraham roryabraham added NewFeature Something to build that is a new item. and removed Bug Something is broken. Auto assigns a BugZero manager. labels Jan 17, 2023
@j-piasecki
Copy link
Contributor

I'm working on this.

@j-piasecki
Copy link
Contributor

facebook/react-native#35878 should fix this issue upstream, I also added a patch for the time being here.

@JmillsExpensify JmillsExpensify changed the title iOS - Second keyboard opens when you click on the cursor in Request money [Fabric Rollout] iOS - Second keyboard opens when you click on the cursor in Request money Jan 18, 2023
@JmillsExpensify
Copy link

Thanks @j-piasecki! I'll create another issue out of band so that we don't forget to update things once the upstream fix is live.

@melvin-bot melvin-bot bot added the Overdue label Jan 20, 2023
@JmillsExpensify JmillsExpensify added Weekly KSv2 and removed Daily KSv2 labels Jan 21, 2023
@melvin-bot melvin-bot bot removed the Overdue label Jan 21, 2023
@JmillsExpensify
Copy link

Moved this to weekly since that's the speed this project is moving at.

@melvin-bot melvin-bot bot added the Overdue label Jan 30, 2023
@JmillsExpensify

This comment was marked as off-topic.

@melvin-bot melvin-bot bot removed the Overdue label Jan 31, 2023
@roryabraham

This comment was marked as off-topic.

@melvin-bot melvin-bot bot added the Overdue label Feb 8, 2023
@JmillsExpensify
Copy link

Sounds like we agreed on a GH project in the linked thread, though we still need to put that in motion. I'm pretty back up early this week, though, so unlikely I'll get back to this for a bit.

@melvin-bot melvin-bot bot removed the Overdue label Feb 14, 2023
@j-piasecki
Copy link
Contributor

@JmillsExpensify It seems like I cannot access the linked slack thread. Could you post the summary here or add me to the relevant channel?

@JmillsExpensify
Copy link

Oh sorry about that! Sure thing, we essentially decide that we were going to create a GH project that helps keep track of all our upstream fixes, forks, and what not so that we can better keep track of how to minimize what's specific to Expensify. So the project would have the following stages:

  • Reviewing
  • Merged in fork
  • Merged upstream
  • Released upstream and resolved

@roryabraham I'm going to go ahead and create this project, because I don't think one already exists. I believe we can also make it public for all too in this repo.

@melvin-bot melvin-bot bot removed the Overdue label Feb 27, 2023
@melvin-bot melvin-bot bot added the Overdue label Mar 7, 2023
@JmillsExpensify
Copy link

Same same.

@melvin-bot melvin-bot bot removed the Overdue label Mar 8, 2023
@melvin-bot melvin-bot bot added the Overdue label Mar 17, 2023
@JmillsExpensify
Copy link

Re-testing of Fabric is commencing soon.

@melvin-bot melvin-bot bot removed the Overdue label Mar 29, 2023
@melvin-bot melvin-bot bot added the Overdue label Apr 6, 2023
@JmillsExpensify
Copy link

Same as above.

@melvin-bot melvin-bot bot removed the Overdue label Apr 12, 2023
@melvin-bot melvin-bot bot added the Overdue label Apr 20, 2023
@JmillsExpensify
Copy link

Same same. Fabric roll out and testing is on-going.

@melvin-bot melvin-bot bot removed the Overdue label Apr 24, 2023
@melvin-bot melvin-bot bot added the Overdue label May 2, 2023
@JmillsExpensify
Copy link

Same

@JmillsExpensify
Copy link

At a conference this week, not prioritizing this.

@melvin-bot melvin-bot bot removed the Overdue label May 17, 2023
@roryabraham roryabraham added Monthly KSv2 and removed Weekly KSv2 labels May 18, 2023
@melvin-bot melvin-bot bot added the Overdue label Jun 19, 2023
@j-piasecki
Copy link
Contributor

It is working as expected when building the app from the current (WoLewicki@e12c1cf) andrew-fabric branch:

ios-request-money-keboard.mov

@JmillsExpensify
Copy link

Nice work! Let's close this one out then.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Monthly KSv2 NewFeature Something to build that is a new item.
Projects
No open projects
Development

No branches or pull requests

5 participants