Skip to content

Add bulk/manage_notifications endpoint - #1414

Merged
synkd merged 3 commits into
SatelliteQE:masterfrom
LadislavVasina1:ManageNotificationsCoverage
Mar 30, 2026
Merged

Add bulk/manage_notifications endpoint#1414
synkd merged 3 commits into
SatelliteQE:masterfrom
LadislavVasina1:ManageNotificationsCoverage

Conversation

@LadislavVasina1

@LadislavVasina1 LadislavVasina1 commented Mar 30, 2026

Copy link
Copy Markdown
Contributor

theforeman/foreman#10919 introduces new bulk/manage_notifications.
This PR adds this functionality to Nailgun.

This PR and nailgun function is tested in this robottelo PR: SatelliteQE/robottelo#21139

@LadislavVasina1 LadislavVasina1 self-assigned this Mar 30, 2026
@LadislavVasina1 LadislavVasina1 added No-CherryPick PR doesnt need CherryPick to previous branches Stream labels Mar 30, 2026

@sourcery-ai sourcery-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Hey - I've reviewed your changes and they look great!


Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

@LadislavVasina1

Copy link
Copy Markdown
Contributor Author

PRT passed in SatelliteQE/robottelo#21139

ColeHiggins2
ColeHiggins2 previously approved these changes Mar 30, 2026

@synkd synkd left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

One non-blocking suggestion, otherwise ACK.

Comment thread nailgun/entities.py Outdated
@synkd
synkd merged commit 648b4c7 into SatelliteQE:master Mar 30, 2026
17 of 20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

No-CherryPick PR doesnt need CherryPick to previous branches PRT-Passed Stream

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants