-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
[$500] Wallet - An error in console then navigated to enable payments page #26207
Comments
Triggered auto assignment to @CortneyOfstad ( |
Bug0 Triage Checklist (Main S/O)
|
Hmm, I don't get the error that you're seeing: Can we confirm that this is not just a local issue @lanitochka17? |
I don't see any error with attached video, is this an issue? |
Same — I don't see the error in the video, only the screenshot |
@lanitochka17 bump on the comments above ^^^ |
Issue reproducible on Build 1.3.62-1 Recording.22.mp4 |
@lanitochka17 I think those errors are way before, it's issue with how browser and HTTP works. @CortneyOfstad We can ask in the slack weather to continue or not. I saw quite a few related to same issue. |
@CortneyOfstad Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
Going to get some internal eyes on this as well 👍 |
Triggered auto assignment to @youssef-lr ( |
@youssef-lr bump on this — thanks! |
@youssef-lr bump on the above. Thanks! |
@youssef-lr @CortneyOfstad this issue was created 2 weeks ago. Are we close to a solution? Let's make sure we're treating this as a top priority. Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks! |
I'm guessing this is coming from a library we use. I think we can make this external to look into it further. |
Job added to Upwork: https://www.upwork.com/jobs/~013b278b7230ab63f1 |
Current assignee @CortneyOfstad is eligible for the External assigner, not assigning anyone new. |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @ArekChr ( |
ProposalPlease re-state the problem that we are trying to solve in this issue.Error in console appears in enable payments page What is the root cause of that problem?Some content origins are not whitelisted using the Content-Security-Policy HTTP header. What changes do you think we should make in order to solve the problem?Add a whitelist for some external resources that we deem safe, e.g.:
What alternative solutions did you explore? (Optional)N/A |
@ArekChr any thoughts on the proposal? |
I could not reproduce the issue on macOS using Chrome, and I don't have access to a Windows machine for testing. @CortneyOfstad Could you please reassign me, as I won't be able to complete the review for this checklist? |
@situchan This console error shows randomly? |
For me, always happens on app launch or refresh Screen.Recording.2023-09-18.at.3.00.26.PM.mov |
@youssef-lr @CortneyOfstad @ArekChr this issue is now 3 weeks old. There is one more week left before this issue breaks WAQ and will need to go internal. What needs to happen to get a PR in review this week? Please create a thread in #expensify-open-source to discuss. Thanks! |
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸 |
Issue not reproducible during KI retests. (First week) |
Alright, @youssef-lr any thoughts on this? We can consistently seem to recreate this. Any ideas? |
Bump @youssef-lr |
Can't reproduce either, 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 #25458
Action Performed:
Expected Result:
No any error in console
Actual Result:
Error in console appears
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Version Number: 1.3.58-1
Reproducible in staging?: Yes
Reproducible in production?: Yes
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.6064.mp4
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal team
Slack conversation:
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: