-
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
[HOLD for payment 2024-09-26] [$250] [Dev] Console Warning: Possible unhandled promise rejection #49060
Comments
Triggered auto assignment to @stephanieelliott ( |
This has been labelled "Needs Reproduction". Follow the steps here: https://stackoverflowteams.com/c/expensify/questions/16989 |
ProposalPlease re-state the problem that we are trying to solve in this issue.Console warning when open native app. What is the root cause of that problem?When we open the app, the first route is the bottom tab navigator which has an empty path. App/src/libs/Navigation/linkingConfig/config.ts Lines 65 to 67 in 7748eff
Line 21 in 7748eff
The warning is shown when we pass the empty string to the URL constructor. App/src/libs/actions/Report.ts Lines 2683 to 2685 in 7748eff
What changes do you think we should make in order to solve the problem?Make sure the url is not empty before passing it to the URL constructor.
|
Job added to Upwork: https://www.upwork.com/jobs/~021834518653490433368 |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @aimane-chnaif ( |
@bernhardoj's proposal looks good. |
Triggered auto assignment to @dangrous, see https://stackoverflow.com/c/expensify/questions/7972 for more details. |
Sounds perfect. Assigning! |
PR is ready cc: @aimane-chnaif |
PR was deployed to prod on 9/19 but it looks like the automation didn't work. Updating manually to add payment date |
Payment Summary
BugZero Checklist (@stephanieelliott)
|
Requested in ND. |
@stephanieelliott mind confirming this payment summary when you're back online? Thank you! |
@stephanieelliott I am still using upwork |
@aimane-chnaif are you able to request via NewDot? It looks like are eligible for ND payments, and we would prefer to pay in ND rather than Upwork if it's ok with you. Otherwise, please let me know so that I can extend the offer to you in Upwork. Summarizing payment on this issue:
|
@stephanieelliott please. thx |
$250 approved for @bernhardoj |
Hey @aimane-chnaif I extended the offer to you in Upwork, please accept when you get a chance! https://www.upwork.com/nx/wm/offer/104253282 |
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!
Version Number: 9.0.30-9
Reproducible in staging?: Needs Reproduction
Reproducible in production?: Needs Reproduction
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
Expensify/Expensify Issue URL:
Issue reported by: @c3024
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1726072317575939?thread_ts=1726072125.690969&cid=C049HHMV9SM
Action Performed:
Logon to ND on iOS App
Expected Result:
No console warning
Actual Result:
Console Warning
Warning: Possible unhandled promise rejection
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Add any screenshot/video evidence
View all open jobs on GitHub
Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @aimane-chnaifThe text was updated successfully, but these errors were encountered: