[HOLD for payment 2024-07-17] [$750] [Free trial] Restrict actions throughout the App of "past due" billing owner

[MitchExpensify]

This covers this section of the design doc:

• Restrict actions throughout the App of "past due" billing owner

cc @fabioh8010 @pac-guerreiro as you'll be picking this up

Issue Owner

Current Issue Owner: @twisterdotcom

[melvin-bot]

Triggered auto assignment to @twisterdotcom (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.

[melvin-bot]

Current assignee @twisterdotcom is eligible for the NewFeature assigner, not assigning anyone new.

[fabioh8010]

Hi, I'm Fábio - expert agency contributor - and I would like to work on this issue.

[twisterdotcom]

@MitchExpensify right now we have @rojiphil and @hoangzinh - is one of them the Contributor, or are they both C+ and we can assign @fabioh8010?

[rojiphil]

Me and @hoangzinh are both C+ assigned to this project. But one of us has to take this up.
Callstack is implementing this project as mentioned here. So we can assign @fabioh8010

[rojiphil]

@fabioh8010 Don't we have to wait until #43672 is done to work on this? Or are we taking this up for implementation now?

[fabioh8010]

Actually this one would be HOLD on #43668 and #43673, since the first one is merged and Restricted Action is close to be merged I will start my work on top of it, so I think this GH issue title was wrong.

[rojiphil]

Actually this one would be HOLD on #43668 and #43673, since the first one is merged and #43855 I will start my work on top of it

Cool. That make sense.
@MitchExpensify I can take this up as C+ here.

[trjExpensify]

Updating the title to reflect the hold on #43673

[trjExpensify]

The PR for #43673 has been deployed, removed the hold.

[melvin-bot]

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

[melvin-bot]

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

• [Free trial] Restrict actions throughout the App of "past due" billing owner #44393

If no regressions arise, payment will be issued on 2024-07-17. 🎊

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

• @rojiphil requires payment (Needs manual offer from BZ)
• @fabioh8010 does not require payment (Contractor)

[melvin-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:

• [@rojiphil] Please propose regression test steps to ensure the new feature will work correctly on production in further releases.
• [@twisterdotcom] Link the GH issue for creating/updating the regression test once above steps have been agreed upon.

[melvin-bot]

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

• [Free trial] Restrict actions throughout the App of "past due" billing owner #44393

If no regressions arise, payment will be issued on 2024-07-22. 🎊

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

• @rojiphil requires payment automatic offer (C+)
• @fabioh8010 does not require payment (Contractor)

[melvin-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:

• [@rojiphil] Please propose regression test steps to ensure the new feature will work correctly on production in further releases.
• [@twisterdotcom] Link the GH issue for creating/updating the regression test once above steps have been agreed upon.

[melvin-bot]

Payment Summary

• @rojiphil paid $750 here (C+)
• @fabioh8010 does not require payment (Contractor)

BugZero Checklist (@twisterdotcom)

• 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

[rojiphil]

@twisterdotcom I think the work in this issue deserves an increase in compensation:

Reasonings:

1. The PR took more time than the usual one or two rounds of testing as the scope involved too many tests (as described here)
2. Though not directly related to this issue, we addressed a related navigation issue on navigating back/away from the restricted actions view as mentioned here
3. Identified and supported an additional case here which was not identified yet in the design document

$750 is reasonable here. What do you think?

[melvin-bot]

⚠️ Could not update price automatically because there is no linked Upwork Job ID. The BZ team member will need to update the price manually in Upwork.

[twisterdotcom]

I'm happy with that, sure.

[twisterdotcom]

Payment Summary:

• @rojiphil paid $750 here (C+)
• @fabioh8010 does not require payment (Contractor)