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

Allow users to set SMTP configuration on portal #919

Closed
3 of 6 tasks
mr9dollars opened this issue Jan 27, 2021 · 4 comments
Closed
3 of 6 tasks

Allow users to set SMTP configuration on portal #919

mr9dollars opened this issue Jan 27, 2021 · 4 comments
Assignees
Labels
cooldown/enhancement minor enhancement that users can see design-required issues that require mockup design ux/major > 70% users will notice the issue as a problem

Comments

@mr9dollars
Copy link

mr9dollars commented Jan 27, 2021

User can configure external SMTP provider thru the Portal, so the transactional emails will be sent with addresses of their domains.
After setting the provider and the default "from" address, user can send a test email from the Portal


Users can choose from common providers or a custom one:


Documentations

@mr9dollars mr9dollars added this to To do in Jan 2021 Milestone via automation Jan 27, 2021
@mr9dollars mr9dollars added cooldown/enhancement minor enhancement that users can see ux/major > 70% users will notice the issue as a problem labels Jan 27, 2021
@mr9dollars mr9dollars removed this from To do in Jan 2021 Milestone Feb 2, 2021
@mr9dollars mr9dollars added this to To Do in Feb 2021 Milestone via automation Feb 2, 2021
@mr9dollars mr9dollars removed this from To Do in Feb 2021 Milestone Mar 8, 2021
@mr9dollars mr9dollars added this to To do in March 2021 Milestone via automation Mar 8, 2021
@mr9dollars mr9dollars removed this from To do in March 2021 Milestone Apr 8, 2021
@mr9dollars mr9dollars added this to To Do in April 2021 Milestone via automation Apr 8, 2021
@mr9dollars mr9dollars removed this from To Do in April 2021 Milestone May 10, 2021
@fungc-io fungc-io added the design-required issues that require mockup design label Jul 15, 2021
@fungc-io fungc-io added this to To do in 2021_07_26-08_06 Sprint via automation Jul 23, 2021
@louischan-oursky louischan-oursky moved this from To do to In progress in 2021_07_26-08_06 Sprint Aug 5, 2021
@louischan-oursky louischan-oursky self-assigned this Aug 5, 2021
@fungc-io
Copy link
Member

fungc-io commented Aug 5, 2021

@louischan-oursky

Summary of our offline discussion:

  • After pressing the "Send Test Email" button, there will be an input dialog for the testing "To" address. The default value is the logged-in user's address. The dialog will dismiss after confirmation.
  • The "Send Test Email" button should be disabled while loading.
  • No need for a rate limit for testing.

Test email content
Subject: [Test] Authgear email
From: {{default from address from the config}}
Body: This email was successfully sent from {{tenent name}}

@fungc-io fungc-io removed this from In progress in 2021_07_26-08_06 Sprint Aug 9, 2021
@fungc-io fungc-io added this to To do in 2021_08_09-08_20 Sprint via automation Aug 9, 2021
@fungc-io fungc-io added this to To do in 2021_07_26-08_06 Sprint via automation Aug 9, 2021
@fungc-io fungc-io moved this from To do to In progress in 2021_07_26-08_06 Sprint Aug 9, 2021
@louischan-oursky louischan-oursky moved this from To do to In progress in 2021_08_09-08_20 Sprint Aug 9, 2021
@louischan-oursky louischan-oursky moved this from In progress to Review in progress in 2021_08_09-08_20 Sprint Aug 10, 2021
louischan-oursky added a commit that referenced this issue Aug 10, 2021
@louischan-oursky louischan-oursky moved this from Review in progress to Reviewer approved in 2021_08_09-08_20 Sprint Aug 11, 2021
@fungc-io
Copy link
Member

fungc-io commented Aug 11, 2021

regarding #1452

  • The on/off toggle should be disabled before re-auth
  • From address should use default sender
  • typo "SMPT" -> "SMTP" in the test email dialog box
  • the help text below the provider selections switched place

@louischan-oursky
Copy link
Collaborator

Point 1, 3, 4 have been fixed and deployed. Point 2 turned out to be a configuration problem.

@fungc-io
Copy link
Member

Replaced by #4217

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cooldown/enhancement minor enhancement that users can see design-required issues that require mockup design ux/major > 70% users will notice the issue as a problem
Projects
No open projects
Development

No branches or pull requests

3 participants