[HOLD for payment 2022-01-27] Mac - Safari - IOU - Unable to paste the amount copied from notes

[kavimuru]

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

---

Issue was found when executing #6811

Action Performed:

1. Copy a numeric value from Notes on MAC (it has to be a "title" in notes)
2. Go to FAB
3. Click on Send money or Request money option
4. Paste the amount

Expected Result:

You should be able to paste the amount

Actual Result:

Unable paste the amount copied from notes in MAC.
Working if I copy paste from text edit and msword
Chrome need to select "paste and match style"

Workaround:

Unknown

Platform:

Where is this issue occurring?

• Web - Safari

Version Number: v1.1.24-0

Reproducible in staging?: Yes

Reproducible in production?: new feature

Logs: https://stackoverflow.com/c/expensify/questions/4856

Notes/Photos/Videos:
Issue was found when executing #6811
https://user-images.githubusercontent.com/43996225/147685859-16450305-f862-4f6d-872f-4594fde2ce6a.mp4

Expensify/Expensify Issue URL:

Issue reported by: Applause

Slack conversation:

View all open jobs on GitHub

[OSBotify]

👋 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.

[MelvinBot]

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

[akshayasalvi]

#6811 is my PR. The change that I made was to allow pasting for mobile devices. This problem exists in the production too (where my code doesn't exist) and seems to be unrelated to my PR.

[mvtglobally]

@akshayasalvi your PR#6811 has Desktop and Web checked off as platforms where PR applies and should be tested. Are you saying it is incorrectly marked?

[akshayasalvi]

Well when I marked Desktop and Web, it was just to ensure that it didn't break anything that worked already. The intended PR changes were supposed to be for Mobile Web, iOS and Android.

and as I described the issue exists in production as well and hence not related to my PR.

[aldo-expensify]

Hmm, I tested this in staging in Safari + Mac (Web) and it seems to work fine:

Screen.Recording.2021-12-29.at.3.45.37.PM.mov

I used Cmd + c to copy and Cmd + v to paste, did you do something different?

[kavimuru]

@aldo-expensify Still able to reproduce. I used right click menu

Recording.10.mp4

.

[aldo-expensify]

@kavimuru hmm I tried with the right click menu and it worked... 😕 I'm no sure about what could be making the difference...

Screen.Recording.2021-12-29.at.5.28.10.PM.mov

[aldo-expensify]

Ohhh, I know now. The number you are copying is at the top of the note... it is part of the "title", if I copy "2021" from the date in the title you see in my video, it doesn't work 😮

Maybe something besides just numbers get copied to the clipboard...

Update: We are getting the following text pasted: *2021*, which gets rejected for not being a number.