Skip to content
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.

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

[Notification] Add group placeholder for title #3538

Closed
1 task done
felixschndr opened this issue Aug 8, 2023 · 3 comments · Fixed by #3414
Closed
1 task done

[Notification] Add group placeholder for title #3538

felixschndr opened this issue Aug 8, 2023 · 3 comments · Fixed by #3414
Labels
feature-request Request for new features to be added

Comments

@felixschndr
Copy link

⚠️ Please verify that this feature request has NOT been suggested before.

  • I checked and didn't find similar feature request

🏷️ Feature Request Type

Other

🔖 Feature description

I love the integration of Groups and use them a lot. For this I'd like to be able to include the Group in the Notification.

Example:
I have the following setup:
image
I have monitors for two nextcloud instances. So there is

  • a group for nextcloud
    • a group for instance 1
      • a monitor for the service itself
      • a monitor for the logviewer of the website
    • a group for instance 2
      • a monitor for the service itself
      • a monitor for the logviewer of the website

Now, when I get a notification I only get the title of the monitor that has changed. So to be able to identify which monitor (in this case instance 1 or 2?) changed I have to include the whole title in the service name. However as a consequence the name of the monitors are long and duplicate what the group names already describe.

✔️ Solution

Add a new placeholder for the group name
image
This is for email. I am unsure about other notification services, something to discuss...

❓ Alternatives

No response

📝 Additional Context

No response

@CommanderStorm
Copy link
Collaborator

CommanderStorm commented Aug 8, 2023

Should I mark it as resolved by #3414? (said PR gives you the ability to access this template-variable)
If you agree, could you please close this Issue, as duplicates only create immortal zombies and are really hard to issue-manage?

@felixschndr
Copy link
Author

Yes, thank you. I didn't see that PR

@felixschndr felixschndr closed this as not planned Won't fix, can't repro, duplicate, stale Aug 8, 2023
@felixschndr
Copy link
Author

If I did the closing wrong, please correct me/it. I am unsure about it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-request Request for new features to be added
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants