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

fix: maximum number of contextual tuples for ListUsersRequest #166

Merged
merged 1 commit into from
May 28, 2024

Conversation

willvedd
Copy link
Contributor

@willvedd willvedd commented May 28, 2024

Description

Adding a 20 contextual tuple maximum for ListUsers requests. This enforcement usually happens on the ContextualTupleKeys definition but ListUsers decided to diverge to eliminate the stuttering that occurs from that type.

Test for this will be implemented in openfga/openfga#1433.

References

Related PR: openfga/openfga#1433

Review Checklist

  • I have clicked on "allow edits by maintainers".
  • I have added documentation for new/changed functionality in this PR or in a PR to openfga.dev [Provide a link to any relevant PRs in the references section above]
  • The correct base branch is being used, if not main
  • I have added tests to validate that the change in functionality is working as expected

@willvedd willvedd requested review from a team as code owners May 28, 2024 15:39
@willvedd willvedd merged commit 03aaac2 into main May 28, 2024
7 checks passed
@willvedd willvedd deleted the list-users-max-20-contextual-tuples branch May 28, 2024 16:09
@willvedd willvedd mentioned this pull request May 28, 2024
4 tasks
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.

3 participants