Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bugfix/fixing paginated query 2018 #2034

Merged

Conversation

ppratikcr7
Copy link
Collaborator

This fixes #2018

Removing left join for featureFlagSegmentInclusion from the paginated feature flags query and adding those separately after the skip take gets 20 records of feature flags for the paginated call.

dependabot bot and others added 3 commits October 4, 2024 16:43
…1893)

Bumps [actions/download-artifact](https://github.com/actions/download-artifact) from 2 to 4.1.7.
- [Release notes](https://github.com/actions/download-artifact/releases)
- [Commits](actions/download-artifact@v2...v4.1.7)

---
updated-dependencies:
- dependency-name: actions/download-artifact
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: danoswaltCL <[email protected]>
Co-authored-by: Zack Lee <[email protected]>
@ppratikcr7
Copy link
Collaborator Author

@danoswaltCL Wanted to confirm again, are we willing to merge this PR to v6-pre-release branch? So, kept this in draft, so that no one merge it by mistake.

@danoswaltCL
Copy link
Collaborator

let's go ahead and get it in there. also, you've included a different change which is the actions/[email protected]... but that's ok because I was going to open a PR to pull that into the release branch anyway, so it can stay.

@ppratikcr7 ppratikcr7 marked this pull request as ready for review October 9, 2024 12:35
@ppratikcr7 ppratikcr7 merged commit 27afa72 into release/v6.0.0-prerelease Oct 9, 2024
14 checks passed
@ppratikcr7 ppratikcr7 deleted the bugfix/fixing-paginated-query-2018 branch October 9, 2024 13:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

FF Paginated returns wrong number of flags due to SQL producing duplicates?
3 participants