-
Notifications
You must be signed in to change notification settings - Fork 5
feat(gitlab): add builtin admin roles to GitLab #1899
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
Conversation
|
🤖 Hey ! The @cpn-console/keycloak-plugin (v2.1.0) package already exists on npm but the source code has changed, you should consider updating the package version. The version update warning should be ignored in the case of modifications that do not affect the code once it has been built, such as code formatting, etc... |
|
🤖 Hey ! The @cpn-console/hooks (v4.0.0) package already exists on npm but the source code has changed, you should consider updating the package version. The version update warning should be ignored in the case of modifications that do not affect the code once it has been built, such as code formatting, etc... |
|
🤖 Hey ! The @cpn-console/shared (v1.3.0) package already exists on npm but the source code has changed, you should consider updating the package version. The version update warning should be ignored in the case of modifications that do not affect the code once it has been built, such as code formatting, etc... |
|
🤖 Hey ! The @cpn-console/gitlab-plugin (v3.3.1) package already exists on npm but the source code has changed, you should consider updating the package version. The version update warning should be ignored in the case of modifications that do not affect the code once it has been built, such as code formatting, etc... |
|
🤖 Hey ! The security scan report for the current pull request is available here. |
2f2f707 to
5e85bcd
Compare
17b2050 to
ef2ce4a
Compare
|
9bea6b4 to
2207f28
Compare
e779210 to
23a9d4c
Compare
For some reasons, the config is never passed down to the plugins hooks. Apparently the config is provided via the adminPlugin table of the database but is empty most of the time so for the moment let's just add default and fix the problem properly later. Signed-off-by: William Phetsinorath <[email protected]>
14d6583 to
38366fd
Compare
We don't want drift to be an option in the future, so we'll keep the purge option as an admin only. Signed-off-by: William Phetsinorath <[email protected]>
Signed-off-by: William Phetsinorath <[email protected]>
|
Merged into #1910 |

2 New Issues
0 Fixed Issues
0 Accepted Issues
Signed-off-by: William Phetsinorath [email protected]
Stack created with Sapling. Best reviewed with ReviewStack.