-
Notifications
You must be signed in to change notification settings - Fork 28
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
feat: Firefox Support #33
Conversation
…est data from there in Firefox
When testing this, I unfortunately got a warning during the upload process: Not super sure what this means - should not really influence a submission though. The
Fortunately it is pretty simple to find the As they need a canonical link, this would be: And for For Providing these links in the "Notes to Reviewers" should be enough for the review to pass from my experience:
|
For the "Use OAuth" button to work, the URL And that's it, with this information this should be fully usable as a Firefox Addon 🚀 🐰 💕 |
hi @janpio , I believe you may only have one redirect url set per Github Oauth app. You would need a trusted proxy to forward them to the right place, which is a bummer. I think that's why you see many other github extensions use only the personal token approach Edit: as an alternative, you could host a github pages site from this repo that acts as the proxy. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks (a lot!!!) 🙌 for your investigations and work, apologies having not looked into this in a long while. After #36 figured I'd look at everything again. Also started using firefox somewhat more so this is great!
submitted to the store (looks like you might have tried it before? and a few people in the queue, but hopefully it gets approved soon)
@janpio looks like it's working! https://addons.mozilla.org/en-US/firefox/addon/contributor-on-github but yeah didn't try the oauth part either @bkucera, maybe yeah we can do the pages thing or just note that it's' only for chrome atm |
This makes a few tiny changes which makes this extension compatible to Firefox and will enable it to be uploaded to the Firefox Addon Store at https://addons.mozilla.org/ via https://addons.mozilla.org/en-US/developers/
fetch
calls by addingapi.github.com
to thepermissions
array inmanifest.json
which is needed in Firefox to be able to request URLs.When this PR is merged, you should be able to just zip the content of
/src
and upload the resulting file as an addon via https://addons.mozilla.org/en-US/developers/closes #27