-
Notifications
You must be signed in to change notification settings - Fork 52
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
Ability to set default instance #43
Comments
I would like to also add my opinion on this, as I have been pondering upon this for a while. We would like to have a self-contained package (Helm Chart/Docker Compose) to spin up a working setup for our clients. The Meilisearch UI is bundled in this package, but currently right now we are not yet able to make the UI integrate automatically to the stack. If I understand the code correctly, we have to do some major refactoring to have this type of feature. Two approaches of the top of my mind:
Another addition that should be implemented for this specific feature set is authentication. Maybe a simple BasicAuth should be enough. Happy to hear your thoughts on this @riccox, I would like to also contribute to the project for this particular feature set. |
@antran22 Your idea is similar with some of my current ideas basicly. First approach, need huge refactoring:
Another way (easy to realize):
|
On a related note, I'd like the ability to pass the server host via a query string, and then be prompted for the password only if that instance isn't already saved in the browser. That is, go to https://meili-manager.vercel.app/#/?host=http://127.0.0.1:7700 I'm guessing many of us go there the first time we try this. |
@tacman The screenshot you posted is not Meilisearch-UI app. 😅 You will find the app version & its hash tag at the bottom of the page if you are using Meilisearch-UI. |
I would like to integrate meilisearch-ui within the dev env, but I need to setup connection everytime I reset the docker containers.
It would be better if we could pass meilisearch instance configurations via env vars.
The text was updated successfully, but these errors were encountered: