-
Notifications
You must be signed in to change notification settings - Fork 144
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
Setup Dashboard on custom endpoint #664
Conversation
Signed-off-by: Leonardo Rossi <leonardo.rossi@gmail.com>
810255c
to
6fcb7e0
Compare
packages/db-dashboard/index.js
Outdated
}) | ||
// app.get(`${dashboardPath}/*`, { hide: true }, function (req, reply) { | ||
// return reply.sendFile('index.html') | ||
// }) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why is this commented out?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fastify-static
gives me the error of #560 , I'm looking how to solve this
I think this also solves #560 |
@leorossi could you fix linting? |
Signed-off-by: Leonardo Rossi <leonardo.rossi@gmail.com>
Ended up a bit more complicated, because the react app relied on the I added a script that fixes the |
Signed-off-by: Leonardo Rossi <leonardo.rossi@gmail.com>
Signed-off-by: Leonardo Rossi <leonardo.rossi@gmail.com>
Signed-off-by: Leonardo Rossi <leonardo.rossi@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
@mcollina this may be a breaking change in configuration because I renamed
dashboard.rootPath
todashboard.path
and that would be a string.@Eomm I used the
{ path: ... }
pattern to be consistent with theplugins
configurationLet me know what you think
Fixes #657
Signed-off-by: Leonardo Rossi leonardo.rossi@gmail.com