= Summary =
As seen in bug #401772, Gmail doesn't display diffs inline unless the
content-type is text/plain or the file extension is .txt.
== Proposed fix ==
Change the file extensions of generated diffs to be .txt. This will
appease Gmail, but clients which respect the text/x-diff content-type
will be unaffected.
== Pre-implementation notes ==
Pre-imp with Thumper
== Demo and Q/A ==
Generate merge proposal emails and branch revision emails containing
diffs. Jam them into Gmail somehow. Possibly download the raw bytes
from staging, and use a Python script to resend to Gmail.
View them on Gmail. Observe whether they're displayed inline.
= Launchpad lint =
Checking for conflicts. and issues in doctests and templates.
Running jslint, xmllint, pyflakes, and pylint.
Using normal rules.
Linting changed files:
lib/lp/code/mail/tests/test_branch.py
lib/lp/code/mail/branchmergeproposal.py
lib/lp/code/mail/tests/test_branchmergeproposal.py
lib/lp/code/mail/branch.py
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
= Summary =
As seen in bug #401772, Gmail doesn't display diffs inline unless the
content-type is text/plain or the file extension is .txt.
== Proposed fix ==
Change the file extensions of generated diffs to be .txt. This will
appease Gmail, but clients which respect the text/x-diff content-type
will be unaffected.
== Pre-implementation notes ==
Pre-imp with Thumper
== Implementation details ==
None
== Tests == ail_with_ diff -t ail_attaches_ diff
bin/test mail.tests -t test_generateEm
test_generateEm
== Demo and Q/A ==
Generate merge proposal emails and branch revision emails containing
diffs. Jam them into Gmail somehow. Possibly download the raw bytes
from staging, and use a Python script to resend to Gmail.
View them on Gmail. Observe whether they're displayed inline.
= Launchpad lint =
Checking for conflicts. and issues in doctests and templates.
Running jslint, xmllint, pyflakes, and pylint.
Using normal rules.
Linting changed files: code/mail/ tests/test_ branch. py code/mail/ branchmergeprop osal.py code/mail/ tests/test_ branchmergeprop osal.py code/mail/ branch. py enigmail. mozdev. org
lib/lp/
lib/lp/
lib/lp/
lib/lp/
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://
iEYEARECAAYFAkp nX2MACgkQ0F+ nu1YWqI2o1wCghd KG6QUdLZYG2yBXf P8Z35+R D5FrgHX2/ 9rsTNJrF
A98AniLXLW0vp2s
=HTT3
-----END PGP SIGNATURE-----