Skip to content
This repository was archived by the owner on May 30, 2024. It is now read-only.

FEATURE: Additional options for pm_created and post_created_edited triggers #210

Merged
merged 1 commit into from
Sep 4, 2023

Conversation

martin-brennan
Copy link
Contributor

This commit introduces some additional options to make the
pm_created and post_created_edited triggers more useable as
auto-responders inside group inboxes.

  • restricted_group - Will only call the trigger if the topic is a PM
    sent to the specified group.
  • ignore_automated - If the post/topic is being created from an incoming
    email, and it is from a noreply address or otherwise automated, then it
    will not be considered valid for the autoresponder to reply to.
  • ignore_group_members - When combined with restricted_group, the
    autoresponder will not fire the trigger if the post/topic is being
    created by a member of the group.

This commit also does some minor refactors and makes some UI
and copy improvements.

image

Needs discourse/discourse#23364

…iggers

This commit introduces some additional options to make the
pm_created and post_created_edited triggers more useable as
auto-responders inside group inboxes.

* restricted_group - Will only call the trigger if the topic is a PM
sent to the specified group.
* ignore_automated - If the post/topic is being created from an incoming
email, and it is from a noreply address or otherwise automated, then it
will not be considered valid for the autoresponder to reply to.
* ignore_group_members - When combined with restricted_group, the
autoresponder will not fire the trigger if the post/topic is being
created by a member of the group.

This commit also does some minor refactors and makes some UI
and copy improvements.
@martin-brennan martin-brennan force-pushed the feature/group-inbox-auto-responder branch from 38729fb to 22d2fb4 Compare September 1, 2023 06:09
@jjaffeux
Copy link
Contributor

jjaffeux commented Sep 1, 2023

backend failures look legit

@martin-brennan
Copy link
Contributor Author

backend failures look legit

@jjaffeux will work now that the core PR has been merged, rerunning 👍

@martin-brennan martin-brennan merged commit 905ef75 into main Sep 4, 2023
@martin-brennan martin-brennan deleted the feature/group-inbox-auto-responder branch September 4, 2023 00:46
OsamaSayegh pushed a commit that referenced this pull request Mar 29, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants