Skip to content

feat(notifications): Support for tags #591

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

feat(notifications): Support for tags #591

wants to merge 2 commits into from

Conversation

hamir-suspect
Copy link
Contributor

📝 Description

✅ Checklist

  • I have tested this change
  • This change requires documentation update

@hamir-suspect
feat(front): Support tags for notifications filters (#586)
0dc4235 
## 📝 Description
- refreshes protos
- adds tag field in notifications form

<img width="416" height="164" alt="image"
src="https://github.com/user-attachments/assets/8d892f05-936b-4a32-9d62-716d81f32c13"
/>


## ✅ Checklist
- [x] I have tested this change
- [ ] This change requires documentation update
@hamir-suspect
feat(notifications): Add tags git ref type to notification filters (#585
af351bd 
)

## 📝 Description
- refresh protos
- implement the updated API semaphoreci/api#39
(this API is also used by `front`)
- adds the filtering by tag rule if `git_ref_type == :TAG` so both
branch and tag rules can be defined in same notification

## ✅ Checklist
- [ ] I have tested this change
- [ ] This change requires documentation update
@github-project-automation github-project-automation bot moved this to Backlog in Roadmap Sep 19, 2025
@hamir-suspect hamir-suspect changed the title Feat/notifications tags feat(notifications): Support for tags Sep 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@VeljkoMaksimovic VeljkoMaksimovic Awaiting requested review from VeljkoMaksimovic VeljkoMaksimovic will be requested when the pull request is marked ready for review VeljkoMaksimovic is a code owner

@DamjanBecirovic DamjanBecirovic Awaiting requested review from DamjanBecirovic DamjanBecirovic will be requested when the pull request is marked ready for review DamjanBecirovic is a code owner

@skipi skipi Awaiting requested review from skipi skipi will be requested when the pull request is marked ready for review skipi is a code owner

@forestileao forestileao Awaiting requested review from forestileao forestileao will be requested when the pull request is marked ready for review forestileao is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
Roadmap
Status: Backlog
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@hamir-suspect