Skip to content

feat: added type filter on get feature endpoint#7598

Merged
Zaimwa9 merged 1 commit into
mainfrom
feat/allow-filtering-per-feature-type-multivariate
May 26, 2026
Merged

feat: added type filter on get feature endpoint#7598
Zaimwa9 merged 1 commit into
mainfrom
feat/allow-filtering-per-feature-type-multivariate

Conversation

@Zaimwa9
Copy link
Copy Markdown
Contributor

@Zaimwa9 Zaimwa9 commented May 26, 2026

Thanks for submitting a PR! Please check the boxes below:

  • I have read the Contributing Guide.
  • I have added information to docs/ if required so people know about the feature.
  • I have filled in the "Changes" section below.
  • I have filled in the "How did you test this code" section below.

Changes

  • Added a new query param type to allow filtering on STANDARD and MULTIVARIATE feature types
  • Needed to get proper lists of multivariate features for experimentation

How did you test this code?

  • Added tests

@Zaimwa9 Zaimwa9 requested a review from matthewelwell May 26, 2026 10:59
@Zaimwa9 Zaimwa9 requested a review from a team as a code owner May 26, 2026 10:59
@vercel
Copy link
Copy Markdown

vercel Bot commented May 26, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

3 Skipped Deployments
Project Deployment Actions Updated (UTC)
docs Ignored Ignored May 26, 2026 10:59am
flagsmith-frontend-preview Ignored Ignored May 26, 2026 10:59am
flagsmith-frontend-staging Ignored Ignored May 26, 2026 10:59am

Request Review

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request introduces the ability to filter features by type (STANDARD or MULTIVARIATE) in the feature query API. It updates the FeatureQuerySerializer to accept a type parameter, applies this filter to the queryset in the view, and adds corresponding integration tests to verify the functionality. There are no review comments, so I have no feedback to provide.

@Zaimwa9 Zaimwa9 merged commit 18230bf into main May 26, 2026
6 checks passed
@Zaimwa9 Zaimwa9 deleted the feat/allow-filtering-per-feature-type-multivariate branch May 26, 2026 11:10
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.

2 participants