Skip to content

Added a message to create a subscription after running the pipeline (#341)#445

Closed
avas27JTG wants to merge 6 commits intomattermost:masterfrom
Brightscout:MI-3583
Closed

Added a message to create a subscription after running the pipeline (#341)#445
avas27JTG wants to merge 6 commits intomattermost:masterfrom
Brightscout:MI-3583

Conversation

@avas27JTG
Copy link
Copy Markdown
Contributor

@avas27JTG avas27JTG commented Jan 18, 2024

Summary

Added a message to create a subscription after running the pipeline (#341)

Screenshot

  • In case of no existing subscription for the given project
    image (1)

  • In case of an existing subscription for the given project but a different feature than the pipeline
    image (2)

Ticket Link

Fixes #341

@codecov
Copy link
Copy Markdown

codecov bot commented Jan 18, 2024

Codecov Report

Attention: 22 lines in your changes are missing coverage. Please review.

Comparison is base (ee780c1) 33.40% compared to head (ef80e1d) 33.21%.

Files Patch % Lines
server/command.go 0.00% 22 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #445      +/-   ##
==========================================
- Coverage   33.40%   33.21%   -0.19%     
==========================================
  Files          22       22              
  Lines        3979     4001      +22     
==========================================
  Hits         1329     1329              
- Misses       2519     2541      +22     
  Partials      131      131              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@avas27JTG avas27JTG added this to the v1.9.0 milestone Jan 18, 2024
@avas27JTG avas27JTG changed the title Mi 3583 Added a message to create a subscription after running the pipeline (#341) Jan 18, 2024
@mickmister
Copy link
Copy Markdown
Contributor

I think this is already implemented here right? #433

@raghavaggarwal2308
Copy link
Copy Markdown

@mickmister I think there was some confusion here. We can close this PR and continue with #433 as that is already in the review process.

@mickmister mickmister closed this Jan 25, 2024
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.

Triggering a pipeline does not automatically create a subscription to pipeline events

4 participants