mattermost-plugin-gitlab icon indicating copy to clipboard operation
mattermost-plugin-gitlab copied to clipboard

[GH-262] Merge request approval subscription

Open MatthewDorner opened this issue 3 years ago • 8 comments

Summary

Add DM and Channel notifications for Merge Request approved and unapproved webhook actions.

Ticket Link

Fixes https://github.com/mattermost/mattermost-plugin-gitlab/issues/282

MatthewDorner avatar Jun 24 '22 18:06 MatthewDorner

Hello @MatthewDorner,

Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

mattermod avatar Jun 24 '22 18:06 mattermod

This PR has been automatically labelled "stale" because it hasn't had recent activity. A core team member will check in on the status of the PR to help with questions. Thank you for your contribution!

mattermod avatar Jul 06 '22 01:07 mattermod

Not sure why the CircleCI checks are timing out.

MatthewDorner avatar Jul 09 '22 15:07 MatthewDorner

This PR has been automatically labelled "stale" because it hasn't had recent activity. A core team member will check in on the status of the PR to help with questions. Thank you for your contribution!

mattermod avatar Jul 20 '22 01:07 mattermod

Changed the messages, anything else to do?

MatthewDorner avatar Jul 22 '22 18:07 MatthewDorner

Changed the messages, anything else to do?

@MatthewDorner Make sure you either mention someone when asking a question or re-request review from the person if that's the context of the question

mickmister avatar Jul 22 '22 21:07 mickmister

Codecov Report

Patch coverage: 100.00% and project coverage change: +0.15 :tada:

Comparison is base (21beec5) 32.27% compared to head (6bc336c) 32.43%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #307      +/-   ##
==========================================
+ Coverage   32.27%   32.43%   +0.15%     
==========================================
  Files          21       21              
  Lines        3433     3441       +8     
==========================================
+ Hits         1108     1116       +8     
  Misses       2215     2215              
  Partials      110      110              
Impacted Files Coverage Δ
server/subscription/subscription.go 95.45% <ø> (ø)
server/webhook/merge_request.go 87.75% <100.00%> (+1.08%) :arrow_up:
server/webhook/webhook.go 53.26% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

codecov[bot] avatar Jul 22 '22 21:07 codecov[bot]

This PR has been automatically labelled "stale" because it hasn't had recent activity. A core team member will check in on the status of the PR to help with questions. Thank you for your contribution!

mattermod avatar Sep 05 '22 01:09 mattermod

/update-branch

hanzei avatar Feb 22 '23 21:02 hanzei

@DHaussermann Gentle reminder to review this PR

hanzei avatar Feb 22 '23 21:02 hanzei

This PR has been automatically labelled "stale" because it hasn't had recent activity. A core team member will check in on the status of the PR to help with questions. Thank you for your contribution!

mattermost-build avatar Mar 07 '23 01:03 mattermost-build

@MatthewDorner Would you please merge master into your branch?

hanzei avatar Mar 10 '23 20:03 hanzei

@hanzei done

MatthewDorner avatar Mar 10 '23 21:03 MatthewDorner

This PR has been automatically labelled "stale" because it hasn't had recent activity. A core team member will check in on the status of the PR to help with questions. Thank you for your contribution!

mattermost-build avatar Mar 22 '23 01:03 mattermost-build

Thank you for the contribution @MatthewDorner :+1:

hanzei avatar Mar 22 '23 06:03 hanzei