-
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
[HOLD for payment 2024-12-20] Add inline Submit action to Search #53640
Comments
SubmitReport is currently not 1:1:1 which means this issue depends on migrating this command first. I'm auditing the code to assess what code needs to be migrated to make this feature feasible. |
Created an issue here |
Triggered auto assignment to @Christinadobrzyn ( |
|
Triggered auto assignment to Design team member for new feature review - @shawnborton ( |
This comment was marked as off-topic.
This comment was marked as off-topic.
|
The solution for this issue has been 🚀 deployed to production 🚀 in version 9.0.75-6 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 2024-12-20. 🎊 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:
|
Preparing for upcoming payment Contributor+: @ishpaul777 owed $250 via NewDot @ishpaul777 do we need a regression test for this? |
Payment Summary
BugZero Checklist (@Christinadobrzyn)
|
@ishpaul777 are you paid in Upwork? https://www.upwork.com/nx/wm/offer/105419289 Do we need a regression test for this? TY! |
yes Accepted Offer!
I think we should be adding all regression test flow for this feature as part of [Search v1] Add bulk actions on Search project Can you confirm @luacmartins ? Just incase not here's Regression test proposet for this issue: Tests:
Do we agree 👍 or 👎 ? |
I'll create the regression test just in case we want to submit one - https://github.com/Expensify/Expensify/issues/455332 I paid you in Upwork @ishpaul777 based on this payment summary - #53640 (comment) We'll keep this open to see if @luacmartins has any input on the regression test - #53640 (comment) |
Can I work on this |
📣 @sonkeydotcom! 📣
|
Contributor details |
✅ Contributor details stored successfully. Thank you for contributing to Expensify! |
Hi @sonkeydotcom! thanks for your interest! This job is already complete. Feel free to check out the open jobs here https://github.com/Expensify/App/issues?q=is%3Aopen+is%3Aissue+label%3A%22Help+Wanted%22 |
I'll close this as complete. @luacmartins is ooo so we'll consider this complete. Thanks everyone! |
We've added the
Approve
andPay
actions to Search in [this PR](#51911. Next, we should add theSubmit
action to the Search results page to make it a complete experience for users looking to batch manage their expensesIssue Owner
Current Issue Owner: @ChristinadobrzynThe text was updated successfully, but these errors were encountered: