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

Require relations to be defined in the namespace config #509

Closed
zepatrik opened this issue Mar 25, 2021 · 4 comments
Closed

Require relations to be defined in the namespace config #509

zepatrik opened this issue Mar 25, 2021 · 4 comments
Assignees
Labels
help wanted We are looking for help on this one.

Comments

@zepatrik
Copy link
Member

zepatrik commented Mar 25, 2021

Is your feature request related to a problem? Please describe.

Currently the namespace config is useless. Later on we will define relations and their effect there: #303

Describe the solution you'd like

For starters we should require relations to be defined in the namespace config. This will allow a seamless migration later on and prevents typos and other small errors.

This feature does not impact the current functionality.

@zepatrik zepatrik added the blocking Blocks milestones or other issues or pulls. label Mar 25, 2021
@zepatrik zepatrik self-assigned this Mar 25, 2021
@zepatrik zepatrik added help wanted We are looking for help on this one. and removed blocking Blocks milestones or other issues or pulls. labels Oct 8, 2021
@xbitres
Copy link

xbitres commented Sep 1, 2022

Isn't this needed in order to have inheritance between relations (example: owner is equivalent of read and write relations) or do you see it implemented in another way?

@zepatrik
Copy link
Member Author

zepatrik commented Sep 1, 2022

It is not strictly needed. See #877 for how we implemented that. This issue was rather about validation pre rewrites, so we can actually close this.

@zepatrik zepatrik closed this as completed Sep 1, 2022
@xbitres
Copy link

xbitres commented Sep 5, 2022

Thank you for your answer. It would be cool that the documentation is updated with an example on how to use it, as it was tricky for me to find the issue

@zepatrik
Copy link
Member Author

zepatrik commented Sep 6, 2022

Yes, we will very soon update a lot of docs around namespace config, e.g. ory/docs#948
Do you have any specific suggestions? What page were you struggling with?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted We are looking for help on this one.
Projects
None yet
Development

No branches or pull requests

2 participants