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

Make it clear on web interface that a list is being moderated #638

Merged
merged 1 commit into from
Jun 26, 2019

Conversation

ldidry
Copy link
Contributor

@ldidry ldidry commented Jun 5, 2019

Fix #636

Adds an alert-box with the text "List not activated yet. Please wait until a listmaster activated it to be able to use it." below list subject.

@racke
Copy link
Contributor

racke commented Jun 5, 2019

Adding the alert box is a good idea, however I don't like the second sentence. We can't give a clue when the list is activated - maybe it never happens. What about "Please contact your listmaster for further questions."

@ldidry
Copy link
Contributor Author

ldidry commented Jun 5, 2019

I’m agnostic on that. @SansPseudoFix, what do you think?

@SansPseudoFix
Copy link

I understand your concern but if we replace it by "Please contact your listmaster for further questions.", I'm afraid we will receive a lot of questions ("When do you think activated my list?"). Imho if people really have questions, they contact us.

@ikedas
Copy link
Member

ikedas commented Jun 10, 2019

I support @SansPseudoFix.

@ikedas
Copy link
Member

ikedas commented Jun 26, 2019

I think we'd be better to merge this pr and then think about better text. Is that ok?

@ldidry
Copy link
Contributor Author

ldidry commented Jun 26, 2019

Sure: anything is better than nothing 👍

@racke
Copy link
Contributor

racke commented Jun 26, 2019

OK fair enough.

@ikedas ikedas merged commit d33863d into sympa-community:sympa-6.2 Jun 26, 2019
@ikedas
Copy link
Member

ikedas commented Jun 26, 2019

Merged. Thanks for comments.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Make it clear that a list is being moderated
4 participants