Skip to content

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

Open
wants to merge 949 commits into
base: main
Choose a base branch
from
Open

Try to get it working with Chrome again #620

wants to merge 949 commits into from

Conversation

Siedlerchr
Copy link
Member

@Siedlerchr Siedlerchr commented Feb 20, 2025

Update to latest zotero stuf

Fixes #616

TODO: test if this actually still works

adomasven and others added 22 commits August 30, 2024 12:47
git-subtree-dir: zotero-connectors
git-subtree-split: eff2dce6a1e772d6d8b2d6b3c59d61e540369695
…c04b

git-subtree-dir: zotero-connectors/src/translate
git-subtree-split: 1f6a1c04b8d06bd2aaaf0c25b981a1ed4e374ace
…236a

git-subtree-dir: zotero-connectors/src/utilities
git-subtree-split: 43142236a282e5e1a3190694628f329aa2e0ba8e
…it '0f54696716337349ab749eb0b2aca54989985f06'

git-subtree-dir: zotero-connectors/src/zotero-google-docs-integration
git-subtree-mainline: 5eb3ab4
git-subtree-split: 0f54696
git-subtree-dir: zotero-scholar-citations
git-subtree-split: d2a392a2ce6afab215e0264e89f6e75344c2ca49
@Siedlerchr
Copy link
Member Author

Siedlerchr commented Feb 20, 2025

@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.

@LyzardKing
Copy link
Collaborator

The issue with Chrome is that we cannot have multiple background scripts.
We can copy what zotero is doing, and have one "service worker" script that loads all the others: https://github.com/zotero/zotero-connectors/blob/master/src/browserExt/background-worker.js

Then we need to make sure that the same works for firefox (that does not support service workers)

@tobiasdiez
Copy link
Member

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.

…fc58cb5209b9bd2db93f5f1020c11'

git-subtree-dir: zotero-connectors/lib/SingleFile-Lite
git-subtree-mainline: 191256f
git-subtree-split: 5e65312
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

Successfully merging this pull request may close these issues.

No longer supported by chrome
7 participants