-
Notifications
You must be signed in to change notification settings - Fork 9
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
Create "App" Schema/CRUD then Link API Keys and Roles #95
Labels
enhancement
New feature or enhancement of existing functionality
priority-2
Second highest priority, should be worked on as soon as the Priority-1 issues are finished
T1d
Time Estimate 1 Day
technical
A technical issue that requires understanding of the code, infrastructure or dependencies
Comments
nelsonic
added
enhancement
New feature or enhancement of existing functionality
priority-2
Second highest priority, should be worked on as soon as the Priority-1 issues are finished
T1h
Time Estimate 1 Hour
technical
A technical issue that requires understanding of the code, infrastructure or dependencies
labels
Aug 26, 2020
After reading through the code that creates API keys, I've realised that it's a bit more work than |
nelsonic
changed the title
Add app_id to role schema/table to separate roles for different apps
Create "App" Schema/CRUD then Link API Keys and Roles
Aug 26, 2020
1 task
nelsonic
added a commit
that referenced
this issue
Sep 1, 2020
nelsonic
added a commit
that referenced
this issue
Sep 1, 2020
nelsonic
added a commit
that referenced
this issue
Sep 1, 2020
nelsonic
added a commit
that referenced
this issue
Sep 1, 2020
nelsonic
added a commit
that referenced
this issue
Sep 1, 2020
nelsonic
added a commit
that referenced
this issue
Sep 1, 2020
13 tasks
nelsonic
added a commit
that referenced
this issue
Sep 1, 2020
This was referenced Sep 7, 2020
Closed
nelsonic
added a commit
that referenced
this issue
Sep 12, 2020
nelsonic
added a commit
that referenced
this issue
Sep 12, 2020
nelsonic
added a commit
that referenced
this issue
Sep 12, 2020
nelsonic
added a commit
that referenced
this issue
Sep 12, 2020
This issue is complete. #85 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
enhancement
New feature or enhancement of existing functionality
priority-2
Second highest priority, should be worked on as soon as the Priority-1 issues are finished
T1d
Time Estimate 1 Day
technical
A technical issue that requires understanding of the code, infrastructure or dependencies
While preparing to create the function that returns the List of Roles for an App https://github.com/dwyl/rbac/issues/3,
I noted that we don't actually have the concept of different "Applications" in Auth right now ... 🙄
Our original idea for the Auth API Key #42 was to allow people to run the DWYL App on their
localhost
without having to setup an Auth server. That goal was achieved. ✅However if we want Auth to be a turnkey multi-app auth solution, we need the concept of "Apps".
Gonna transfer a few of the fields over from the API key specifically:
auth/lib/auth/apikey.ex
Lines 9 to 16 in fafc5e0
Todo
Create
App
schemaperson
apikeys
Migrate data from API Key to App
Update
apikeys
schema to have abelongs_to
so theapp
owns the key.Once this is done we will need to:
add
app_id
to Roles so that we can have distinct roles for different applicationsand can create unlimited Apps/Roles. Add app_id to roles schema #108
Add
app_id
to Roles table in/people
(index view) and/people/:id
The text was updated successfully, but these errors were encountered: