-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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 2023-11-30] [$500] Request money - Hmm it's not here page opens when click on Learn more via the banner below #31567
Comments
Triggered auto assignment to @hayata-suenaga ( |
ProbemHmm it's not here page opens when click on Learn more via the banner below Root Causewe are using a wrong route for referral page https://help.expensify.com/articles/new-expensify/getting-started/Referral-Program instead of https://help.expensify.com/articles/new-expensify/billing-and-plan-types/Referral-Program Changesupdate AlternativeMatch the file location to the route |
Job added to Upwork: https://www.upwork.com/jobs/~01fc74f5c607d04925 |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @mananjadhav ( |
Triggered auto assignment to @lschurr ( |
Bug0 Triage Checklist (Main S/O)
|
@ishpaul777 are you able to raise a PR soon? |
Yes i am available |
📣 @mananjadhav Please request via NewDot manual requests for the Reviewer role ($500) |
📣 @ishpaul777 🎉 An offer has been automatically sent to your Upwork account for the Contributor role 🎉 Thanks for contributing to the Expensify app! Offer link |
thank you! raising a PR in few minutes |
Removing label since we CPed a fix |
|
The solution for this issue has been 🚀 deployed to production 🚀 in version 1.4.2-3 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue: If no regressions arise, payment will be issued on 2023-11-30. 🎊 After the hold period is over and BZ checklist items are completed, please complete any of the applicable payments for this issue, and check them off once done.
For reference, here are some details about the assignees on this issue:
|
BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:
|
@mananjadhav could you work on the checklist for this one if needed? |
Payment summary:
|
Bump on the checklist @mananjadhav |
@lschurr I wasn't able to review this PR. Do you want me to still work on the checklist for the payout? |
Ah, thanks for clarifying @mananjadhav! Nope, if you didn't work on it, I'll update the payment summary. |
@luacmartins - do we need a regression test for this one? |
just clarifying, I meant the payout for the external contributor if it is stuck because of this. |
I don't think we do. Applause caught this during regression test. |
Cool, we're good! I'll pay @ishpaul777 tomorrow and we'll be set to close. |
Paid! Closing this one out. |
If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!
Version Number: 1.4.1-5
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
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:
Expected Result:
The additional information page opens
Actual Result:
Hmm it's not here page opens
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Add any screenshot/video evidence
Bug6284020_1700498537224.Recording__6807.mp4
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: