Conversations addon's pill for DKIM verfier is missing about dkim_verifier HOT 2 CLOSED

The problem appears to be this bit of the code: https://github.com/search?q=repo%3Alieser%2Fdkim_verifier+tab.url%3F.startsWith%28%22chrome%3A%2F%2Fconversations%2F%22%29&type=code

I suspect this due to how the Thunderbird rework has changed all the tab processing - the tab no longer has a url field option, and so chrome://conversations can't be matched.

I'm not sure of an easy option of what to do there, except for maybe detecting if Conversations is installed/active and running it regardless.

from dkim_verifier.

Thanks for reporting the issue, this should be fixed now. If you want to try it out already you can use the following version:
[email protected]

@Standard8 You there correct about the problematic code.
I extended your proposed check a little to work with the case that a message is viewed in a tab or windows (in the classic view).

I noticed that if Conversations is installed browser.messageDisplay.onMessageDisplayed is not triggered even then viewing single messages in Thunderbird's 3pane tab. I guess this is something Conversations prevents?
This is nothing that interferes with my add-on, it actually means the check I do fo conversation there is not needed currently. I still decided to leave it in in case it changes in the future.

from dkim_verifier.