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

Online/Web Editor Default Project Directory is bad #44754

Closed
mrjustaguy opened this issue Dec 28, 2020 · 1 comment · Fixed by #45705
Closed

Online/Web Editor Default Project Directory is bad #44754

mrjustaguy opened this issue Dec 28, 2020 · 1 comment · Fixed by #45705

Comments

@mrjustaguy
Copy link
Contributor

mrjustaguy commented Dec 28, 2020

Godot version:

Godot Web Editor 3.2.4 beta 4 https://godotengine.org/editor/latest/godot.tools.html (Currently b5e8b48)

OS/device including version:

Windows 10 1909, Google Chrome

Issue description:

the Default Path When Creating a new Project is "/home/web_user" however creating a new project at "/home/web_user/New Game Project" results in the new going bye bye the moment it is created.

Default Path Should be "/home/web_user/projects" instead of "/home/web_user" as "/home/web_user/projects" keeps the projects and doesn't instantly discard them.

Steps to reproduce:

  1. Load the Web Editor, Make a New project with the default path
  2. Wait like a second, after that the New project will be missing

Minimal reproduction project:

Cannot make MRP

@Faless
Copy link
Collaborator

Faless commented Dec 28, 2020

Yeah, I can confirm this. It was originally done to avoid storing the cache in the IndexedDB to save space, but in the end I opted to include it too (so opening projects is faster).
I'll try to come up with a better solution for the next version. I wonder if we can add boundaries to the project manager too (so it won't allow creating a project in /home/MyProject either).

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.

3 participants