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

Send welcome mail to proton created users #1099

Merged
merged 3 commits into from
Jun 20, 2022
Merged

Send welcome mail to proton created users #1099

merged 3 commits into from
Jun 20, 2022

Conversation

acasajus
Copy link
Collaborator

Users created directly will receive a special welcome email

@acasajus acasajus marked this pull request as ready for review June 16, 2022 13:05
Copy link
Contributor

@nguyenkims nguyenkims left a comment

Choose a reason for hiding this comment

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

LGTM, just left a comment.

@@ -0,0 +1,36 @@
{% extends "base.html" %}

{% block content %}
Copy link
Contributor

Choose a reason for hiding this comment

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

We need to create a logo for Proton and SL here, let me ping the designer :)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

👍

Copy link
Contributor

Choose a reason for hiding this comment

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

I think we can use this logo:
sl-provided-by-proton
sl-provided-by-proton

Copy link
Contributor

@nguyenkims nguyenkims left a comment

Choose a reason for hiding this comment

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

LGTM!

@nguyenkims nguyenkims merged commit fbb59a1 into master Jun 20, 2022
@nguyenkims nguyenkims deleted the ac-pa-151 branch June 20, 2022 09:36
@acasajus
Copy link
Collaborator Author

acasajus commented Oct 11, 2022 via email

@nguyenkims
Copy link
Contributor

@acasajus I already change the logo used in every email.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants