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

Rename the plugin and and update readme #113

Open
DHaussermann opened this issue Feb 2, 2023 · 2 comments
Open

Rename the plugin and and update readme #113

DHaussermann opened this issue Feb 2, 2023 · 2 comments
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed

Comments

@DHaussermann
Copy link

Maybe this is far more obvious for the intended obvious but to people not familiar with Prometheus AlertManager it may not be obvious what this plugin does.
I thought this plugin maybe communicated with Prometheus directly or maybe even with Graphana somehow (which also has an add-on called AlertManager). From what I see, the read-me here https://github.com/cpanato/mattermost-plugin-alertmanager#readme does not explicitly say that it communicates Prometheus AlertManager it's only implied by the screen shots.

Therefore I would suggest we rename the plugin to prometheus-alertmanager
and Update the top level header in the screenshot here.
image

@DHaussermann
Copy link
Author

I don't have permissions to label things in tis repo but this in an enhancement request.

@cpanato
Copy link
Owner

cpanato commented Mar 13, 2023

are you willing to update the readme for that?

@cpanato cpanato added enhancement New feature or request help wanted Extra attention is needed good first issue Good for newcomers labels Mar 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants