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.

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

feat: add deployment approval config attribute #2022

Conversation

loozhengyuan
Copy link
Contributor

The allow_pipeline_trigger_approve_deployment attribute exists on the Project entity and is returned in the Get Project API1 even though it is not documented.

Related: #1693

The `allow_pipeline_trigger_approve_deployment` attribute exists on the
Project entity and is returned in the Get Project API[1] even though it
is not documented.

[1]: https://docs.gitlab.com/ee/api/projects.html#get-a-single-project
@loozhengyuan
Copy link
Contributor Author

Apart from the diffs in the original GitLab issue that implemented this change, I also wanted to provide screenshots that this attribute indeed exists on the entity itself. When retrieving a single GitLab project, we can observe that this attribute is returned:

Screenshot 2024-10-04 at 9 19 55 AM

I have created a separate ticket in GitLab to update the documentation: https://gitlab.com/gitlab-org/gitlab/-/issues/497730.

Copy link
Member

@svanharmelen svanharmelen left a comment

Choose a reason for hiding this comment

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

While I normally don't add undocumented fields, I'm OK with this one 👍🏻 But please do follow through on the doc update at GitLab as well...

@loozhengyuan
Copy link
Contributor Author

@svanharmelen Thank you, I appreciate it! I will also report back once GitLab Docs are updated as well, just to close the loop.

@svanharmelen svanharmelen merged commit 8a715ef into xanzy:main Oct 4, 2024
3 checks passed
@loozhengyuan loozhengyuan deleted the feat/add-allow-pipeline-trigger-approve-deployment branch October 4, 2024 11:29
@loozhengyuan
Copy link
Contributor Author

loozhengyuan commented Oct 10, 2024

@svanharmelen Just to close the loop: the updates to GitLab docs have been merged in, thanks!

@svanharmelen
Copy link
Member

Thanks for closing the loop! 👍🏻

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