-
Notifications
You must be signed in to change notification settings - Fork 3k
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
[$1000] URL appears as html in thread title and alternate text in search page #20520
Comments
Triggered auto assignment to @johncschuster ( |
Bug0 Triage Checklist (Main S/O)
|
👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open
|
Triggered auto assignment to @grgia ( |
@grgia is out for the weekend, going to grab this. |
I confirmed that this bug is caused in the back-end. The root cause is that the text for the reportAction is being overwritten by HTML. If you take these same actions offline, it will work. |
That means that, if it's not too late, this should actually be a deploy blocker for the API, not E/App. |
Going to create an internal deploy blocker |
Internal issue link: https://github.com/Expensify/Expensify/issues/291036 |
PR to fix this ready here: https://github.com/Expensify/Web-Expensify/pull/37784 |
@grgia requested your review on that PR because this issue was auto-assigned to you originally 😉 |
This is fixed |
@roryabraham, is this eligible for the reporting compensation? |
cc: @johncschuster |
Looks like @Puneet-here is eligible for the bug reporting bonus here. |
Sorry I didn't notice it earlier, @Puneet-here |
No problem! Thanks for opening it again. |
@johncschuster last TODO here is paying out the reporting bonus |
Thanks for the callout, @roryabraham! I'll apply the |
Job added to Upwork: https://www.upwork.com/jobs/~01b674a204efbb9b04 |
Current assignee @johncschuster is eligible for the External assigner, not assigning anyone new. |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @sobitneupane ( |
Sorry for the ping, @sobitneupane. I applied the label just to have the job get created automatically. Nothing to do here! |
@Puneet-here I've just invited you to the job via Upwork. Can you ping me once you've accepted that? |
@Puneet-here friendly bump so I can issue you the reporting bonus! |
Sorry for the delay. Just accepted the offer. |
Thanks! I've just issued payment! |
Thanks. We can close the issue @johncschuster |
@johncschuster @roryabraham this issue was created 2 weeks ago. Are we close to approving a proposal? If not, what's blocking us from getting this issue assigned? Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks! |
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:
Normal text should be shown
Actual Result:
URL shows in html format
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.26-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
Expensify/Expensify Issue URL:
Issue reported by: @Puneet-here
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1686286858935369
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: