-
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
[Due for payment 2025-01-18] [$250] Mark the self tour as completed when user returns from Navattic #54815
Comments
Triggered auto assignment to @RachCHopkins ( |
@c3024 will implement this change and @rushatgabhane is here for c+ role. Thanks for picking this us @c3024, what is your ETA for the PR? |
Job added to Upwork: https://www.upwork.com/jobs/~021876310402323638158 |
Current assignees @rushatgabhane and @c3024 are eligible for the External assigner, not assigning anyone new. |
@mountiny , raised a PR. |
|
The solution for this issue has been 🚀 deployed to production 🚀 in version 9.0.83-5 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 2025-01-18. 🎊 For reference, here are some details about the assignees on this issue:
|
BugZero Checklist: The PR adding this new feature has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:
|
Payment Summary
BugZero Checklist (@RachCHopkins)
|
@rushatgabhane, @RachCHopkins, @mountiny, @c3024 Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
Regression test proposal:
|
Payment Summary:
Upwork job here |
No payments via Upwork, the contract has been completed, and the Upwork post has been closed. |
$250 approved for @c3024 |
$250 approved for @rushatgabhane |
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:
Reproducible in staging?:
Reproducible in production?:
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?:
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:
Slack conversation (hyperlinked to channel name):
Action Performed:
Break down in numbered steps
When user completes the Self Tour in Navattic, then the url will newly include
?navattic=completed
query param.Expected Result:
Describe what you think should've happened
When the user returns to Expensify to the ConciergePage with this param, make sure to call the SelfTourViewed API
Actual Result:
Describe what actually happened
Nothing happens now
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
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
Issue Owner
Current Issue Owner: @Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @RachCHopkinsThe text was updated successfully, but these errors were encountered: