Skip to content
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.

Sign up for GitHub

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

Jump to bottom

browser.runtime.onMessage.addListener parameters signature incorrect #31

Closed
mantou132 opened this issue Jun 25, 2020 · 2 comments
Closed

browser.runtime.onMessage.addListener parameters signature incorrect #31

mantou132 opened this issue Jun 25, 2020 · 2 comments

Comments

@mantou132
Copy link

mantou132 commented Jun 25, 2020 •
edited
Loading

https://developer.mozilla.org/en-US/docs/Mozilla/Add-ons/WebExtensions/API/runtime/onMessage#Parameters
@Lusito
Copy link
Owner

Lusito commented Jun 25, 2020

sendResponse is not part of the signature, because of this part:

Returning a Promise is now preferred, as sendResponse() will be removed from the W3C spec.

Does the promise version not work for you?

@mantou132
Copy link
Author

The promise is good, I forgot.

@Lusito Lusito mentioned this issue Jan 31, 2021
Closed
@Lusito Lusito mentioned this issue Apr 26, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Lusito @mantou132