Closed dbachrach closed 4 years ago
Proxy stores already support external connections but those won't work from a webpage currently. There is an assumption that this.browserAPI.runtime.onMessage exists. This is not true for a webpage, so this PR conditionally skips the safety call.
this.browserAPI.runtime.onMessage
I'm not sure how I missed this. Sorry about that. LGTM as long as merge works as expected.
Proxy stores already support external connections but those won't work from a webpage currently. There is an assumption that
this.browserAPI.runtime.onMessage
exists. This is not true for a webpage, so this PR conditionally skips the safety call.