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

Dashboard #17

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Dashboard #17

wants to merge 7 commits into from

Conversation

czystyl
Copy link
Collaborator

@czystyl czystyl commented Sep 14, 2019

The dashboard runs 6 commands from all of the directories and shows the output.

We need to define the commands that are running in the watch mode to get the real benefit of this dashboard.

image

close #10

@czystyl czystyl changed the title Dashboard [WIP] Dashboard Sep 14, 2019
@czystyl czystyl changed the title [WIP] Dashboard Dashboard Sep 14, 2019
@wokalski
Copy link
Collaborator

this is fucking awesomee

@czystyl
Copy link
Collaborator Author

czystyl commented Sep 16, 2019

Added some options:

  • full - full project overview
  • core - build only core packages
  • examples - build only examples

You can check it by running yarn dashboard {type}

full core examples
image image image

@wokalski
Copy link
Collaborator

I can't run the dashboard from the root dir. I'd ideally run "yarn dashboard type"from the root directory of the project.

The React.js code looks great. I don't want to nitpick but I was wondering whether hooks + functions could be used here?
I was also worndegin if TypeScript would be an option here?

Copy link
Collaborator

@wokalski wokalski left a comment

Choose a reason for hiding this comment

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

The pictures look great but I wish I could try it :D. Let me know how to run it and what you think about the refactors I've mentioned.

packages/dashboard/Client.js Outdated Show resolved Hide resolved
packages/dashboard/run.js Outdated Show resolved Hide resolved
@czystyl
Copy link
Collaborator Author

czystyl commented Sep 18, 2019

I try the hook/function component approach, but there the spawn wasn't worked as expected. But now we have the all of the needed bit, so I can come back and refactor it.

I'll go and try what you propose and let's see 🚀

@@ -1,13 +1,15 @@
{
"private": true,
"scripts": {
"dashboard": "node packages/dashboard/run",
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

That's why you was not able to run the dashboard. Sorry.

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.

Dev dashboard + watcher
2 participants