-
Notifications
You must be signed in to change notification settings - Fork 17
Add contextual tuples to Expand API #202
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
Add contextual tuples to Expand API #202
Conversation
|
b0500ee
to
c4517f4
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
After applying the suggestion I made, please run make
before commiting. Thanks!
c4517f4
to
22fde33
Compare
@sujitha-av looks like there is a conflict with main. Could you please |
@miparnisari please make sure not to merge this until the relevant openfga/openfga PR is ready to be merged, otherwise autogenerated SDKs & Docs would be wrong |
22fde33
to
af11c9d
Compare
be89aa5
to
50d1c0a
Compare
* fix: only retry requests up to 3 times by default (reduced from 15) Generator PR: openfga/sdk-generator#420 * chore(docs): replace readable names with uuid to discourage storing PII in OpenFGA Generator PR: openfga/sdk-generator#433 * chore: update api definition, support assertion context & contextual tuples * chore!: update the name request & response interfaces of BatchCheck This is in preparation to support server-side BatchCheck * feat: add contextual tuples to Expand API api pr: openfga/api#202 * feat: add support for start time in read changes request * fix: send the start time to ReadChanges in the format the API expects * fix: list relations should not swallow errors closes #95 * feat: support contextual tuples in Expand * chore(docs): update CHANGELOG with unreleased changes * feat!: add support for filtering stores by name --------- Co-authored-by: Ovindu Atukorala <ovindu.atukorala@gmail.com> Co-authored-by: sccalabr <4111230+sccalabr@users.noreply.github.com> Co-authored-by: Ryan Quinn <ryanpq@gmail.com> Co-authored-by: Ewan Harris <ewanharris93@gmail.com>
Description
Based on this issue in the openfga repo, it was required to add contextual tuples support to the Expand API. And @miparnisari had broken down the steps for the same. This PR does the first step mentioned, to add contextual tuples to the Expand API.
The technical documentation has been skipped.
References
Reference issue
Review Checklist
main