-
Notifications
You must be signed in to change notification settings - Fork 8
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
Support for strict mode (TB 60 compat) #7
Conversation
Apparently thunderbird 60 executes the extension code using strict mode. While most of the code worked fine, there were a few problems that lead to errors under strict mode. Now everything works under strict mode.
* Support for thunderbird 60
Thank you for your work! Sadly, it seems that this add-on is no longer maintained by the developer. There hasn't been any activity since September 2015. |
@krukai: There weren't any breaking changes in thunderbird or lightning that affected this add-on since 2015, so I think it is pretty reasonable to not push any updates. Let's just give Ingo some time. |
Up to which version of Thunderbird does your custom add-on work, Klemens? |
@Freudentaumel: The maxVersion is set to 60.* and I have tested the changes with 60.0. I have not tested the minVersion. Do you have problems with the patched version? |
@klemens What does your reference to the minVersion mean? |
Well, these versions don't exist yet, so it is hard to tell it they will be compatible. 😉 But the stable thunderbird 60 will probably be supported for around a year from now (see firefox esr schedule), so you probably don't need to worry.
The manifest currently claims that the add-on works with thunderbird 3 and above, but I only tested with the most recent (and only supported) version. |
Thanks for looking into this and the PR, let's hope this gets merged. Cheers! |
Works perfectly here with Thunderbird 60.2.1. |
@klemens ,
I'd say: "Some time" is up ;) Would you be willing to release your fork as a Thunderbird add-on? |
I wrote him an email to ask if he has seen this pull request and if he would like help maintaining the add-on. Let's wait until this weekend and if he has not responded until then, I'll see if I can release a fork. |
Thank you so much @klemens for the MR and everybody else for looking into it. I'll do my best to release a new version with it. Also, sorry for not being more responsive. My life has moved on and I am not using Thunderbird myself anymore for years. This makes it hard to motivate myself to do anything. But I see that at least this maintenance is absolutely required. Hopefully, with this fix, it will continue to work for a while more ;) |
OK, the new version is waiting for review: https://addons.thunderbird.net/en-US/developers/addon/thunderbirthday/versions/2468767. |
that's the developer link. the url for users is |
Apparently thunderbird 60 executes the extension code using strict mode. While most of the code worked fine, there were a few problems that lead to errors under strict mode. Now everything works under strict mode.
I also added
"use strict";
to all js files and a commit to bump the version.