github-for-jira icon indicating copy to clipboard operation
github-for-jira copied to clipboard

Feature request: Pull request approved

Open felixveysseyre opened this issue 4 years ago • 61 comments

Hi !

Just as an automation already exists for when a pull-request is declined, it would be nice to trigger action when a pull-request is approved.

Does this feature is already part of your roadmap ?

Thanks !

felixveysseyre avatar Jun 07 '21 17:06 felixveysseyre

Hi @felixveysseyre. The 3 pull request automations currently include what's seen in the below screenshot:

Screen Shot 2021-07-14 at 4 09 09 pm

Does merging the PR support the automation you are after? If not, I can try and point you to the right team to make a feature request. We look after the GitHub for Jira integration whereas you'd need to get in contact with the automations team.

rachellerathbone avatar Jul 14 '21 06:07 rachellerathbone

Haven't heard back in a month, I'm going to close this one. Please reopen it if we haven't answered your question. Cheers.

mboudreau avatar Aug 17 '21 08:08 mboudreau

@mboudreau I'm sorry, I missed this notification and apparently I do not have sufficient permission to reopen this issue.

@rachellerathbone These three events do not fit my needs. I would like to trigger and action when the pull-request has been approved, not merged To give you some more context, we do have an automated process which deploys automatically the code of the pull-request. Once the the pull-request has been approved, we manually change (that's the behavior we would like to automate) the status of the related Jira ticket in order to let our QA engineers know about it. From here, they are able to test the related changes and merge (or not) the original pull-request

Let me know if it's sill unclear 🙇‍♂️

felixveysseyre avatar Aug 17 '21 08:08 felixveysseyre

Alright, reopening the issue. Sound like a good idea for a new event. We'll have to bring it up with the team responsible for jira automation.

mboudreau avatar Aug 17 '21 11:08 mboudreau

+1

We're looking at integrating this same workflow mentioned by @felixveysseyre and glad we found this!

iamstej avatar Nov 23 '21 10:11 iamstej

I'd appreciate this, as well.

It would be good to include information, for github integrations at least, about all outstanding reviews in all statuses, e.g. requested, approved, etc. If that doesn't make sense, then just having the number of approvals would be useful.

rileywhite avatar Dec 06 '21 14:12 rileywhite

+1 would also appreciate this.

edelegormley avatar Jan 04 '22 17:01 edelegormley

+1; some folks don't have access to GitHub and all they need to know is whether a PR has been approved yet (vs. still waiting for review). So being able to distinguish this from the Jira issue would be very useful.

simonyun avatar Jan 25 '22 20:01 simonyun

+1; we need to promote approved PRs to further development stages

drdaemos avatar Feb 11 '22 08:02 drdaemos

Surely this can't be that difficult. The info from GitHub is already synced to Jira.

https://github.com/atlassian/github-for-jira/blob/5d0e37c544031319ebf8afdb8c2e23055c461677/src/transforms/pull-request.ts#L118

The info is there to use and is already shown alongside the pull request in the development info window

(ignore the red boxes - just best screenshot I could find)

The development panel already has an implementation for review which Crucible uses: https://support.atlassian.com/jira-software-cloud/docs/view-development-information-for-an-issue/#See-the-status-of-reviews

Why is the review information shown for pull requests not surfaced in the same way?

It also means the JQL development[reviews].all doesn't work as expected

jackbentley avatar Feb 25 '22 18:02 jackbentley

+1

Would help a lot with automating our workflows.

SirSaunders avatar Mar 14 '22 20:03 SirSaunders

+1

christianfds avatar Apr 04 '22 18:04 christianfds

This would be awesome!

revitteth avatar Apr 06 '22 18:04 revitteth

+1

malenazk avatar Apr 12 '22 14:04 malenazk

+1

grancier avatar Apr 27 '22 16:04 grancier

+1

jchavjim avatar May 05 '22 14:05 jchavjim

+1

NicholasMiller avatar May 17 '22 22:05 NicholasMiller

+1

yanbrod avatar May 24 '22 11:05 yanbrod

+1, and also should this be linked to https://jira.atlassian.com/browse/JRACLOUD-71798 somehow?

msde avatar Jun 02 '22 00:06 msde

+1 on this

ncapdevi avatar Jun 06 '22 23:06 ncapdevi

+1

jenswet avatar Jun 12 '22 09:06 jenswet

+1

pietmichal avatar Jun 27 '22 12:06 pietmichal

+1

jessicaw-iology avatar Jul 14 '22 22:07 jessicaw-iology

+1

caique39 avatar Jul 19 '22 20:07 caique39

+1

coberhauser avatar Aug 16 '22 13:08 coberhauser

+1

Murzic avatar Aug 18 '22 12:08 Murzic

+1

SethNuteTwilio avatar Aug 19 '22 17:08 SethNuteTwilio

+1

danielleos avatar Aug 25 '22 11:08 danielleos

+1

TFarleyAZ avatar Aug 26 '22 19:08 TFarleyAZ

+1

bquay avatar Sep 02 '22 15:09 bquay