Skip to content

[HOLD for payment 2024-11-05] Update Invoices setting screen #45175

Closed

Description

@madmax330

Activity

  1. self-assigned this
    on Jul 10, 2024
  2. moved this from Low to Critical in [#whatsnext] #billpayon Aug 27, 2024
  3. danielrvidal commented on Aug 27, 2024

    @danielrvidal
    Contributor

    I think this is just waiting for payment, right @madmax330 ?

  4. madmax330 commented on Sep 2, 2024

    @madmax330
    ContributorAuthor

    Yeah we need to pay the c+ for their review

  5. cretadn22 commented on Sep 6, 2024

    @cretadn22
    Contributor

    @madmax330
    @danielrvidal

    I am working on #48345 and need to remove ROUTES.WORKSPACE_INVOICES

    But I see a note:

    // TODO: Uncomment the following line when the invoices screen is ready - https://github.com/Expensify/App/issues/45175.

    Should we remove this comment?

  6. madmax330 commented on Sep 6, 2024

    @madmax330
    ContributorAuthor

    cc @VickyStash I don't think we want to remove it right?

  7. VickyStash commented on Sep 6, 2024

    @VickyStash
    Contributor

    Yeah, ROUTES.WORKSPACE_INVOICES route will be reused in the future.

  8. added
    ReviewingHas a PR in review
    and removed
    ReviewingHas a PR in review
    on Oct 1, 2024
  9. melvin-bot commented on Oct 2, 2024

    @melvin-bot

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

  10. 13 remaining items

  11. melvin-bot commented on Oct 29, 2024

    @melvin-bot

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

  12. melvin-bot commented on Oct 29, 2024

    @melvin-bot

    The solution for this issue has been 🚀 deployed to production 🚀 in version 9.0.54-11 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-11-05. 🎊

  13. fedirjh commented on Oct 29, 2024

    @fedirjh
    Contributor

    @madmax330 Could you please assign me, was C+ on #50565.

  14. melvin-bot commented on Nov 5, 2024

    @melvin-bot

    Issue is ready for payment but no BZ is assigned. @greg-schroeder you are the lucky winner! Please verify the payment summary looks correct and complete the checklist. Thanks!

  15. melvin-bot commented on Nov 5, 2024

    @melvin-bot

    Payment Summary

    Upwork Job

    • Reviewer: @fedirjh owed $250 via NewDot

    BugZero Checklist (@greg-schroeder)

    • 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
  16. greg-schroeder commented on Nov 5, 2024

    @greg-schroeder
    Contributor

    Payment summary:

    C+: @fedirjh - $250 - You can make a manual request

  17. melvin-bot commented on Nov 5, 2024

    @melvin-bot

    @madmax330 @greg-schroeder Be sure to fill out the Contact List!

  18. garrettmknight commented on Dec 10, 2024

    @garrettmknight
    Contributor

    $250 approved for @fedirjh

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

Awaiting PaymentAuto-added when associated PR is deployed to productionDailyKSv2

Type

No type

Projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions