-
Notifications
You must be signed in to change notification settings - Fork 756
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
Update selenium test to support system-app
#3210
Conversation
3f440a5
to
1b626a9
Compare
Can you explain what this is for? |
It's to fix the CI failure: #3209 (comment) |
Thanks so much for the pointer to that |
It seems that didn't work: https://github.com/zotero/translators/actions/runs/7469963343/job/20327889161 Are you sure the extension ID is guaranteed to be constant? |
Hmm, actually that failure happens earlier than getting the URL, hmm. |
In https://github.com/zotero/translators/actions/runs/7469963343/job/20327889161 it seems that |
Per https://github.com/zotero/translators/actions/runs/7540520982/job/20525278512 no, they all look proper files with metadata. (I added |
Submitted #3231. |
Alright, #3213 is green now, I'll close this one. Thanks! |
For now I did this, but can we use something like this: https://developer.chrome.com/docs/extensions/reference/manifest/key#keep-consistent-id ?