ldapinfo icon indicating copy to clipboard operation
ldapinfo copied to clipboard

incompatible with Thunderbird 68.0

Open linkp opened this issue 6 years ago • 12 comments

Following an update to Thunderbird 68.0 Awesome ldapInfoShow is now disabled and marked as incompatible.

MozillaThunderbird Version 68.0-lp151.6.1 x86_64 from obs://build.opensuse.org/mozilla Awesome ldapInfoShow Version 1.2

linkp avatar Aug 30 '19 15:08 linkp

The same here, but using Gentoo Linux. Thunderbird was upgraded to 68.1.0 , and Awesome ldapInfoShow was disabled as it does not support the new MailExtensions APIs

jmfernandez avatar Sep 23 '19 12:09 jmfernandez

After having a look at the latest commits, I found one referring to TB 68, which led me to think @wangvisual had the extension almost ready for Thunderbird 68.0.

I have adapted the manifest.json and release.sh from the sibling project https://github.com/wangvisual/autoarchive and generated an xpi file, in order to know whether it would work. The configuration dialog is currently broken (black background and no correct values in the fields). The LDAP images fetch logic seems to work, as well as the underlying logic which shows the contact photos on composition, address book and e-mail reading.

As it is a trivial change and I have developed nothing, instead of forking the project I have just included the tar.gz file ldapinfo-81b83d50de4f32a141ba9c6a02a242a0f51b81c7.tar.gz , which contains a pre-generated ldapinfo-1.3beta-tb.xpi file in src subdir (in case someone is as desperate as me to keep this awesome extension in her/his updated thunderbird installation).

jmfernandez avatar Oct 25 '19 17:10 jmfernandez

Has anyone any update on an official release ?

CharlesDelorme avatar Jan 27 '20 10:01 CharlesDelorme

Thank you so much @jmfernandez ! I was badly missing it. It would indeed be great if someone involved in this project found the time to prepare a fully functional and official 1.3 release....

defacto64 avatar Feb 04 '20 09:02 defacto64

@jmfernandez Thank you for providing the beta for TB 68. Unfortunally I cannot install it in TB 68.10.0 - did something change since the initial release of TB 68? Anyway, do you have a version which also works with TB 68.10.0? Or is there any light at the end of the tunnel for an official release? ;-)

msebald avatar Jul 21 '20 15:07 msebald

I took the version of @jmfernandez and updated the manifest.json to work with TB 68.* which works for my needs: ldapinfo-1.3beta-tb.xpi.tar.gz

{
  "applications": {
    "gecko": {
      ...
      "strict_max_version": "68.99"
    }
  },
...

gruniversal avatar Dec 01 '20 21:12 gruniversal

Good news, I have ported this extension to >=TB78 using Expreriments API:

  • showing photos from Local Directory works only in messages headers (not in Composition);
  • functionality of showing photos from other places was not tested (not available for me);
  • Addon's Options does not work at all as should be totally re-designed to new API (but you can still modify setting from TB Config editor, searching values by "ldapInfo" and toggling/modifying them).

(don't forget to rename file extension from "zip" to "xpi" before adding) LdapInfo78_experimental2.zip

Good luck!

AlexUkr73 avatar Dec 11 '20 17:12 AlexUkr73

Great, but wouldn't it make more sense to create a pull request?

rgpublic avatar Dec 11 '20 17:12 rgpublic

rgpublic - I don't know nothing about this (pull request). I don't have enough skills in programming, and was forced to make my favorite addon working and have read a tons of manuals how to do it. I found this page while was searching for solutions with TB68 some time ago and @jmfernandez helped me a lot. So, I played around with new TB78 trying to make it working and just shared this here.

AlexUkr73 avatar Dec 11 '20 17:12 AlexUkr73

@AlexUkr73 : Ah, sure, understood. Thanks for sharing it anyway - certainly useful.

rgpublic avatar Dec 11 '20 18:12 rgpublic

Good luck!

Cool, it works for me. At least when having a brief look for 1 minute just now. Keep you posted. Thank you very much!

Is it possible to merge this in here? :-)

msebald avatar Dec 11 '20 20:12 msebald

@msebald : Great. Let's continue to discuss TB78 in issue# 43 thread to avoid confusing people who search for solutions for TB68 here. As far as I understood - they are already solved by @gruniversal

AlexUkr73 avatar Dec 11 '20 20:12 AlexUkr73