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

feat: New automation actions #4033

Merged
merged 21 commits into from
Mar 29, 2022
Merged

Conversation

fayazara
Copy link
Contributor

@fayazara fayazara commented Feb 21, 2022

Description

Adds new automation events

  1. Send the email notification to the team (team_ids as an input)
  2. Assign to an agent (agent_id as an input in the action)
  3. Send an email transcript to specified email (email as an input in the action)
  4. Mute conversation (no input)
  5. Change status to snooze/resolved (no input)
  6. Send a webhook event to specified URL (webhook URL as an input in the action)

Fixes #4051

Type of change

Please delete options that are not relevant.

  • New feature (non-breaking change which adds functionality)

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented on my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules

@pr-triage pr-triage bot added the PR: draft The pull request is not ready to be reviewed label Feb 21, 2022
@fayazara fayazara self-assigned this Feb 21, 2022
@fayazara fayazara marked this pull request as ready for review February 22, 2022 07:55
@pr-triage pr-triage bot added PR: unreviewed This pull request is yet to be reviewed. and removed PR: draft The pull request is not ready to be reviewed labels Feb 22, 2022
@fayazara fayazara requested review from muhsin-k, iamsivin, nithindavid and pranavrajs and removed request for muhsin-k February 22, 2022 07:55
@fayazara fayazara changed the title New automation conditions and features Feat - New automation actions Feb 22, 2022
@iamsivin
Copy link
Member

@fayazara Found a small issue, adding an action eg: Assign a team and select a team, if we change that action with another action the multi-select value is not changing or resetting, you can see in bellow screencast.

Screen.Recording.2022-02-23.at.8.20.38.AM.mov

@fayazara
Copy link
Contributor Author

@fayazara Found a small issue, adding an action eg: Assign a team and select a team, if we change that action with another action the multi-select value is not changing or resetting, you can see in bellow screencast.

@iamsivin fixed this.

iamsivin
iamsivin previously approved these changes Feb 23, 2022
Copy link
Member

@iamsivin iamsivin left a comment

Choose a reason for hiding this comment

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

LGTM 👍🏻 @nithindavid can you take a look

@pr-triage pr-triage bot added PR: partially-approved Not all reviewers have approved the PR and removed PR: unreviewed This pull request is yet to be reviewed. labels Feb 23, 2022
@pranavrajs
Copy link
Member

Folks, I guess this is blocked on the backend, please make sure the relevant backend changes are merged to master before merging this PR.

@pr-triage pr-triage bot added PR: unreviewed This pull request is yet to be reviewed. and removed PR: partially-approved Not all reviewers have approved the PR labels Feb 23, 2022
@muhsin-k muhsin-k changed the title Feat - New automation actions feat: New automation actions Feb 28, 2022
@fayazara fayazara temporarily deployed to staging-chatwoot March 26, 2022 05:06 Inactive
tejaswinichile
tejaswinichile previously approved these changes Mar 28, 2022
Copy link
Contributor

@tejaswinichile tejaswinichile left a comment

Choose a reason for hiding this comment

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

LGTM 💯

@pr-triage pr-triage bot added PR: partially-approved Not all reviewers have approved the PR and removed PR: unreviewed This pull request is yet to be reviewed. labels Mar 28, 2022
@pr-triage pr-triage bot added PR: unreviewed This pull request is yet to be reviewed. and removed PR: partially-approved Not all reviewers have approved the PR labels Mar 29, 2022
Copy link
Contributor

@tejaswinichile tejaswinichile left a comment

Choose a reason for hiding this comment

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

LGTM

@pr-triage pr-triage bot added PR: partially-approved Not all reviewers have approved the PR and removed PR: unreviewed This pull request is yet to be reviewed. labels Mar 29, 2022
@fayazara fayazara merged commit c674393 into develop Mar 29, 2022
@fayazara fayazara deleted the feat/3951-automation-enhancements branch March 29, 2022 07:57
@pr-triage pr-triage bot added PR: merged The pull request is merged to another branch and removed PR: partially-approved Not all reviewers have approved the PR labels Mar 29, 2022
@github-actions
Copy link

github-actions bot commented Aug 8, 2022

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 8, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
on-hold Issue is currently on hold PR: merged The pull request is merged to another branch
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Automation Enhancement Actions
6 participants