-
Notifications
You must be signed in to change notification settings - Fork 675
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
Secret approval with groups #2061
Conversation
@maidul98 After review, merge when your also available as this is migration change |
@sheensantoscapadngan ping me in slack when you are done with the review and i'll skim over it before deploy |
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.
backend/src/ee/services/secret-approval-policy/secret-approval-policy-service.ts
Show resolved
Hide resolved
backend/src/ee/services/secret-approval-policy/secret-approval-policy-service.ts
Show resolved
Hide resolved
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.
Good idea to go from project membership to users. We'll need to do something similar for access requests (not yet though)
Description 📣
This PR enables secret approval to work with users part of groups. Instead of project membership we now take user id in secret approval. The api now returns the corresponding user details for various user entities like statusChangeBy, committer etc
Type ✨
Tests 🛠️
# Here's some code block to paste some code snippets