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

--startup.project option does not work with the new dashboard. #6666

Closed
farmaazon opened this issue May 12, 2023 · 4 comments · Fixed by #6671 or #6728
Closed

--startup.project option does not work with the new dashboard. #6666

farmaazon opened this issue May 12, 2023 · 4 comments · Fixed by #6671 or #6728
Assignees
Labels
--bug Type: bug --regression Important: regression

Comments

@farmaazon
Copy link
Contributor

When providing --startup.project option, the IDE went directly to the project view, now it displays Dashboard instead. We should fix this or remove the option from the list.

The latter is less preferable, as it's convenient to me to run something like enso-build-cli ide watch --skip-wasm-opt --ide-option --debug.dev-tools --ide-option --startup.project --ide-option Unnamed_7 so I don't have to pick the project on every electron restart.

@farmaazon
Copy link
Contributor Author

Moreover, this option breaks the project: No matter what project do I open in the dashboard, the appointed one is opened, but the node are not colored.

@somebody1234
Copy link
Contributor

should it be able to go back to the dashboard? (it's a lot easier to not allow it to go back to the dashboard, just wondering what is the more correct thing to do here)

@somebody1234
Copy link
Contributor

on second thought, i think the dashboard should be completely disabled, since manipulating the URL query string to remove -startup.project to allow opening a different project is A Bad Idea

@somebody1234
Copy link
Contributor

for posterity: it will be changed to open the dashboard soon, since we want to avoid introducing different modes where e.g. something is disabled. see this comment for more info:
#6671 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
--bug Type: bug --regression Important: regression
Projects
Archived in project
3 participants