Skip to content

Conversation

raviks789
Copy link
Contributor

@raviks789 raviks789 commented Jun 20, 2024

fix #212

requires Icinga/ipl-web#225

@cla-bot cla-bot bot added the cla/signed CLA is signed by all contributors of a PR label Jun 20, 2024
@raviks789 raviks789 self-assigned this Jun 20, 2024
Copy link
Contributor

@ncosta-ic ncosta-ic left a comment

Choose a reason for hiding this comment

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

We should stay consistent and use Icinga\Module\Notifications\Common\Links for path references to strengthen the maintainability.

@raviks789 raviks789 force-pushed the fix/active-list-item-handling branch from 3e84c7a to 2653a53 Compare June 21, 2024 08:35
ncosta-ic

This comment was marked as duplicate.

ncosta-ic
ncosta-ic previously approved these changes Jun 21, 2024
@raviks789 raviks789 force-pushed the fix/active-list-item-handling branch from 2653a53 to 68f7fd1 Compare June 26, 2024 08:15
@raviks789 raviks789 requested a review from nilmerg June 26, 2024 13:14
@nilmerg
Copy link
Member

nilmerg commented Jan 21, 2025

Requires a rebase.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla/signed CLA is signed by all contributors of a PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Track active items in lists properly
3 participants