Skip to content

Update PR details after merge#1185

Merged
kuychaco merged 2 commits intomasterfrom
kuychaco/update-after-merge
May 30, 2019
Merged

Update PR details after merge#1185
kuychaco merged 2 commits intomasterfrom
kuychaco/update-after-merge

Conversation

@kuychaco
Copy link
Contributor

Fixes #1183

This PR updates the PR details page to reflect the fact that the PR has been merged after the "Merge" button is clicked. It also disables the merge button while the operation is in progress to prevent multiple merge requests.

@msftclas
Copy link

msftclas commented May 30, 2019

CLA assistant check
All CLA requirements met.

Copy link
Contributor

@RMacfarlane RMacfarlane left a comment

Choose a reason for hiding this comment

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

LGTM!

@kuychaco kuychaco merged commit 66a11bf into master May 30, 2019
@kuychaco kuychaco deleted the kuychaco/update-after-merge branch May 30, 2019 17:42
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.

PR details page doesn't update after merge

3 participants