Skip to content
This repository has been archived by the owner on Dec 13, 2018. It is now read-only.

Fix exception on jsimports startup (#1448) #1480

Closed
wants to merge 1 commit into from
Closed

Conversation

hansonw
Copy link
Contributor

@hansonw hansonw commented Mar 14, 2018

@FDiskas I think this is the issue in #1448. Could you try manually patching this into ~/.atom/packages/nuclide/pkg/nuclide-js-imports-server/src/lib/UndefinedSymbolManager.js to see if that does the trick?

@facebook-github-bot
Copy link

@hansonw has updated the pull request.

@FDiskas
Copy link

FDiskas commented Mar 14, 2018

Couldn't initialize jsimports server - Request initialize failed with message: globalsJSON is not defined

@FDiskas
Copy link

FDiskas commented Mar 14, 2018

Updated atom to 1.25.0-beta3 got the same error

Couldn't initialize jsimports server - Request initialize failed with message: globalsJSON is not defined

in devtools there is some errors to
-1521030784468.log

@hansonw
Copy link
Contributor Author

hansonw commented Mar 14, 2018 via email

@FDiskas
Copy link

FDiskas commented Mar 14, 2018

No errors 👯
noerrors

@hansonw
Copy link
Contributor Author

hansonw commented Mar 14, 2018

Awesome. thanks for verifying!

Copy link

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@hansonw has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants