Skip to content

feat(nimbus): Add holdback checkbox to Experimenter#16137

Open
yashikakhurana wants to merge 1 commit into
mainfrom
16085
Open

feat(nimbus): Add holdback checkbox to Experimenter#16137
yashikakhurana wants to merge 1 commit into
mainfrom
16085

Conversation

@yashikakhurana

Copy link
Copy Markdown
Contributor

Because

  • Users need a way to mark an experiment as a holdback so the weekly task can identify and process it

This commit

  • Adds an is_holdback checkbox to the Audience page in the Experimenter UI

Fixes #16085

@yashikakhurana

Copy link
Copy Markdown
Contributor Author

Blocked on this #16136

@yashikakhurana

Copy link
Copy Markdown
Contributor Author
Screenshot 2026-06-29 at 10 57 19 AM

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this is the same change as in #16136 so shouldn't be in this PR.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeah branching from the pr

Comment thread experimenter/experimenter/nimbus_ui/constants.py Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe this is the best spot, but I wonder if this should go in the Overall form? Or maybe Branches since that's where the "Is this a rollout?" question is?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeah was not sure what's the best place to put, I can move it to branches

Comment thread experimenter/experimenter/settings.py
Comment thread experimenter/experimenter/jetstream/tests/test_tasks.py
@yashikakhurana yashikakhurana force-pushed the 16085 branch 3 times, most recently from 9b5bd4d to 10aab52 Compare June 30, 2026 22:55
@yashikakhurana

Copy link
Copy Markdown
Contributor Author

@mikewilli okay I have cleaned this PR since is holdback is available on main now, previously I was branching out from the other branches and hence all the changes, also moved the field to the Branches page

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add is_holdback checkbox to experiment configuration UI

2 participants