Skip to content

Enable dashboard in dev build#777

Merged
marcelveldt merged 4 commits intomatter-js:mainfrom
Chapoly1305:matterproxy_dev
Jul 3, 2024
Merged

Enable dashboard in dev build#777
marcelveldt merged 4 commits intomatter-js:mainfrom
Chapoly1305:matterproxy_dev

Conversation

@Chapoly1305
Copy link
Copy Markdown

Fix for #776

Super Chiba added 2 commits June 29, 2024 16:45
Enabling Dashboard
Copy link
Copy Markdown
Collaborator

@marcelveldt marcelveldt left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@marcelveldt marcelveldt added the maintenance Code (quality) improvement or small enhancement which not a new feature label Jul 1, 2024
@marcelveldt
Copy link
Copy Markdown
Collaborator

@Chapoly1305 can you have a look at the issues found by the linter ?

Use the installer provided on [Node.js website]. (https://nodejs.org/en/download/package-manager/current)
@Chapoly1305
Copy link
Copy Markdown
Author

@marcelveldt thanks for the notice. I adjusted the nodejs installer and use the official version, the previous in code was from a third-party website. Lint issue should also be fixed.

@MartinHjelmare MartinHjelmare changed the title #776 Enable Dashboard in Dev Build Enable Dashboard in Dev Build Jul 1, 2024
@MartinHjelmare MartinHjelmare changed the title Enable Dashboard in Dev Build Enable dashboard in dev build Jul 1, 2024
@marcelveldt
Copy link
Copy Markdown
Collaborator

Still a lint issue found ?

Remove trailing spaces (right trim)
@Chapoly1305
Copy link
Copy Markdown
Author

Now it shall be all fixed. Please help to trigger the scan again, thanks! :) @marcelveldt

Copy link
Copy Markdown
Collaborator

@marcelveldt marcelveldt left a comment

Choose a reason for hiding this comment

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

Thanks, @Chapoly1305 !

@marcelveldt marcelveldt merged commit 2579f04 into matter-js:main Jul 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintenance Code (quality) improvement or small enhancement which not a new feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants