-
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
[Payment 2/21] [$500] [Wave 8] [Ideal Nav] WS - When deleting a WS offline, it is not crossed out #35702
Comments
Triggered auto assignment to @CortneyOfstad ( |
👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open `StagingDeployCash` deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:
|
Triggered auto assignment to @puneetlath ( |
We think that this bug might be related to #wave6 |
Hmm. This isn't really a Wave 6 specific issue, it's kind of just general to the app |
It should obviously be fixed, but I don't think it needs to be tracked on the Wave 6 board |
@puneetlath Just let me know if there is anything you need from me as BZ! 👍 |
Job added to Upwork: https://www.upwork.com/jobs/~0119aa0aa444a2b68a |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @mananjadhav ( |
Thanks @hayata-suenaga! |
📣 @mkhutornyi 🎉 An offer has been automatically sent to your Upwork account for the Contributor role 🎉 Thanks for contributing to the Expensify app! Offer link |
Ohh yeah I marked it outdated because of this comment. Sorry I forgot to unhide it. |
ohhh you can actually make comment |
Can we confirm? I prefer hiding it as we don't have any pattern yet to disable 3 dots menu |
I believe we can just disable the dots 👍 |
@mkhutornyi @mananjadhav — any update on the PR/review? Thanks! |
PR was merged and hit staging |
@CortneyOfstad This should be ready for payout. Can you help with the payout summary? Added a comment on the offending PR. @hayata-suenaga @CortneyOfstad Can one of you confirm if a test exists for the offline feedback of WS deletion already? If not, then I think it makes sense to add a regression test here. Copied the steps from the PR for regression test proposal.
|
@mananjadhav I did not see a regression test in relation to this, so I believe that creating one should be fine. If you're confident in those steps, I'll get regression test created 👍 Other Offline Mode tests were available in TestRail, but not removing a workspace, so would recommend adding here Payment summary incoming! |
Payment Summary@mkhutornyi (Contributor) — paid $500 via Upwork |
Yes we used the same steps to test the PR and they worked fine. |
Perfect! Regression test has been created here Also updated the title to reflect the payment date 👍 |
@mananjadhav has a pay request been sent in NewDot yet? |
If you are the assigned CME please investigate whether the linked PR caused a regression and leave a comment with the results. If a regression has occurred and you are the assigned CM follow the instructions here. If this regression could have been avoided please consider also proposing a recommendation to the PR checklist so that we can avoid it in the future. |
yeah above Melvin comment is false alarm. That's because I referenced this issue here |
This is good to close! @mananjadhav if you have any trouble getting the NewDot request through because this is closed, feel free to reopen 👍 |
$500 approved for @mananjadhav based on this summary. |
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: 1.4.36-0
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/4278525&group_by=cases:section_id&group_order=asc&group_id=229065
Email or phone of affected tester (no customers): applausetester+emilio@applause.expensifail.com
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:
Expected Result:
User expects the WS to be crossed out when deleted offline and then after returning online to disappear
Actual Result:
The WS instantly disappears
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Add any screenshot/video evidence
Bug6365190_1706899405908.WS_deleted_offline_is_not_crossed_out_.mp4
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: