Skip to content

Comments

feat(nimbus): exclude language/locale/country#13753

Merged
jaredlockhart merged 3 commits intomainfrom
13752
Oct 23, 2025
Merged

feat(nimbus): exclude language/locale/country#13753
jaredlockhart merged 3 commits intomainfrom
13752

Conversation

@jaredlockhart
Copy link
Collaborator

Becuase

  • We need a way to optionally exclude selected languages/locales/countries

This commit

  • Adds a bool flag to exclude languages/locales/countries
  • Negates the JEXL expressions for each if exclude is selected
  • Adds checkboxes for each on the audience page
  • Updates the summary page to show whether each is included or excluded

fixes #13752

Becuase

* We need a way to optionally exclude selected languages/locales/countries

This commit

* Adds a bool flag to exclude languages/locales/countries
* Negates the JEXL expressions for each if exclude is selected
* Adds checkboxes for each on the audience page
* Updates the summary page to show whether each is included or excluded

fixes #13752
@jaredlockhart jaredlockhart added the preview Create Preview Environment label Oct 22, 2025
@jaredlockhart
Copy link
Collaborator Author

image image image

@mozcloud-preview-environments

Preview environment has been deployed.

Preview URL: https://pr13753-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://pr13753-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.

Tested locally works pefectly fine, interesting to see the approach, thanks @jaredlockhart

@mozcloud-preview-environments

Preview environment has been deployed.

Preview URL: https://pr13753-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://pr13753-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.

@jaredlockhart jaredlockhart added this pull request to the merge queue Oct 23, 2025
Merged via the queue into main with commit 0ceb6fb Oct 23, 2025
18 checks passed
@jaredlockhart jaredlockhart deleted the 13752 branch October 23, 2025 17:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

preview Create Preview Environment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

create an exclusion option for EU countries + countries with EU like privacy laws

3 participants