Skip to content

Comments

feat(nimbus): create update leading branch image forms#14086

Merged
moibra05 merged 1 commit intomainfrom
14015
Dec 11, 2025
Merged

feat(nimbus): create update leading branch image forms#14086
moibra05 merged 1 commit intomainfrom
14015

Conversation

@moibra05
Copy link
Contributor

Because

  • Gives user the ability to change the first image associated with a branch after the experiment has already started

This commit

  • Gives each branch a modal form that allows the first branch image to be changed

Fixes #14015

@moibra05
Copy link
Contributor Author

image image image

@moibra05 moibra05 added preview Create Preview Environment pr-preview and removed preview Create Preview Environment labels Nov 28, 2025
Comment on lines 705 to 709
form.save()
resp = HttpResponse()
resp.headers["HX-Refresh"] = "true"
return resp

Copy link
Contributor

Choose a reason for hiding this comment

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

oh instead of refreshing the whole page, we can use hx-target and hx-swap

Copy link
Contributor Author

Choose a reason for hiding this comment

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

will be addressed in a future pr b/c closing modals after submitting forms is a little more complicated #14094

Copy link
Contributor

@yashikakhurana yashikakhurana left a comment

Choose a reason for hiding this comment

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

In general implementation looks good, just one minor suggestion. Thank you @moibra05

@moibra05 moibra05 added the preview Create Preview Environment label Nov 28, 2025
@mozcloud-preview-environments

Preview environment has been deployed.

Preview URL: https://pr14086-experimenter.preview.mozilla.cloud

It may take up to 5 minutes for the environment to become available.
If the link doesn't work immediately, please wait a few moments and try again.

You can monitor deployment status in Argo CD.

Copy link
Contributor

@yashikakhurana yashikakhurana left a comment

Choose a reason for hiding this comment

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

Thank you @moibra05 some small suggestions to improve, overall its coming good together

@mozcloud-preview-environments
Copy link

Preview environment has been deployed.

Preview URL: https://pr14086-experimenter.preview.mozilla.cloud

It may take up to 5 minutes for the environment to become available.
If the link doesn't work immediately, please wait a few moments and try again.

You can monitor deployment status in Argo CD.

1 similar comment
@mozcloud-preview-environments
Copy link

Preview environment has been deployed.

Preview URL: https://pr14086-experimenter.preview.mozilla.cloud

It may take up to 5 minutes for the environment to become available.
If the link doesn't work immediately, please wait a few moments and try again.

You can monitor deployment status in Argo CD.

@mozcloud-preview-environments
Copy link

Preview environment has been deployed.

Preview URL: https://pr14086-experimenter.preview.mozilla.cloud

It may take up to 5 minutes for the environment to become available.
If the link doesn't work immediately, please wait a few moments and try again.

You can monitor deployment status in Argo CD.

1 similar comment
@mozcloud-preview-environments
Copy link

Preview environment has been deployed.

Preview URL: https://pr14086-experimenter.preview.mozilla.cloud

It may take up to 5 minutes for the environment to become available.
If the link doesn't work immediately, please wait a few moments and try again.

You can monitor deployment status in Argo CD.

@mozcloud-preview-environments
Copy link

Preview environment has been deployed.

Preview URL: https://pr14086-experimenter.preview.mozilla.cloud

It may take up to 5 minutes for the environment to become available.
If the link doesn't work immediately, please wait a few moments and try again.

You can monitor deployment status in Argo CD.

2 similar comments
@mozcloud-preview-environments
Copy link

Preview environment has been deployed.

Preview URL: https://pr14086-experimenter.preview.mozilla.cloud

It may take up to 5 minutes for the environment to become available.
If the link doesn't work immediately, please wait a few moments and try again.

You can monitor deployment status in Argo CD.

@mozcloud-preview-environments
Copy link

Preview environment has been deployed.

Preview URL: https://pr14086-experimenter.preview.mozilla.cloud

It may take up to 5 minutes for the environment to become available.
If the link doesn't work immediately, please wait a few moments and try again.

You can monitor deployment status in Argo CD.

@mozcloud-preview-environments

Preview environment has been deployed.

Preview URL: https://pr14086-experimenter.preview.mozilla.cloud

It may take up to 5 minutes for the environment to become available.
If the link doesn't work immediately, please wait a few moments and try again.

You can monitor deployment status in Argo CD.

1 similar comment
@mozcloud-preview-environments

Preview environment has been deployed.

Preview URL: https://pr14086-experimenter.preview.mozilla.cloud

It may take up to 5 minutes for the environment to become available.
If the link doesn't work immediately, please wait a few moments and try again.

You can monitor deployment status in Argo CD.

Copy link
Contributor

@yashikakhurana yashikakhurana left a comment

Choose a reason for hiding this comment

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

Thank you @moibra05 tested locally, and using pr-preview mode, looks good to me, great job getting this done 🎉

@mozcloud-preview-environments

Preview environment has been deployed.

Preview URL: https://pr14086-experimenter.preview.mozilla.cloud

It may take up to 5 minutes for the environment to become available.
If the link doesn't work immediately, please wait a few moments and try again.

You can monitor deployment status in Argo CD.

1 similar comment
@mozcloud-preview-environments

Preview environment has been deployed.

Preview URL: https://pr14086-experimenter.preview.mozilla.cloud

It may take up to 5 minutes for the environment to become available.
If the link doesn't work immediately, please wait a few moments and try again.

You can monitor deployment status in Argo CD.

@moibra05 moibra05 enabled auto-merge December 11, 2025 15:56
@moibra05 moibra05 added this pull request to the merge queue Dec 11, 2025
Merged via the queue into main with commit 6c945b1 Dec 11, 2025
18 checks passed
@moibra05 moibra05 deleted the 14015 branch December 11, 2025 16:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-preview preview Create Preview Environment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Create branch image preview and description

4 participants