App icon indicating copy to clipboard operation
App copied to clipboard

Add Submit action to Search

Open luacmartins opened this issue 1 year ago • 6 comments

We've added the Approve and Pay actions to Search in [this PR](https://github.com/Expensify/App/issues/51911. Next, we should add the Submit action to the Search results page to make it a complete experience for users looking to batch manage their expenses

luacmartins avatar Dec 05 '24 17:12 luacmartins

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.

luacmartins avatar Dec 06 '24 18:12 luacmartins

Created an issue here

luacmartins avatar Dec 06 '24 19:12 luacmartins

Triggered auto assignment to @Christinadobrzyn (NewFeature), see https://stackoverflowteams.com/c/expensify/questions/14418#:~:text=BugZero%20process%20steps%20for%20feature%20requests for more details. Please add this Feature request to a GH project, as outlined in the SO.

melvin-bot[bot] avatar Dec 06 '24 21:12 melvin-bot[bot]

:warning: It looks like this issue is labelled as a New Feature but not tied to any GitHub Project. Keep in mind that all new features should be tied to GitHub Projects in order to properly track external CAP software time :warning:

melvin-bot[bot] avatar Dec 06 '24 21:12 melvin-bot[bot]

Triggered auto assignment to Design team member for new feature review - @shawnborton (NewFeature)

melvin-bot[bot] avatar Dec 06 '24 21:12 melvin-bot[bot]

monitoring https://github.com/Expensify/Expensify/issues/451223

Christinadobrzyn avatar Dec 09 '24 23:12 Christinadobrzyn

⚠️ Looks like this issue was linked to a Deploy Blocker here

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.

melvin-bot[bot] avatar Dec 12 '24 09:12 melvin-bot[bot]

Reviewing label has been removed, please complete the "BugZero Checklist".

melvin-bot[bot] avatar Dec 13 '24 15:12 melvin-bot[bot]

The solution for this issue has been :rocket: deployed to production :rocket: in version 9.0.75-6 and is now subject to a 7-day regression period :calendar:. Here is the list of pull requests that resolve this issue:

  • https://github.com/Expensify/App/pull/53641

If no regressions arise, payment will be issued on 2024-12-20. :confetti_ball:

For reference, here are some details about the assignees on this issue:

  • @ishpaul777 requires payment (Needs manual offer from BZ)

melvin-bot[bot] avatar Dec 13 '24 15:12 melvin-bot[bot]

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:

  • [x] [@ishpaul777] Please propose regression test steps to ensure the new feature will work correctly on production in further releases.
  • [x] [@Christinadobrzyn] Link the GH issue for creating/updating the regression test once above steps have been agreed upon. https://github.com/Expensify/Expensify/issues/455332

melvin-bot[bot] avatar Dec 13 '24 15:12 melvin-bot[bot]

Preparing for upcoming payment

Contributor+: @ishpaul777  owed $250 via NewDot

@ishpaul777 do we need a regression test for this?

Christinadobrzyn avatar Dec 17 '24 15:12 Christinadobrzyn

Payment Summary

Upwork Job

  • ROLE: @ishpaul777 paid $250 via Upwork (https://www.upwork.com/nx/wm/workroom/39299132/overview)

BugZero Checklist (@Christinadobrzyn)

  • [x] I have verified the correct assignees and roles are listed above and updated the neccesary manual offers
  • [x] I have verified that there are no duplicate or incorrect contracts on Upwork for this job (https://www.upwork.com/ab/applicants//hired)
  • [x] I have paid out the Upwork contracts or cancelled the ones that are incorrect
  • [x] I have verified the payment summary above is correct

melvin-bot[bot] avatar Dec 20 '24 09:12 melvin-bot[bot]

@ishpaul777 are you paid in Upwork? https://www.upwork.com/nx/wm/offer/105419289

Do we need a regression test for this? TY!

Christinadobrzyn avatar Dec 20 '24 21:12 Christinadobrzyn

@ishpaul777 are you paid in Upwork? https://www.upwork.com/nx/wm/offer/105419289

yes Accepted Offer!

Do we need a regression test for this?

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:

  1. Create a workspace
  2. Enable workflows and Delay submissions
  3. Submit an expense to the workspace
  4. Navigate to the Search page
  5. Verify that you see the submit button next to the expense
  6. Press submit
  7. Verify that the report is submitted

Do we agree 👍 or 👎 ?

ishpaul777 avatar Dec 20 '24 21:12 ishpaul777

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 - https://github.com/Expensify/App/issues/53640#issuecomment-2556567337

We'll keep this open to see if @luacmartins has any input on the regression test - https://github.com/Expensify/App/issues/53640#issuecomment-2557744660

Christinadobrzyn avatar Dec 20 '24 23:12 Christinadobrzyn

Can I work on this

sonkiee avatar Dec 21 '24 21:12 sonkiee

📣 @sonkeydotcom! 📣 Hey, it seems we don’t have your contributor details yet! You'll only have to do this once, and this is how we'll hire you on Upwork. Please follow these steps:

  1. Make sure you've read and understood the contributing guidelines.
  2. Get the email address used to login to your Expensify account. If you don't already have an Expensify account, create one here. If you have multiple accounts (e.g. one for testing), please use your main account email.
  3. Get the link to your Upwork profile. It's necessary because we only pay via Upwork. You can access it by logging in, and then clicking on your name. It'll look like this. If you don't already have an account, sign up for one here.
  4. Copy the format below and paste it in a comment on this issue. Replace the placeholder text with your actual details. Screen Shot 2022-11-16 at 4 42 54 PM Format:
Contributor details
Your Expensify account email: <REPLACE EMAIL HERE>
Upwork Profile Link: <REPLACE LINK HERE>

melvin-bot[bot] avatar Dec 21 '24 21:12 melvin-bot[bot]

Contributor details Your Expensify account email: [email protected] Upwork Profile Link: https://www.upwork.com/freelancers/~0126988bb835c03de3

sonkiee avatar Dec 21 '24 21:12 sonkiee

✅ Contributor details stored successfully. Thank you for contributing to Expensify!

melvin-bot[bot] avatar Dec 21 '24 21:12 melvin-bot[bot]

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

Christinadobrzyn avatar Dec 23 '24 16:12 Christinadobrzyn

I'll close this as complete. @luacmartins is ooo so we'll consider this complete. Thanks everyone!

Christinadobrzyn avatar Dec 23 '24 16:12 Christinadobrzyn