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

[EHN]Email filtering: Cypht Sieve filters: Better handling of 'Stop processing more rules' #1098

Conversation

christer77
Copy link
Member

Sometimes, you might set up several rules that could apply to a single message. If you created a rule to move all messages with a subject line of "Wedding plans" to a folder, for example, and then created another rule to delete all messages with attachments, you probably wouldn't want a message related to wedding plans that has an attachment to be deleted. To prevent this from happening, you can set the option Stop processing more rules on the first rule.

stop-filtering

@marclaporte
Copy link
Member

@josaphatim
Copy link
Member

@christer77 I think you should also remove stop filtering in list of available actions as it is replaced by the checkbox.

@christer77 christer77 force-pushed the Email-filtering-Cypht-Sieve-filters-Better-handling-of-Stop-processing-more-rules branch from 4570075 to 579e0a7 Compare June 22, 2024 17:01
@christer77
Copy link
Member Author

@christer77 I think you should also remove stop filtering in list of available actions as it is replaced by the checkbox.

Done

@marclaporte marclaporte requested a review from josaphatim June 29, 2024 19:53
@christer77 christer77 force-pushed the Email-filtering-Cypht-Sieve-filters-Better-handling-of-Stop-processing-more-rules branch from 579e0a7 to 780beb3 Compare July 6, 2024 18:49
@marclaporte marclaporte requested a review from josaphatim August 8, 2024 21:43
@josaphatim
Copy link
Member

@christer77 one last thing: can you reset priority, test and the new checkbox you added when clicking add filter ? See $('.add_filter').on('click', function ()

@christer77 christer77 force-pushed the Email-filtering-Cypht-Sieve-filters-Better-handling-of-Stop-processing-more-rules branch 2 times, most recently from 439f54a to b35661a Compare August 21, 2024 08:10
@christer77 christer77 force-pushed the Email-filtering-Cypht-Sieve-filters-Better-handling-of-Stop-processing-more-rules branch from b35661a to f70a5bb Compare August 22, 2024 08:13
@josaphatim
Copy link
Member

Now good! Thanks

@josaphatim josaphatim merged commit 0992ff0 into cypht-org:master Aug 22, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants