-
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
[$500] Web - Request money - Console error after refreshing page and requesting money #27726
Comments
Triggered auto assignment to @mallenexpensify ( |
Bug0 Triage Checklist (Main S/O)
|
@lanitochka17 I see the below on Chrome. Can you screenshot a pic of the error you're seeing so I can confirm they're the same? ( it's a lil hard for me t make out in the vid) |
Job added to Upwork: https://www.upwork.com/jobs/~010ab36420ceccc863 |
Current assignee @mallenexpensify is eligible for the External assigner, not assigning anyone new. |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @fedirjh ( |
Guessing/hoping this can be external, I'm unsure though. @fedirjh , what do you think? |
This is a CSP error and needs to be fixed on the server side. |
cc @mallenexpensify This should be internal, should be fixed on server. |
Current assignee @fedirjh is eligible for the Internal assigner, not assigning anyone new. |
Awaiting internal assignment. |
hmm... are we treating console errors with the same level of priority as other bugs? |
It depends on the specific console error. Some errors indicate broken functionality and require higher priority, while in this case, it's a CSP (Content Security Policy) error that doesn't seem to affect functionality. IMO, it's not as high a priority as other bugs. |
@mallenexpensify, @fedirjh Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
Still waiting internal pickup, since it's not high priority, I'll wait a lil bit longer then drop in #engineering-chat to get more 👀 and feedback about next steps |
@mallenexpensify @fedirjh this issue is now 4 weeks old and preventing us from maintaining WAQ. This should now be your highest priority. Please post below what your plan is to get a PR in review ASAP. Thanks! |
@mallenexpensify, @fedirjh Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
Posted internally |
No takers in the post above, will post again next week if no one picks this up |
Maybe post about next week, it's not a priority and it the holidays here... |
Likely closing this soon since it's a console error and we're prioritizing working on roadmap issues |
If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!
Issue found when executing PR #27395
Action Performed:
Expected Result:
There is no console error
Actual Result:
There is a console error after refreshing a page and selecting "Request money". Console error does not show up if user navigates to different areas of the app before requesting money
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Version Number: 1.3.71-5
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
Bug6205468_Recording__234.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: