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

Version 2.1.1 does not work on Thunderbird 78.14.0 #105

Open
guilhermepiccoli opened this issue Jan 6, 2022 · 5 comments
Open

Version 2.1.1 does not work on Thunderbird 78.14.0 #105

guilhermepiccoli opened this issue Jan 6, 2022 · 5 comments
Labels

Comments

@guilhermepiccoli
Copy link

Hey all, first I'd like to thank you for the very interesting add-on - this is pretty useful!

So, when checking plain/text patches (sent with git send-email), I cannot see the colored diffs. I'm using Thunderbird 78.14 (from Ubuntu 20.04), but I don't have any other kind of patches to compare, I only use git send-email and check kernel lists, so no Github patches for example.

Is the add-on supposed to work with plain/text messages? Thanks!

@Qeole
Copy link
Owner

Qeole commented Jan 6, 2022

Hi, and thank you for the report!
Yes, the add-on is supposed to work on plain text emails sent with git, it's the primarily use case (for me, at least).

I have not tested on Thunderbird 78.14. The add-on should work as expected on 78.4, on some later versions, and then on 91+, but I have not tested all versions in between.

What version of the add-on do you run? It should be 2.1.1? If this is the case, do you have a chance to try 2.2.0, to see if it works (not sure if the add-on repository lets you do it, you can install manually from this repo otherwise)? I remember we had to update something between Thunderbird 78.* and 91, maybe the change in Thunderbird that caused this was backported to the 78 branch?

Otherwise I'll give a try myself on 78.14 when I have a moment.

@guilhermepiccoli
Copy link
Author

Hi @Qeole , thanks a lot for your prompt response!

So, I was using 2.1.1 as you said...I removed that, and tried to install 2.2.0, the XPI from repo. But..TB didn't allow me, claiming it's not compatible. And that even after I turn off the "extensions.strictCompatibility" config in the advanced editor.

Last resort was to hack the manifest json inside the xpi, and set the min version to 78 (it was set to 91). With that, I could install the add-on, but it still doesn't work. So, I think it's not compatible anymore with versions < 91, right?

Thanks again!!

@guilhermepiccoli
Copy link
Author

And guess what? I've just updated TB, running 91 now..and it's working like a charm! I'm using the version from the add-on repository, it seems to be 2.2.0 ... congrats, the extension is very nice and useful =)

Cheers!

@Qeole
Copy link
Owner

Qeole commented Jan 7, 2022

Great news, thanks for testing and letting me know!!

I also appreciate a lot the feedback. I tried to change the compatibility of version 2.1.1 to reflect that it does not work on Thunderbird, v78.14, but the list doesn't have all 78.* versions, and I prefer to have a few users hitting this issue rather than to take the risk to disable the add-on for Thunderbird versions where it should work.

Given that the latest version of the add-on works on the current stable version of Thunderbird, I won't spend time investigating the issue. But I'll leave this issue open for reference.

@Qeole Qeole changed the title Should plain-text emails work? They aren't on Thunderbird 78.14.0 Version 2.1.1 does not work on Thunderbird 78.14.0 Jan 7, 2022
@Qeole Qeole added the bug label Jan 7, 2022
@guilhermepiccoli
Copy link
Author

guilhermepiccoli commented Jan 8, 2022 via email

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

No branches or pull requests

2 participants