Skip to content
This repository has been archived by the owner on Nov 5, 2021. It is now read-only.

fix(workerManager.js) : Added a check for the existence of the worker #15

Merged
merged 1 commit into from
Aug 17, 2021
Merged

fix(workerManager.js) : Added a check for the existence of the worker #15

merged 1 commit into from
Aug 17, 2021

Conversation

Pranomvignesh
Copy link
Contributor

Description about the issue

monaco-editor-version : 0.25.2
OS : Mac OS
Browser : Chrome

In my application there are 2 instances of monaco editor and when I create the second instance it was throwing this kind of error ( I'm not sure about the proper origin of this issue )

Screen Shot 2021-07-16 at 2 00 40 PM

Upon checking in I found a similar issue microsoft/monaco-editor#1254
I found a similar check in monaco-typescript microsoft/monaco-typescript@e4ab9e4#diff-871939a01a574ea561538fe0d1a0d69d238662de6f7bd102d3a1b2a1212f7a5bR120

So I have added a similar check to monaco-html

@alexdima alexdima added this to the August 2021 milestone Aug 17, 2021
@alexdima
Copy link
Member

Thank you!

@alexdima alexdima merged commit a53ac54 into microsoft:main Aug 17, 2021
@Pranomvignesh Pranomvignesh deleted the check-worker branch November 9, 2021 04:53
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.

2 participants