Skip to content
This repository was archived by the owner on Apr 28, 2020. It is now read-only.

Firefox extension#243

Merged
deansheather merged 5 commits intomasterfrom
firefox-extension
Sep 17, 2019
Merged

Firefox extension#243
deansheather merged 5 commits intomasterfrom
firefox-extension

Conversation

@deansheather
Copy link
Copy Markdown
Member

Adds Firefox support to the native host installer and makes the extension web extensions compatible.

#225

deansheather and others added 3 commits September 17, 2019 11:57
Renamed install-for-chrome-ext to install-ext-host, added Firefox
support and added a new deprecated command alias for the old name.

Co-authored-by: Luca Casonato <luca.casonato@antipy.com>
@deansheather deansheather merged commit 862904b into master Sep 17, 2019
@deansheather deansheather deleted the firefox-extension branch September 17, 2019 19:58
Copy link
Copy Markdown

@sr229 sr229 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM otherwise

Comment thread extension.go
}
nativeHostDirsChrome, err := nativeMessageHostManifestDirectoriesChrome()
if err != nil {
flog.Fatal("failed to get chrome native message host manifest directory: %v", err)
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Probably a more generic message here sincr we're aiming for two vendors

1. [Install Sail if you haven't already](/docs/installation)
1. Run `sail install-for-chrome-ext` to install the chrome extension manifest.json
1. Run `sail install-ext-host` to install the extension manifest.json
1. [Install the extension from the Chrome Marketplace](https://chrome.google.com/webstore/detail/sail/deeepphleikpinikcbjplcgojfhkcmna)
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Make sure you edit this post merge for the Firefox extensions link.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants