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

Patch for TB 68 #92

Open
rzroth opened this issue May 29, 2020 · 20 comments
Open

Patch for TB 68 #92

rzroth opened this issue May 29, 2020 · 20 comments

Comments

@rzroth
Copy link

rzroth commented May 29, 2020

I made a hacked version (notes below) you can find at:
http://www.tnrglobal.com/2020/05/thunderbird-addons/
Hack issues:
1) It does require TB restart although it doesn't report that.
2) I've mainly used search and Ctl-Shift FIELD for searching,
SO please try your favorite options and let me know if it works.
3) I don't have access to add better update info
4) FYI, This is a legacy mode extension

I call it hacked because I not a extension pro, even if I am a Kernel level professional programmer.
good luck. It is not in any way compromised as many use hacked to mean.
If I get github access I will make a branch for this.

@Harliff
Copy link

Harliff commented Jun 2, 2020

I'm just installed it and tested.
@rzroth, thanks for your work!

@karlbu
Copy link

karlbu commented Jun 4, 2020

Hello rzroth. Thank you for continuing to develop expression search for current versions of T-Bird. Greatly appreciated.

I installed 1.4.2-tb.xpi into version 68.9. While T-Bird launched, I could not access any emails. Nor could I access the Add-ons module to uninstall it. I uploaded the console error report as a word file to this post. (Did not know how to export the file.)

console error 20200604.docx

@gabbello
Copy link

gabbello commented Jun 5, 2020

Man this is awesome

@markwmuller
Copy link

While this sounds promising, I would strongly caution against installing something from a website that doesn't even have HTTPS set up, where the dev claims they don't have github access (from a github account -- what access is lacking? Anyone can fork their repo and develop there)

@aleqx
Copy link

aleqx commented Jun 5, 2020

While this sounds promising, I would strongly caution against installing something from a website that doesn't even have HTTPS set up, where the dev claims they don't have github access (from a github account -- what access is lacking? Anyone can fork their repo and develop there)

@markwmuller I second that. That being said, though, the .xpi is just a zip which you can unpack and verify the Javascript code for any malicious/suspicious actions (I haven't done that myself).

@Roy-Orbison
Copy link

Roy-Orbison commented Jun 6, 2020

@markwmuller

I would strongly caution against installing something from a website that doesn't even have HTTPS set up

It's unlikely anyone has a MITM attack server on people's routes but, yes, HTTPS would be better.

the dev claims they don't have github access (from a github account -- what access is lacking?

They're talking about write access on the existing repository.

It's still wise to exercise caution, but remember that .xpi files are just zip archives, and all source on an extension can be inspected.

@rzroth We appreciate your work, but could you please fork the repo on Github, commit the changes to your fork, then open a pull request? Then everyone can see what's changed and why.

@daniel-lo
Copy link

daniel-lo commented Jun 6, 2020

Super! Installs well on Win10Pro TB68.8.1 .. but layout needs optimisation. It seems to consist of wo fields .. the left one works .. the right on (which shouldn't be there) doesn't.

expression

Edit: No change after updating to TB68.9.0

@riedel riedel mentioned this issue Jun 7, 2020
@riedel
Copy link

riedel commented Jun 7, 2020

Instead of complaining I created a PR (#93) with @rzroth's commit and set him as author, please feel free to review it.

@riedel
Copy link

riedel commented Jun 7, 2020

@rzroth: I will close my pull request if you create one. If you are unsure how there is a short documentation , e.g. here: https://opensource.com/article/19/7/create-pull-request-github

Thanks alot for your efforts. Hope you will maintain a fork of this great add-on!

@fpierrat
Copy link

fpierrat commented Jun 8, 2020

@rzroth: thanks for the patch
@riedel: what do I have to install it? Just download the fork as a zip and rename this .zip to .xpi ?

@markwmuller
Copy link

Works for me, thank you @rzroth @riedel !
How to install, @fpierrat , follow these instructions https://stackoverflow.com/questions/19240653/how-to-create-xpi-file-with-7zip (I just zipped from linux file browser, renamed to .xpi, and could open it. Note the folder structure seems to matter)

@fpierrat
Copy link

fpierrat commented Jun 9, 2020

Installed. Happy :-D
Just this second input field mentioned by daniel-lo above. Looks a bit strange, but it doesn't seem to affect the searches... when you type them in the left input ;-)

@gsappa
Copy link

gsappa commented Jul 2, 2020

I've tried to install it (both @rzroth xpi than following instructions reporte by @markwmuller), but it's saying that is non compatible with TB 68.10. Any suggestion?

@rzroth
Copy link
Author

rzroth commented Jul 2, 2020 via email

@nkntk
Copy link

nkntk commented Jul 8, 2020

I changed the manifest.json like this, and it works fine.
It is ok without "strict_max_version". There were other addons that didn't have "strict_max_version" either.

The manifest.json
"applications": { "gecko": { "id": "{03EF8A6E-C972-488f-92FA-98ABC2C9F8B9}", "strict_min_version": "68.0" } },

@dimitrior
Copy link

Hi all.

Thanks @riedel and @rzroth .

Disclaimer: Not a coder. Never used github to pull, fork or anything else code-related. Help gratefully received!

I tried the following, but so far nothing worked for me:

  1. Modifying the manifest.json in @rzroth 's original hack, re-zipping, re-naming to xpi, install from file in thunderbird. I tried both the editing of the max_version to 68.*, and the removal of this line as per @TNknKde1201 's comment.
  2. Downloading the files from here, unzipping (just to be sure), then re-zipping (with no top-level extension name folder), rename xpi, etc.

I keep getting the message that the add-on cannot be installed as it is incompatible with t-bird 68.10.0 (64).

Does anyone have any suggestions on what might be wrong, or what else I could try?

@ecorreia
Copy link

ecorreia commented Aug 3, 2020

I really need this extension for my daily work... It stopped working again on TB 68.11.0...
Please help...
I cannot understand why this is not part of core TB...

@dimitrior
Copy link

Does anyone have any suggestions on what might be wrong, or what else I could try?
Note: Fixed thanks to this: #94 (comment)

@dimitrior
Copy link

I really need this extension for my daily work... It stopped working again on TB 68.11.0...
Please help...
I cannot understand why this is not part of core TB...

Check here: #95

@malix0
Copy link

malix0 commented Aug 8, 2020

The best solution will be a native TB support, please add your vote here
https://bugzilla.mozilla.org/show_bug.cgi?id=538821

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

No branches or pull requests