-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Fix mini dashboard #1755
Fix mini dashboard #1755
Conversation
This commit is a fix to issue meilisearch#1750. As a part of the changes to solve this issue, the following changes have been made - 1. Route registration for static assets has been modified 2. the `mut` keyword on the `scope` has been removed.
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.
Thank you so much for this fix!!
I tested, it works 😄
We wait for the Marin's approval before any merge 🙂
meilisearch-http/src/lib.rs
Outdated
@@ -147,7 +147,7 @@ pub fn dashboard(config: &mut web::ServiceConfig, enable_frontend: bool) { | |||
|
|||
if enable_frontend { | |||
let generated = generated::generate(); | |||
let mut scope = web::scope("/"); | |||
let scope = web::scope("/"); |
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.
let scope = web::scope("/"); |
you can remove this line, and the one at line 167, since they are now useless
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.
One small change and we can merge :)
thank you for contributing to MeiliSearch!
All tests green :) thank you for this opportunity. |
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.
Thank you so much @anirudhRowjee! Especially for your reactivity
If you are participating in Hacktoberfest, and you would like to receive a small gift from MeiliSearch too, please complete this form.
bors merge
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.
Thanks you!! @curquiza you can merge this pr
This commit is a fix to issue #1750.
As a part of the changes to solve this issue, the following changes have
been made -
mut
keyword on thescope
has been removed.