-
Notifications
You must be signed in to change notification settings - Fork 20
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
MINOR - Donations email issues #460
Comments
First is a known issue/feature - but I think this should be changed. Second is new to me, but agree that this looks very messy and should be fixed. |
We need to check the template here: https://github.com/akvo/akvo-rsr/blob/develop/akvo/templates/rsr/project/donate/donation_confirmation_email.html and fix the styling issues. |
@adriancollier apologies - I should have included that in the original issue (I was using gmail when I took the screenshot). |
@loicsans did you find anything that might explain this? The Template is here by the way: https://github.com/akvo/akvo-rsr/blob/develop/akvo/templates/rsr/project/donate/donation_confirmation_email.html |
It's actually a known issue with Gmail. Some tag level inlined CSS are ignored. A safest way would be to use table to layout the email properly instead of floating divs. |
@loicsans is this much work? Would you be able to do this? 😉 |
Shoot it my way ;) |
Well this issue is now assigned to you :) just needs a feature/460-donationemail branch to make the changes in |
@zzgvh How could I test this out on my local machine? |
[#460] Removed floating Div / Replaced by a table
in the table and left aligned the title
@zzgvh If all code is merged to develop can you please close this issue? |
I see all code now in develop, so closing this issue. |
I noticed a few minor issues with thank you emails whilst testing:
The text was updated successfully, but these errors were encountered: