Skip to content

Add Worker.postMessage override #703

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

Merged
merged 2 commits into from
May 23, 2019
Merged

Conversation

sandersn
Copy link
Member

@sandersn sandersn commented May 22, 2019

Matches MessagePort, ServiceWorker and DedicatedWorkerGlobalScope, followup to #679

Fixes microsoft/TypeScript#31448

Matches MessagePort, ServiceWorker and DedicatedWorkerGlobalScope
@saschanaz
Copy link
Contributor

Cherry-picking 1429ea2 will fix the error.

@sandersn
Copy link
Member Author

sandersn commented May 23, 2019

Got it, thanks. (I had those changes locally but thought it might just be my local clone's problem since I hadn't tried deleting node_modules yet.)

@sandersn sandersn merged commit 73f17d8 into master May 23, 2019
@sandersn sandersn deleted the add-worker-postmessage-override branch May 23, 2019 15:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Worker.postMessage missing optional options parameter
2 participants