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

Mailing lists check functionnality #30

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Mailing lists check functionnality #30

wants to merge 2 commits into from

Conversation

eltrai
Copy link
Member

@eltrai eltrai commented Feb 26, 2017

This PR is a follow-up from #25 that fixes the check list functionnality. This will allow to correct settings that have diverged from the default directly on the website.

This will need to be quality checked, as it reverts commit 95bb095 from @Fruneau that was a bug fix, though I couldn't reproduce the issue.

This change will make it so that new ML include the headers:
 * List-Id
 * List-Subscribe
 * List-Unsubscribe
 * List-Archive (link currently broken, to be fixed)
 * List-Help
 * List-Post
@Fruneau
Copy link
Member

Fruneau commented Mar 1, 2017

Just in order to prove me useless, I have absolutely no idea what the bug was and why I did the patch that way (or why it would need a revert). Maybe some archeology on flyspray might help.

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.

2 participants