-
-
Notifications
You must be signed in to change notification settings - Fork 22
Try to get it working with Chrome again #620
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
base: main
Are you sure you want to change the base?
Conversation
for self-extracting pages
git-subtree-dir: zotero-connectors git-subtree-split: eff2dce6a1e772d6d8b2d6b3c59d61e540369695
…c04b git-subtree-dir: zotero-connectors/src/translate git-subtree-split: 1f6a1c04b8d06bd2aaaf0c25b981a1ed4e374ace
…nnectors/src/translate'
…236a git-subtree-dir: zotero-connectors/src/utilities git-subtree-split: 43142236a282e5e1a3190694628f329aa2e0ba8e
…nnectors/src/utilities'
git-subtree-dir: zotero-scholar-citations git-subtree-split: d2a392a2ce6afab215e0264e89f6e75344c2ca49
@tobiasdiez @LyzardKing I updated the zotero submodule stuff etc and the gulp file etc so that it at least compiles and can be loaded locally in the browser. I commented out the scripts that are no long there; however, I have no idea what kind of scripts we now need to include and how to continue here. |
The issue with Chrome is that we cannot have multiple background scripts. Then we need to make sure that the same works for firefox (that does not support service workers) |
Thanks for looking into this. For the missing scripts, I would start by tracking where the went in zotero and then add the new location. |
Update to latest zotero stuf
Fixes #616
TODO: test if this actually still works