Skip to content

Add capability to control GitLab features per project or group #1008

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

filipowm
Copy link
Contributor

@filipowm filipowm commented Feb 5, 2020

Currently GitLab Features API supports configuring features not only globally or per user, but also per group or project. python-gitlab is currently missing this option, thus this PR.

Additionally fixed issue when setting up feature with None (null) values, e.g. none feature_group or user. It caused GitLab to respond with HTTP 500. Thus now we're filtering out None values from data.

@filipowm filipowm requested a review from max-wittig February 5, 2020 13:16
@filipowm filipowm force-pushed the feature/feature-flags-additional-config branch 2 times, most recently from 4f4c4ef to 739666f Compare February 5, 2020 13:27
@filipowm
Copy link
Contributor Author

filipowm commented Feb 7, 2020

@max-wittig Could you please take a look?
This one is currently blocking PR in gitlab configuration-as-code tool: Roche/gitlab-configuration-as-code#11

@max-wittig
Copy link
Member

@filipowm Thanks for the MR. Could you please drop the duplicate commits from master and rebase instead?

@filipowm filipowm force-pushed the feature/feature-flags-additional-config branch from ad2ed38 to 1ec1816 Compare February 7, 2020 10:00
@filipowm
Copy link
Contributor Author

filipowm commented Feb 7, 2020

@filipowm Thanks for the MR. Could you please drop the duplicate commits from master and rebase instead?

Done. I missed to notice that :)

@max-wittig max-wittig merged commit 066fc9b into python-gitlab:master Feb 10, 2020
@max-wittig
Copy link
Member

@filipowm Thanks for your efforts!

@filipowm filipowm deleted the feature/feature-flags-additional-config branch February 14, 2020 07:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants