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

added sieve example for subaddress sorting #2410

Merged
merged 4 commits into from
Feb 17, 2022
Merged

added sieve example for subaddress sorting #2410

merged 4 commits into from
Feb 17, 2022

Conversation

mpldr
Copy link
Contributor

@mpldr mpldr commented Feb 15, 2022

Description

This just adds an example on how to automatically sort incoming mails, based on their subaddresses. I use this filter myself and thought it might be of interest to people setting up their sieves.

Type of change

  • Documentation update

Checklist:

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation (README.md or the documentation under docs/)
  • New and existing unit tests pass locally with my changes

@georglauterbach georglauterbach added area/documentation kind/improvement Improve an existing feature, configuration file or the documentation labels Feb 15, 2022
@georglauterbach georglauterbach added this to the v10.5.0 milestone Feb 15, 2022
casperklein
casperklein previously approved these changes Feb 15, 2022
Copy link
Member

@casperklein casperklein left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good contribution 👍

@casperklein casperklein requested a review from a team February 15, 2022 19:29
wernerfred
wernerfred previously approved these changes Feb 15, 2022
Copy link
Member

@wernerfred wernerfred left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for your first contribution 🚀

One question: Will this also work if you use another delimiter or would you need to adjust something?

@mpldr
Copy link
Contributor Author

mpldr commented Feb 15, 2022 via email

@mpldr
Copy link
Contributor Author

mpldr commented Feb 15, 2022 via email

@wernerfred
Copy link
Member

I do not mean the delimiter option on the dms in general but in your sieve code snippert. Can't see where it matches to + (which could be changed). Or will this sieve snippet also work if someone configured dms to use another delimiter out of the box

@mpldr
Copy link
Contributor Author

mpldr commented Feb 15, 2022 via email

@mpldr mpldr dismissed stale reviews from wernerfred and casperklein via 0b283dc February 17, 2022 09:45
@github-actions
Copy link
Contributor

Documentation preview for this PR is ready! 🎉

Built with commit: b2d0379

@georglauterbach georglauterbach merged commit 54f2181 into docker-mailserver:master Feb 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/documentation kind/improvement Improve an existing feature, configuration file or the documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants