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

Api key management #615

Merged
merged 30 commits into from
Sep 23, 2024
Merged

Api key management #615

merged 30 commits into from
Sep 23, 2024

Conversation

dpatel017
Copy link
Contributor

No description provided.

@dpatel017 dpatel017 marked this pull request as draft September 13, 2024 16:31
@dpatel017
Merge branch 'main' into feature/dp-256-generate-api-keys
778b14b
@dpatel017
Added migration for new column
0030c08
@dpatel017
Added repository method to get list of Api keys
fd2a0be
@dpatel017
Merge branch 'main' into feature/dp-256-generate-api-keys
3bf0991
@dpatel017
Added model
9655202
@dpatel017
Added use case to get Api keys based on Organisation id
cfad020
@dpatel017
Added use case to create new api key
feb005d
@dpatel017
Merge branch 'main' into feature/dp-256-generate-api-keys
46521c8
@dpatel017
Merge branch 'main' into feature/dp-256-generate-api-keys
1f7dad3
@dpatel017
Merge branch 'main' into feature/dp-256-generate-api-keys
f82a291
@dpatel017
Merge branch 'main' into feature/dp-256-generate-api-keys
a2b63b8
@dpatel017
Added test to register authentication key
b6f8e30
@dpatel017
Merge branch 'main' into feature/dp-256-generate-api-keys
3859f1b
@dpatel017
Added Register Authentication key endpoint in api
45124c4
@dpatel017
updated return type of endpoints
310387f
@dpatel017
Added Patch endpoint to revoke authentication key
253b189 
Added repo method
Added test for repo & usecase
@dpatel017
Merge branch 'main' into feature/dp-256-generate-api-keys
e86690a
@dpatel017
Added revoke unctionality
6e30414 
Api key listing
unit test
@dpatel017
Merge branch 'main' into feature/dp-256-generate-api-keys
c1b56dc
@dpatel017
APIKey Name already exist check while saving
da4bf2a
@dpatel017
Merge branch 'main' into feature/dp-256-generate-api-keys
d700c77
@dpatel017
Added authorize policy for editor role
3e5a651
@dpatel017 dpatel017 marked this pull request as ready for review September 23, 2024 08:14
jakzal
jakzal approved these changes Sep 23, 2024
View reviewed changes
Copy link
Member

@jakzal jakzal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved, but left few comments to consider before the merge.

@dpatel017 dpatel017 merged commit 8708d07 into main Sep 23, 2024
8 checks passed
@dpatel017 dpatel017 deleted the feature/dp-256-generate-api-keys branch September 23, 2024 12:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jakzal jakzal jakzal approved these changes

@JBaigGoaco JBaigGoaco JBaigGoaco 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.
4 participants
@dpatel017 @jakzal @JBaigGoaco @dharmverma