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

Load pull request draft status from GraphQL API #742

Merged
merged 1 commit into from
Mar 26, 2024

Conversation

bluekeyes
Copy link
Member

Back in 2019, we disabled this field because it was part of a GraphQL preview. As a result, IsDraft was always false for PRs that we loaded via the GraphQL query. For the most part, this doesn't matter, but can cause problems with reviewer requests triggered via webhooks (like status events) that do not provide complete pull request details.

This field has been out of preview for a long time now and is available in all supported versions of GitHub, so it should be safe to load.

Fixes #741.

Back in 2019, we disabled this field because it was part of a GraphQL
preview. As a result, IsDraft was always false for PRs that we loaded
via the GraphQL query. For the most part, this doesn't matter, but can
cause problems with reviewer requests triggered via webhooks (like
status events) that do not provide complete pull request details.

This field has been out of preview for a long time now and is available
in all supported versions of GitHub, so it should be safe to load.
@bluekeyes bluekeyes requested a review from a team March 26, 2024 19:43
Copy link
Member

@asvoboda asvoboda left a comment

Choose a reason for hiding this comment

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

It's always nice fixing an old "temporary" fix from a few years ago (#125)

@bluekeyes bluekeyes merged commit 1000bcf into develop Mar 26, 2024
7 checks passed
@bluekeyes bluekeyes deleted the bkeyes/pull-draft-field branch March 26, 2024 23:08
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.

has_successful_status causes review requests while PR has draft status
2 participants