Skip to content

feat(nimbus): support channels in review serializer#13306

Merged
jaredlockhart merged 1 commit intomainfrom
13266
Aug 20, 2025
Merged

feat(nimbus): support channels in review serializer#13306
jaredlockhart merged 1 commit intomainfrom
13266

Conversation

@jaredlockhart
Copy link
Collaborator

Becuase

  • We have added a new field called channels for desktop experiments
  • We need to support validating against channels for desktop experiments
  • We need to preserve the existing validation for channel for desktop experiments until we finish switching the ui over

This commit

  • Adds new tests to validate against the channels field for desktop experiments
  • Preserves existing channel validation logic for desktop experiments

fixes #13266

Becuase

* We have added a new field called channels for desktop experiments
* We need to support validating against channels for desktop experiments
* We need to preserve the existing validation for channel for desktop experiments until we finish switching the ui over

This commit

* Adds new tests to validate against the channels field for desktop experiments
* Preserves existing channel validation logic for desktop experiments

fixes #13266
Copy link
Contributor

@b4handjr b4handjr left a comment

Choose a reason for hiding this comment

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

Looks good!

@jaredlockhart jaredlockhart added this pull request to the merge queue Aug 20, 2025
Merged via the queue into main with commit c15b20b Aug 20, 2025
15 checks passed
@jaredlockhart jaredlockhart deleted the 13266 branch August 20, 2025 15:45
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.

Support channels in review serializer

2 participants