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

Missing SharedWorkerGlobalScope #36027

Closed
jiripudil opened this issue Jan 6, 2020 · 8 comments
Closed

Missing SharedWorkerGlobalScope #36027

jiripudil opened this issue Jan 6, 2020 · 8 comments
Labels
Bug A bug in TypeScript Domain: lib.d.ts The issue relates to the different libraries shipped with TypeScript Help Wanted You can do this
Milestone
Backlog

Comments

@jiripudil
Copy link

Similar to #24323, the SharedWorkerGlobalScope type is missing in the webworker lib
@RyanCavanaugh RyanCavanaugh added Bug A bug in TypeScript Domain: lib.d.ts The issue relates to the different libraries shipped with TypeScript Help Wanted You can do this labels Jan 7, 2020
@RyanCavanaugh RyanCavanaugh added this to the Backlog milestone Jan 7, 2020
@DCtheTall
Copy link

I'd be interested in adding this to the webworker lib 😄

@DCtheTall
Copy link

@jiripudil does there need to be anything added besides what is in this PR?

@jiripudil
Copy link
Author

@DCtheTall yes, the PR adds the SharedWorker class for registering and communicating with the worker. The global scope I'm talking about is the type of self in the worker script itself.

@DCtheTall
Copy link

DCtheTall commented Jan 31, 2020 •
edited
Loading

@jiripudil thanks for the reply. I will open a PR once the one I linked to is merged/closed to avoid conflicting with their work.

@DCtheTall
Copy link

@jiripudil is there any estimate on when that PR is going to be merged? It's been open for some time, and now that it's blocking this issue I am wondering what the plan was (if there is one)?

@jiripudil
Copy link
Author

@DCtheTall honestly no idea, I don't work for Microsoft 😅

@DCtheTall
Copy link

@RyanCavanaugh should I wait for the existing PR or just create a new one

@orta
Copy link
Contributor

orta commented Mar 4, 2020

This got merged on the JSTS side, and should be coming up in 3.9

@orta orta closed this as completed Mar 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug A bug in TypeScript Domain: lib.d.ts The issue relates to the different libraries shipped with TypeScript Help Wanted You can do this
Projects
None yet
Milestone
Backlog
Development

No branches or pull requests
4 participants
@orta @jiripudil @RyanCavanaugh @DCtheTall