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: email styling #547

Closed
wants to merge 9 commits into from
Closed

Conversation

LeandroCarvalho200409
Copy link

Description

Provide a detailed description of the purpose of the changes included in this pull request. Optionally, include background information, relevant screenshots, and any other context that helps explain the work.

The purpose of this PR is to change the style of all the E-mail templates, so that they look legit and have a user-friendly design.

Related Issues

If applicable, specify the main parts of the application that will be impacted by this pull request.

#544

Steps to reproduce or test

Describe the steps that you did to reproduce this.

@LeandroCarvalho200409
Copy link
Author

Here's a first prototype of the E-mail styling for the E-mail confirmation mail.

image

Copy link
Member

@joaodiaslobo joaodiaslobo left a comment

Choose a reason for hiding this comment

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

I like how it is looking! 🤩
Keep up the good work! 💪

@joaodiaslobo joaodiaslobo changed the title Styling of all E-mails #544 feat: email styling Dec 6, 2024
@RicoPleasure RicoPleasure self-assigned this Feb 21, 2025
@RicoPleasure
Copy link
Contributor

Activity certificate:
image
This one cannot be tested on atomic because we don't have how to trigger this event yet, so I have just opened in another file.

Confirm account:
image

Reset password:
image

Collaborator accepted:
image

Collaborator request:
image

@RicoPleasure RicoPleasure marked this pull request as ready for review February 27, 2025 19:21
Copy link
Member

@joaodiaslobo joaodiaslobo left a comment

Choose a reason for hiding this comment

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

Did you merge this accidentally with main? There are a lot of files (mainly images) being added from the old project

Copy link
Member

@joaodiaslobo joaodiaslobo left a comment

Choose a reason for hiding this comment

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

Style-wise this is looking really good!! 🤩

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

Successfully merging this pull request may close these issues.

3 participants