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

Value for tuples in transactional writes is incorrect #815

Open
rhamzeh opened this issue Aug 19, 2024 · 1 comment
Open

Value for tuples in transactional writes is incorrect #815

rhamzeh opened this issue Aug 19, 2024 · 1 comment
Labels
bug Something isn't working documentation Improvements or additions to documentation

Comments

@rhamzeh
Copy link
Member

rhamzeh commented Aug 19, 2024

We mention the limit to be 10 here: https://github.com/openfga/openfga.dev/blob/main/docs/content/interacting/transactional-writes.mdx?plain=1#L141

In fact, by default it is 100

https://github.com/openfga/openfga/blob/main/internal/server/config/config.go#L19

Please mention that it can be configured and link to the config schema: https://github.com/openfga/openfga/blob/main/.config-schema.json#L3-L7

@rhamzeh rhamzeh added bug Something isn't working documentation Improvements or additions to documentation labels Aug 19, 2024
@VividhPandey003
Copy link

Hi @rhamzeh Can I work on this under Hacktoberfest.

From what I understood from the issue description is that since the limit for the max tuples per write can be configured from the schema, the line 139 'The Write API allows you to send up to 100 unique tuples in the request. ' wrongly defines the limit to be 100.

We would need to edit that to properly communicate that the default value is 100 but it can be configured as per the dev's requirements

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working documentation Improvements or additions to documentation
Projects
None yet
Development

No branches or pull requests

2 participants