-
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
[$1000] Share code - Unable to go to the chat with a user whose email address has "+" #19039
Comments
Triggered auto assignment to @sonialiap ( |
Bug0 Triage Checklist (Main S/O)
|
ProposalPlease re-state the problem that we are trying to solve in this issue.The
What is the root cause of that problem?The reason for this is because the string is not URI encoded before converting it into a QR code. What changes do you think we should make in order to solve the problem?To solve this, we first encode the string and then convert it into a QR code. In the line below, App/src/pages/ShareCodePage.js Line 41 in fa0ec1b
The email part of it the url has to be URI encoded using the const url = isReport ? `${CONST.NEW_EXPENSIFY_URL}r/${this.props.report.reportID}` : `${CONST.NEW_EXPENSIFY_URL}details?login=${encodeURIComponent(this.props.session.email)}`; What alternative solutions did you explore? (Optional)None. |
This comment was marked as resolved.
This comment was marked as resolved.
@sonialiap the email address of the QR you're scanning should contain a |
Whops, I didn't notice the |
Oh yeah, when testing with an email address that has a screen-20230522-163822.mp4 |
Job added to Upwork: https://www.upwork.com/jobs/~01efcc2d2440f70193 |
Current assignee @sonialiap is eligible for the External assigner, not assigning anyone new. |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @sobitneupane ( |
Triggered auto assignment to @francoisl ( |
This was already fixed in #19249, this was a duplicate issue we didn't realize. |
Reopening this so we can pay @Nathan-Mulugeta for reporting the issue once #19249 is deployed (as discussed starting here). |
Issue not reproducible during KI retests. (First week) |
@francoisl, @sonialiap, @sobitneupane Whoops! This issue is 2 days overdue. Let's get this updated quick! |
@francoisl, @sonialiap, @sobitneupane Eep! 4 days overdue now. Issues have feelings too... |
Looks like payment for reporting the issue is complete (#19076 (comment)), closing. |
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 found when executing PR #18636
Action Performed:
+
in the email, ex:example.user+test@applause.expensifail.com
Massage example.user+test@applause.expensifail.com
Expected Result:
User can jump to a conversation with another user using a QR code.
Actual Result:
The user sees an error in the chat and is not able to send messages.
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Version Number: 1.3.14.9
Reproducible in staging?: Yes
Reproducible in production?: new feature
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
Screen_Recording_20230516_112421_New.Expensify.mp4
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team @Nathan-Mulugeta
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1684165146584739
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: