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.

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

Polish: Remove any mention of slack #140

Closed
hmpf opened this issue Sep 22, 2020 · 0 comments · Fixed by #141
Closed

Polish: Remove any mention of slack #140

hmpf opened this issue Sep 22, 2020 · 0 comments · Fixed by #141
Labels
backend Needs/has companion issue in backend polish Nice to have
Milestone
Version 1.0

Comments

@hmpf
Copy link
Contributor

hmpf commented Sep 22, 2020

We're not supporting Slack yet, so remove it everywhere.

Problem: ProfileList.mediaOptions is hard coded. For media where everything can be configured simply (like for email or sms since there is table support for these) these should be looked up via an endpoint. If the configuration of media is made very dynamic and possible via this front-end, a new front-end must be released for every new notification media. Something to keep in mind for Uninett/Argus#121.
@hmpf hmpf added backend Needs/has companion issue in backend polish Nice to have labels Sep 22, 2020
This was referenced Sep 22, 2020
Closed
Merged
@hmpf hmpf added this to the Version 1 milestone Sep 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
backend Needs/has companion issue in backend polish Nice to have
Projects
None yet
Milestone
Version 1.0
Development

Successfully merging a pull request may close this issue.

Polish: Remove mentions of Slack hmpf/Argus-frontend
1 participant
@hmpf