Skip to content
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

Closed
luacmartins opened this issue Dec 5, 2024 · 21 comments
Closed

[HOLD for payment 2024-12-20] Add inline Submit action to Search #53640

luacmartins opened this issue Dec 5, 2024 · 21 comments
Assignees
Labels
Awaiting Payment Auto-added when associated PR is deployed to production Daily KSv2 NewFeature Something to build that is a new item.

Comments

@luacmartins
Copy link
Contributor

luacmartins commented Dec 5, 2024

We've added the Approve and Pay actions to Search in [this PR](#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

Issue OwnerCurrent Issue Owner: @Christinadobrzyn
@luacmartins luacmartins added the Daily KSv2 label Dec 5, 2024
@luacmartins luacmartins self-assigned this Dec 5, 2024
@luacmartins luacmartins mentioned this issue Dec 5, 2024
50 tasks
@luacmartins
Copy link
Contributor Author

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
Copy link
Contributor Author

Created an issue here

@melvin-bot melvin-bot bot added Reviewing Has a PR in review Weekly KSv2 and removed Daily KSv2 labels Dec 6, 2024
@luacmartins luacmartins added the NewFeature Something to build that is a new item. label Dec 6, 2024
Copy link

melvin-bot bot commented Dec 6, 2024

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.

Copy link

melvin-bot bot commented Dec 6, 2024

⚠️ 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 ⚠️

Copy link

melvin-bot bot commented Dec 6, 2024

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

@Christinadobrzyn
Copy link
Contributor

@luacmartins luacmartins changed the title Add Submit action to Search Add inline Submit action to Search Dec 11, 2024

This comment was marked as off-topic.

@melvin-bot melvin-bot bot added Weekly KSv2 Awaiting Payment Auto-added when associated PR is deployed to production and removed Weekly KSv2 labels Dec 13, 2024
@melvin-bot melvin-bot bot changed the title Add inline Submit action to Search [HOLD for payment 2024-12-20] Add inline Submit action to Search Dec 13, 2024
@melvin-bot melvin-bot bot removed the Reviewing Has a PR in review label Dec 13, 2024
Copy link

melvin-bot bot commented Dec 13, 2024

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

Copy link

melvin-bot bot commented Dec 13, 2024

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:

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

Copy link

melvin-bot bot commented Dec 13, 2024

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:

@Christinadobrzyn
Copy link
Contributor

Preparing for upcoming payment

Contributor+: @ishpaul777  owed $250 via NewDot

@ishpaul777 do we need a regression test for this?

@melvin-bot melvin-bot bot added Daily KSv2 and removed Weekly KSv2 labels Dec 20, 2024
Copy link

melvin-bot bot commented Dec 20, 2024

Payment Summary

Upwork Job

BugZero Checklist (@Christinadobrzyn)

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

@Christinadobrzyn
Copy link
Contributor

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

Do we need a regression test for this? TY!

@ishpaul777
Copy link
Contributor

ishpaul777 commented Dec 20, 2024

@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 👎 ?

@Christinadobrzyn
Copy link
Contributor

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)

@sonkeydotcom
Copy link

Can I work on this

Copy link

melvin-bot bot commented Dec 21, 2024

📣 @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>

@sonkeydotcom
Copy link

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

Copy link

melvin-bot bot commented Dec 21, 2024

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

@melvin-bot melvin-bot bot added the Overdue label Dec 23, 2024
@Christinadobrzyn
Copy link
Contributor

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

@melvin-bot melvin-bot bot removed the Overdue label Dec 23, 2024
@Christinadobrzyn
Copy link
Contributor

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Awaiting Payment Auto-added when associated PR is deployed to production Daily KSv2 NewFeature Something to build that is a new item.
Projects
None yet
Development

No branches or pull requests

5 participants