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.

Sign up for GitHub

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

Jump to bottom

feat: add type restrictions to dsl syntax #8

Merged
merged 10 commits into from
Dec 14, 2022
Merged

feat: add type restrictions to dsl syntax #8

merged 10 commits into from
Dec 14, 2022

Conversation

craigpastro
Copy link

@craigpastro craigpastro commented Oct 13, 2022 •
edited by matthewpereira
Loading

Description

Type restrictions on an authorization model will allow us to restrict the types of users that may be directly related to (type, relation) pairs.

Rendered Version

Click here to view the RFC as a rendered preview

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

aaguiarz
aaguiarz requested changes Oct 13, 2022
View reviewed changes
@adriantam adriantam mentioned this pull request Nov 7, 2022
Closed
@matthewpereira matthewpereira changed the title feat: add type restrictions to dsl feat: add type restrictions to dsl syntax Nov 8, 2022
@matthewpereira matthewpereira mentioned this pull request Nov 8, 2022
Merged
4 tasks
@adriantam adriantam mentioned this pull request Nov 16, 2022
Closed
@adriantam adriantam marked this pull request as ready for review December 14, 2022 16:08
@adriantam adriantam merged commit 800c5c4 into main Dec 14, 2022
@adriantam adriantam deleted the type-restriction-dsl branch December 14, 2022 16:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jon-whit jon-whit jon-whit left review comments

@miparnisari miparnisari miparnisari left review comments

@aaguiarz aaguiarz aaguiarz approved these changes

@adriantam adriantam adriantam approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@craigpastro @aaguiarz @jon-whit @miparnisari @adriantam