Skip to content

[HOLD for payment 2023-07-06] [HOLD for payment 2023-07-05] Removing CP Staging label #20764

Closed

Description

@luacmartins

Problem

Coming from this thread and in the spirit of discussing the necessity of each CP, we decided to remove the CP Staging label so that NewDot CPs can only be manually CPed by App deployers after a discussion about the real need for that CP.

Why is this important

Ensures that we always discuss each CP by making authors request the CP in Slack.

Solution

Remove CP Staging label and its functionality

cc @Expensify/mobile-deployers

Activity

  1. melvin-bot commented on Jun 14, 2023

    @melvin-bot
  2. self-assigned this
    on Jun 14, 2023
  3. AndrewGable commented on Jun 14, 2023

    @AndrewGable
    Contributor

    I can implement this

  4. johncschuster commented on Jun 14, 2023

    @johncschuster
    Contributor

    Am I needed on this one?

  5. roryabraham commented on Jun 14, 2023

    @roryabraham
    Contributor

    This will be internal so you're probably good here @johncschuster

  6. roryabraham commented on Jun 17, 2023

    @roryabraham
    Contributor

    Let's plz try to get #20791 over the finish line before we start implementation on this.

  7. AndrewGable commented on Jun 23, 2023

    @AndrewGable
    Contributor

    Held on #20791

  8. 4 remaining items

  9. added
    Awaiting PaymentAuto-added when associated PR is deployed to production
    and removed on Jun 28, 2023
  10. changed the title [-]Removing CP Staging label[/-] [+][HOLD for payment 2023-07-05] Removing CP Staging label[/+] on Jun 28, 2023
  11. melvin-bot commented on Jun 28, 2023

    @melvin-bot

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

  12. melvin-bot commented on Jun 28, 2023

    @melvin-bot

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

    After the hold period is over and BZ checklist items are completed, please complete any of the applicable payments for this issue, and check them off once done.

    • External issue reporter
    • Contributor that fixed the issue
    • Contributor+ that helped on the issue and/or PR

    As a reminder, here are the bonuses/penalties that should be applied for any External issue:

    • Merged PR within 3 business days of assignment - 50% bonus
    • Merged PR more than 9 business days after assignment - 50% penalty
  13. thienlnam commented on Jun 28, 2023

    @thienlnam
    Contributor

    Closing - no payments here to send out here!

    There is one linked issue I opened up here: #21763 about the CP process but the CP Staging label is no more!

  14. removed
    Awaiting PaymentAuto-added when associated PR is deployed to production
    on Jun 28, 2023
  15. added
    Awaiting PaymentAuto-added when associated PR is deployed to production
    and removed on Jun 29, 2023
  16. changed the title [-][HOLD for payment 2023-07-05] Removing CP Staging label[/-] [+][HOLD for payment 2023-07-06] [HOLD for payment 2023-07-05] Removing CP Staging label[/+] on Jun 29, 2023
  17. melvin-bot commented on Jun 29, 2023

    @melvin-bot

    The solution for this issue has been 🚀 deployed to production 🚀 in version 1.3.34-1 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 2023-07-06. 🎊

    After the hold period is over and BZ checklist items are completed, please complete any of the applicable payments for this issue, and check them off once done.

    • External issue reporter
    • Contributor that fixed the issue
    • Contributor+ that helped on the issue and/or PR

    As a reminder, here are the bonuses/penalties that should be applied for any External issue:

    • Merged PR within 3 business days of assignment - 50% bonus
    • Merged PR more than 9 business days after assignment - 50% penalty
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

Awaiting PaymentAuto-added when associated PR is deployed to productionEngineeringNewFeatureSomething to build that is a new item.WeeklyKSv2

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions