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

Email verifycation template is not styled #435

Closed
humitos opened this issue Jul 25, 2024 · 3 comments · Fixed by #479
Closed

Email verifycation template is not styled #435

humitos opened this issue Jul 25, 2024 · 3 comments · Fixed by #479
Assignees
Labels
Accepted Accepted issue on our roadmap Bug A bug

Comments

@humitos
Copy link
Member

humitos commented Jul 25, 2024

It seems we are using a default allauth template. This one in particular: https://github.com/pennersr/django-allauth/blob/301324e3ed2bf7c95987f24625ee8ebd7480ce85/allauth/templates/account/verification_sent.html

I think we want to override them to:

Screenshot_2024-07-25_17-50-58

@humitos
Copy link
Member Author

humitos commented Jul 25, 2024

Our looks like:

Screenshot_2024-07-25_17-52-13

@agjohnson agjohnson added Bug A bug Accepted Accepted issue on our roadmap labels Aug 23, 2024
@agjohnson agjohnson added this to the Dashboard minor bugs milestone Aug 23, 2024
@agjohnson
Copy link
Contributor

You know, I have noticed a lot of regressions on allauth templates that look like this too. I wonder if we lost something in an upgrade. I know this is just a missing template that needs added, but I've seen this fairly broken styling on a number of allauth views lately.

I opened up #462 to investigate. Maybe I'm wrong though.

@agjohnson agjohnson self-assigned this Sep 9, 2024
@agjohnson
Copy link
Contributor

Just a note, on the examples above, the first template is missing, the second template is the one that users see after clicking the email verification email they are sent, so is a separate template.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Accepted Accepted issue on our roadmap Bug A bug
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants