Skip to content

Send email once integration setup is done#341

Merged
mariobalca merged 5 commits intomainfrom
feature/send-email-integration-setup
Dec 15, 2022
Merged

Send email once integration setup is done#341
mariobalca merged 5 commits intomainfrom
feature/send-email-integration-setup

Conversation

@mariobalca
Copy link
Copy Markdown
Contributor

@mariobalca mariobalca commented Dec 10, 2022

Changes proposed ✍️

  • Add sendEmailAt column/attribute to integrations
  • Tweak integrationProcessor.ts to send an email to all tenant users once onboarding is done
  • Add alert bar to the frontend to inform users that an email will be sent once setup is completed

Screenshots (front-end changes only)

Screenshot 2022-12-10 at 02 24 24

Screenshot 2022-12-10 at 02 24 40

Checklist ✅

  • Label appropriately with Feature, Enhancement, or Bug.
  • Tests are passing.
  • New backend functionality has been unit-tested.
  • Environment variables have been updated:
    • Local frontend configuration: frontend/.env.dist.local, frontend/.env.dist.composed.
    • Local backend: backend/.env.dist.local, backend/.env.dist.composed.
    • Configuration docs have been updated.
    • Team members only: update environment variables in override, staging and production env. files and trigger update config script.
  • API documentation has been updated (if necessary) (see docs on API documentation).
  • Quality standards are met.
  • All changes have been tested in a staging site.
  • All changes are working locally running crowd.dev's Docker local environment.

@mariobalca mariobalca added the Feature Created by Linear-GitHub Sync label Dec 10, 2022
@mariobalca mariobalca self-assigned this Dec 10, 2022
@mariobalca mariobalca merged commit a8b669b into main Dec 15, 2022
@mariobalca mariobalca deleted the feature/send-email-integration-setup branch December 15, 2022 15:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature Created by Linear-GitHub Sync

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants