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.

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

3.4.3rc-1 web editor not working #57592

Closed
tavurth opened this issue Feb 3, 2022 · 3 comments · Fixed by #57598
Closed

3.4.3rc-1 web editor not working #57592

tavurth opened this issue Feb 3, 2022 · 3 comments · Fixed by #57598

Comments

@tavurth
Copy link
Contributor

tavurth commented Feb 3, 2022

Godot version

https://editor.godotengine.org/releases/3.4.3.rc1/

System information

OSX (Chrome)

Issue description

The web editor no longer seems to be working:

https://editor.godotengine.org/releases/3.4.3.rc1

godot.tools.js:380 Uncaught (in promise) TypeError: Cannot read properties of undefined (reading 'godot.tools.wasm')
    at b.B (godot.tools.js:380:298)
    at startEditor ((index):641:12)
    at (index):667:5

Steps to reproduce

Go here

https://editor.godotengine.org/releases/3.4.3.rc1/

Minimal reproduction project

No response

@SkanerSoft
Copy link

Windows 11, Google Chrome
1643906915

@Calinou
Copy link
Member

Calinou commented Feb 3, 2022

I can confirm this on Firefox 96 on Linux:

Uncaught (in promise) TypeError: this.i.A is undefined
    B https://editor.godotengine.org/releases/3.4.3.rc1/godot.tools.js:380
    startEditor https://editor.godotengine.org/releases/3.4.3.rc1/:641
    <anonymous> https://editor.godotengine.org/releases/3.4.3.rc1/:667
    promise callback* https://editor.godotengine.org/releases/3.4.3.rc1/:666
    EventHandlerNonNull* https://editor.godotengine.org/releases/3.4.3.rc1/:665
[godot.tools.js:380:281](https://editor.godotengine.org/releases/3.4.3.rc1/godot.tools.js)

@Faless
Copy link
Collaborator

Faless commented Feb 3, 2022

Confirmed, likely a regression from this: 2f509f1 (#57482) .
I did forget the CC would complain about that. I'll see if I can tell it to leave the names in the config alone, or revert that commit in case.

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

Successfully merging a pull request may close this issue.

6 participants