-
Notifications
You must be signed in to change notification settings - Fork 16
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
Compatible with the latest version of Zotero #19
base: master
Are you sure you want to change the base?
Conversation
Can you also drop a compiled XPI here as well? Thanks. |
👍 Don't forget to rename the file to |
here's one that also includes #12 edit: except I used |
i have renamed zotero-memento.zip to zotero-memento.xpi but zotero (v. 6) is unable load the extension. Any hints? |
I downloaded this and renamed to xpi, as you did, and it installed just now: Mac OS 12.6, Zotero 6.0.15 |
Probs Windows do not allow extension name editing (unless you explicitly change the settings to allow for it). |
For some reason @Spehhhhh's version works on my computer (latest macOS and latest Zotero), while @sneakers-the-rat's version doesn't. |
This would be a great PR to get merged into a new release - would love to install without having to munge the version field. |
@tillgrallert whoa i forgot about this. do you get any error message when trying to use this? I feel like this package has been sort of abandoned (normal!) but @leonkt i'd be willing to adopt it since I use it all the time and have wanted to expand it in a few ways |
Now is the time to release here a version compatible with Zotero 7 beta. I'm not willing to do it and break something, as simple as it appears to be. |
+1 for a zotero 7 version |
ref #18