Merge lp:~daker/launchpad/fix.1600499 into lp:launchpad
Proposed by
Adnane Belmadiaf
Status: | Needs review |
---|---|
Proposed branch: | lp:~daker/launchpad/fix.1600499 |
Merge into: | lp:launchpad |
Diff against target: |
119 lines (+72/-0) 6 files modified
lib/lp/bugs/emailtemplates/bug-notification-verbose.txt (+12/-0) lib/lp/bugs/emailtemplates/bug-notification.txt (+12/-0) lib/lp/code/emailtemplates/branch-merge-proposal-created.txt (+12/-0) lib/lp/code/emailtemplates/branch-merge-proposal-updated.txt (+12/-0) lib/lp/code/emailtemplates/branch-modified.txt (+12/-0) lib/lp/code/emailtemplates/review-requested.txt (+12/-0) |
To merge this branch: | bzr merge lp:~daker/launchpad/fix.1600499 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Colin Watson (community) | Needs Fixing | ||
Review via email: mp+299616@code.launchpad.net |
Description of the change
Add support for Gmail go-to actions
To post a comment you must log in.
Unmerged revisions
- 18128. By Adnane Belmadiaf
-
Restore more trailing space
- 18127. By Adnane Belmadiaf
-
Restore trailing space
- 18126. By Adnane Belmadiaf
-
Add more gmail actions
- 18125. By Adnane Belmadiaf
-
Add Gmail actions markup
Thanks for the patch.
These were previously plain-text emails sent to non-Gmail users as well. Dumping HTML-like markup into the start of them without even a Content-Type change is not going to look good at all to anyone who doesn't use Gmail; and I would be wary of making them anything other than text/plain anyway.
Can you point to the specification for this actions markup? We need to find some approach that's a bit less horribly intrusive, if indeed it's worth doing. As a last resort it may require a new PersonSettings column, but it will depend on what options the spec affords us. For example, if it were possible to do this by way of additional message headers, that would be *much* better than dumping JSON into the message body.
Whatever approach ends up being workable, this needs to include tests.