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 8/28] [$250] Billable - Billable toggle becomes disabled when tracking expense with billable toggle enabled #43768

Closed
6 tasks done
izarutskaya opened this issue Jun 14, 2024 · 46 comments
Assignees
Labels
Awaiting Payment Auto-added when associated PR is deployed to production Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 Hot Pick Ready for an engineer to pick up and run with Internal Requires API changes or must be handled by Expensify staff

Comments

@izarutskaya
Copy link

izarutskaya commented Jun 14, 2024

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.83-1
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/4631171
Logs: https://stackoverflow.com/c/expensify/questions/4856
Issue reported by: Applause-Internal team

Action Performed:

Precondition:

  • "Re-bill expenses to clients" is set to "Default to Billable" on Old Dot.
  1. Go to staging.new.expensify.com
  2. Go to workspace chat.
  3. Go to + > Track expense > Manual.
  4. Track a manual expense with billable toggle enabled.
  5. Go to transaction thread.

Expected Result:

Billable toggle will remain enabled.

Actual Result:

Billable toggle becomes disabled.
This issue only happens with tracked expenses and not submitted expenses.

Workaround:

Unknown

Platforms:

Which of our officially supported platforms is this issue occurring on?

  • Android: Native
  • Android: mWeb Chrome
  • iOS: Native
  • iOS: mWeb Safari
  • MacOS: Chrome / Safari
  • MacOS: Desktop

Screenshots/Videos

Bug6513030_1718361681022.bandicam_2024-06-14_18-37-26-151.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~01d112a7853dd092a2
  • Upwork Job ID: 1808520953526046967
  • Last Price Increase: 2024-07-24
@izarutskaya izarutskaya added Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels Jun 14, 2024
Copy link

melvin-bot bot commented Jun 14, 2024

Triggered auto assignment to @CortneyOfstad (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.

@izarutskaya
Copy link
Author

We think this issue might be related to the #collect project.

@jacobkim9881
Copy link
Contributor

I think this issue might be related to BE.

@melvin-bot melvin-bot bot added the Overdue label Jun 17, 2024
Copy link

melvin-bot bot commented Jun 17, 2024

@CortneyOfstad Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

@CortneyOfstad
Copy link
Contributor

I am not able to recreate as I am getting the infinite skeleton loader (context here). Going to see if this can be retested by QA

@melvin-bot melvin-bot bot removed the Overdue label Jun 17, 2024
@CortneyOfstad CortneyOfstad added retest-weekly Apply this label if you want this issue tested on a Weekly basis by Applause Needs Reproduction Reproducible steps needed labels Jun 17, 2024
@MelvinBot
Copy link

This has been labelled "Needs Reproduction". Follow the steps here: https://stackoverflowteams.com/c/expensify/questions/16989

@melvin-bot melvin-bot bot added the Overdue label Jun 19, 2024
@CortneyOfstad
Copy link
Contributor

Not overdue as we're waiting for it to be retested by QA

@melvin-bot melvin-bot bot added Overdue and removed Overdue labels Jun 19, 2024
@CortneyOfstad
Copy link
Contributor

Still waiting for it to be retested, as I cannot recreate

@melvin-bot melvin-bot bot added Overdue and removed Overdue labels Jun 25, 2024
@CortneyOfstad
Copy link
Contributor

Still waiting for this to be tested, as I still cannot recreate!

@melvin-bot melvin-bot bot removed the Overdue label Jun 28, 2024
Copy link

melvin-bot bot commented Jun 28, 2024

@CortneyOfstad this issue was created 2 weeks ago. Are we close to a solution? Let's make sure we're treating this as a top priority. Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks!

@melvin-bot melvin-bot bot added the Overdue label Jul 1, 2024
Copy link

melvin-bot bot commented Jul 1, 2024

@CortneyOfstad Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

@CortneyOfstad
Copy link
Contributor

Still waiting for this to be tested since my test account is still impacted by the infinite skeleton loader (GH here)

@melvin-bot melvin-bot bot removed the Overdue label Jul 1, 2024
@mvtglobally
Copy link

Issue is reproducible during KI retests.

1719762687593.bandicam_2024-06-30_18-50-02-842.mp4

@mvtglobally mvtglobally removed the retest-weekly Apply this label if you want this issue tested on a Weekly basis by Applause label Jul 3, 2024
@CortneyOfstad CortneyOfstad added the External Added to denote the issue can be worked on by a contributor label Jul 3, 2024
@melvin-bot melvin-bot bot removed the Overdue label Aug 5, 2024
@trjExpensify
Copy link
Contributor

Ah, okay.. it's using track on a workspace. Okay cool, we can move it in.

I could be wrong, but I think it's because TrackExpense here in the TransactionAPI in Web-E doesn't include billable. So we aren't doing anything with that toggle on the frontend.

CC: @thienlnam

@melvin-bot melvin-bot bot added the Overdue label Aug 8, 2024
@trjExpensify trjExpensify removed the Needs Reproduction Reproducible steps needed label Aug 8, 2024
@thienlnam
Copy link
Contributor

That was by design though, tracked expenses are by nature not billable or reimbursable. Is there a use-case for that?

@trjExpensify
Copy link
Contributor

If it was by design, why can you select the Billable toggle when you use Track expense on a workspace?

But either way, yes, I think there's a case for "I'm tracking the expenses I incur for my business on a workspace, and billing some of those expenses on to my clients".

@ishpaul777
Copy link
Contributor

not overdue, actively discussing what should be the expected behaviour.

@melvin-bot melvin-bot bot removed the Overdue label Aug 8, 2024
@thienlnam
Copy link
Contributor

If it was by design, why can you select the Billable toggle when you use Track expense on a workspace?

This was probably added later - but we can have the billable flag get passed to the API. Will require some auth changes as well

@melvin-bot melvin-bot bot added the Overdue label Aug 12, 2024
@CortneyOfstad
Copy link
Contributor

I'm back from OoO @johncschuster — thanks for holding down the fort while I was gone!

@melvin-bot melvin-bot bot added Overdue and removed Overdue labels Aug 12, 2024
Copy link

melvin-bot bot commented Aug 15, 2024

@tylerkaraszewski, @johncschuster, @CortneyOfstad, @ishpaul777 Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

@johncschuster
Copy link
Contributor

Just catching up on this one. It sounds like we've come to the conclusion that we could pass the billable flag via the API. Do we all agree that that's the direction we're going to go? If so, who's doing that?

@melvin-bot melvin-bot bot removed the Overdue label Aug 16, 2024
@thienlnam
Copy link
Contributor

These PRs are currently in review by @pecanoro - looks like she is OOO but should be back next week

@melvin-bot melvin-bot bot added the Overdue label Aug 19, 2024
@CortneyOfstad
Copy link
Contributor

Thanks @thienlnam for the update!

@melvin-bot melvin-bot bot removed the Overdue label Aug 19, 2024
@thienlnam thienlnam added the Reviewing Has a PR in review label Aug 20, 2024
@CortneyOfstad
Copy link
Contributor

So the PRs went into production today, so with the 7-day holding period, the date will be 8/28, so changing the title to reflect that 👍

@CortneyOfstad CortneyOfstad added Awaiting Payment Auto-added when associated PR is deployed to production and removed Reviewing Has a PR in review labels Aug 21, 2024
@CortneyOfstad CortneyOfstad changed the title [$250] Billable - Billable toggle becomes disabled when tracking expense with billable toggle enabled [HOLD for Payment 8/28] [$250] Billable - Billable toggle becomes disabled when tracking expense with billable toggle enabled Aug 21, 2024
@ishpaul777
Copy link
Contributor

I dont there was a App PR so there is no payment due

@mvtglobally
Copy link

Issue not reproducible during KI retests. (First week)

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 Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 Hot Pick Ready for an engineer to pick up and run with Internal Requires API changes or must be handled by Expensify staff
Projects
Status: Done
Development

No branches or pull requests

10 participants