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-03-18] [HIGH] [Instant Submit] for Collect: Improve creating expenses on Instant Submit paid plans, from NewDot #34955

Closed
Beamanator opened this issue Jan 23, 2024 · 25 comments
Assignees
Labels
Awaiting Payment Auto-added when associated PR is deployed to production Daily KSv2 Internal Requires API changes or must be handled by Expensify staff

Comments

@Beamanator
Copy link
Contributor

Beamanator commented Jan 23, 2024

Holding on https://github.com/Expensify/Expensify/issues/360667

Instant Submit for Collect Workspaces Design Doc

Tracking GH issue: https://github.com/Expensify/Expensify/issues/334111

Goal for this issue: Make sure these flows work perfectly in NewDot, for Paid plans on Instant Submit

  1. In Workspace chat, click report preview of a report to open the expense report
  2. Request money (this currently isn't possible for submitted reports, in NewDot)
  3. Verify the request / expense gets added to that correct expense report
  4. Navigate back to the workspace chat & request money there
  5. Verify the request / expense gets added to the most recently created expense report
    1. It looks like we currently optimistically try to find an OPEN report in NewDot to add a money request to & if we don't find one, we optimistically create a new Report Preview. We will need to update this to look for Processing as well, only if the policy is Instant Submit!
    2. We're also having a discussion about "do we need this to be added to the most recent expense report or any" in slack here.
Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~01f9e4f88d7ec8247b
  • Upwork Job ID: 1749734239297155072
  • Last Price Increase: 2024-01-23
Issue OwnerCurrent Issue Owner: @garrettmknight
@Beamanator Beamanator added Daily KSv2 Internal Requires API changes or must be handled by Expensify staff labels Jan 23, 2024
Copy link

melvin-bot bot commented Jan 23, 2024

Job added to Upwork: https://www.upwork.com/jobs/~01f9e4f88d7ec8247b

Copy link

melvin-bot bot commented Jan 23, 2024

Triggered auto assignment to Contributor Plus for review of internal employee PR - @hoangzinh (Internal)

@Beamanator
Copy link
Contributor Author

Technically all backend changes are ready on staging, so I'll take this off hold soon!

@Beamanator Beamanator changed the title [HOLD E/E#360667] [Instant Submit for Collect] CRITICAL: Allow creating expenses on Instant Submit paid plans, from NewDot [HOLD E/E#360667] [Instant Submit for Collect] CRITICAL: Improve creating expenses on Instant Submit paid plans, from NewDot Jan 25, 2024
@Beamanator Beamanator changed the title [HOLD E/E#360667] [Instant Submit for Collect] CRITICAL: Improve creating expenses on Instant Submit paid plans, from NewDot [Instant Submit for Collect] CRITICAL: Improve creating expenses on Instant Submit paid plans, from NewDot Jan 26, 2024
@melvin-bot melvin-bot bot added the Overdue label Jan 26, 2024
@Beamanator Beamanator added the Hot Pick Ready for an engineer to pick up and run with label Jan 26, 2024
@Beamanator
Copy link
Contributor Author

Looking for volunteer via Hot Pick label

@youssef-lr youssef-lr self-assigned this Jan 26, 2024
@melvin-bot melvin-bot bot removed the Overdue label Jan 26, 2024
@youssef-lr youssef-lr removed the Hot Pick Ready for an engineer to pick up and run with label Jan 26, 2024
@melvin-bot melvin-bot bot added the Overdue label Jan 29, 2024
Copy link

melvin-bot bot commented Jan 29, 2024

@hoangzinh, @youssef-lr Whoops! This issue is 2 days overdue. Let's get this updated quick!

@youssef-lr
Copy link
Contributor

Starting work on this one today.

@melvin-bot melvin-bot bot removed the Overdue label Jan 29, 2024
@greg-schroeder greg-schroeder changed the title [Instant Submit for Collect] CRITICAL: Improve creating expenses on Instant Submit paid plans, from NewDot [HIGH] Instant Submit for Collect: Improve creating expenses on Instant Submit paid plans, from NewDot Jan 31, 2024
@melvin-bot melvin-bot bot added the Overdue label Jan 31, 2024
@youssef-lr
Copy link
Contributor

Still being worked on.

@melvin-bot melvin-bot bot removed the Overdue label Jan 31, 2024
@youssef-lr
Copy link
Contributor

I should have an update by EOD tomorrow. I'm switching focus temporarily to investigate a high priority issue affecting some large customers https://expensify.slack.com/archives/CC7NECV4L/p1706641694016129

@melvin-bot melvin-bot bot added the Overdue label Feb 5, 2024
Copy link

melvin-bot bot commented Feb 5, 2024

@hoangzinh, @youssef-lr Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

@youssef-lr
Copy link
Contributor

On it today again.

@melvin-bot melvin-bot bot added Overdue and removed Overdue labels Feb 6, 2024
Copy link

melvin-bot bot commented Feb 12, 2024

@hoangzinh, @youssef-lr Huh... This is 4 days overdue. Who can take care of this?

@youssef-lr
Copy link
Contributor

I should have an update in a couple of days.

@melvin-bot melvin-bot bot added the Weekly KSv2 label Feb 13, 2024
@youssef-lr
Copy link
Contributor

App PR is ready, but held on an Auth PR.

@melvin-bot melvin-bot bot added Weekly KSv2 and removed Weekly KSv2 labels Feb 23, 2024
@trjExpensify trjExpensify changed the title [HIGH] Instant Submit for Collect: Improve creating expenses on Instant Submit paid plans, from NewDot [HIGH] [Instant Submit] for Collect: Improve creating expenses on Instant Submit paid plans, from NewDot Mar 5, 2024
@trjExpensify
Copy link
Contributor

App PR merged and on staging!

@youssef-lr
Copy link
Contributor

Not yet @trjExpensify :(

This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

@trjExpensify
Copy link
Contributor

Huh, is this wrong then? #36388 (comment)

@youssef-lr
Copy link
Contributor

oh weird it really wasn't showing for me.

@melvin-bot melvin-bot bot added Weekly KSv2 Awaiting Payment Auto-added when associated PR is deployed to production and removed Weekly KSv2 labels Mar 11, 2024
@melvin-bot melvin-bot bot changed the title [HIGH] [Instant Submit] for Collect: Improve creating expenses on Instant Submit paid plans, from NewDot [HOLD for payment 2024-03-18] [HIGH] [Instant Submit] for Collect: Improve creating expenses on Instant Submit paid plans, from NewDot Mar 11, 2024
@melvin-bot melvin-bot bot removed the Reviewing Has a PR in review label Mar 11, 2024
Copy link

melvin-bot bot commented Mar 11, 2024

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

Copy link

melvin-bot bot commented Mar 11, 2024

The solution for this issue has been 🚀 deployed to production 🚀 in version 1.4.49-4 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-03-18. 🎊

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

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

@garrettmknight garrettmknight self-assigned this Mar 11, 2024
@garrettmknight
Copy link
Contributor

Assigning myself to handle BZ stuff.

@garrettmknight garrettmknight added Daily KSv2 and removed Weekly KSv2 labels Mar 15, 2024
@garrettmknight
Copy link
Contributor

Updating to daily to pay on Monday. @hoangzinh are you getting paid via NewDot these days? Linking SO to for myself update if you are.

@hoangzinh
Copy link
Contributor

Hi @garrettmknight nope, I'm still getting paid via Upwork 😢

@garrettmknight
Copy link
Contributor

Bummer - offer is out to you!

@hoangzinh
Copy link
Contributor

Accepted. Thanks @garrettmknight

@melvin-bot melvin-bot bot added Daily KSv2 Overdue and removed Daily KSv2 labels Mar 17, 2024
@garrettmknight
Copy link
Contributor

Paid, closing

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 Internal Requires API changes or must be handled by Expensify staff
Projects
Archived in project
Development

No branches or pull requests

5 participants