-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
[HOLD for payment 2023-04-27] LHN edited message UI shows the whole multiline message for a few seconds #17574
Comments
Triggered auto assignment to @sakluger ( |
Bug0 Triage Checklist (Main S/O)
|
👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open
|
Triggered auto assignment to @iwiznia ( |
Seems a regression from this PR |
Fix is being CPed to staging now. |
Hi, sorry but it seems that this issue is pretty similar with what found here, that was reported first in the Slack conversation. https://expensify.slack.com/archives/C049HHMV9SM/p1681809609978589 |
Should I close this then? It is already being handled in #17212 |
Yes it seems we need to change the "Issue reported by" for this issue @kavimuru, though I'm not sure about the exact procedure and if there's something else to to change too. @iwiznia I'd say let's wait until the original issue and the fix are QA'ed on staging and then we can close. |
Thanks for handling this, I have update the issue body to mention @kerupuksambel as reported as they reported it first. Confirmed that this is fixed in staging @sakluger could you please handle the $250 report bonus to @kerupuksambel for this and then we can close this |
This is ready to pay the reporting bonus of $250 |
I'm having a hell of a time creating a job manually in Upwork, it keeps returning errors. I'm going to apply the internal label to get the job created, then I'll fix everything from there. |
Job added to Upwork: https://www.upwork.com/jobs/~01b50c1ba8e6e2cc4f |
Triggered auto assignment to Contributor Plus for review of internal employee PR - @mollfpr ( |
@mollfpr, you can disregard, this issue is just for paying a reporting bonus and so I've unassigned you. |
@kerupuksambel I've sent you an offer through upwork. |
All paid 👍 |
If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!
Action Performed:
Expected Result:
LHN should show only the first line of the message
Actual Result:
It shows the whole message for a few seconds in LH
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:
Which of our officially supported platforms is this issue occurring on?
Version Number: 1.3.1
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
Notes/Photos/Videos: Any additional supporting documentation
Recording.263.mp4
Expensify/Expensify Issue URL:
Issue reported by: @kerupuksambel
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1681820355045809
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: