-
Notifications
You must be signed in to change notification settings - Fork 744
SOLR-17658: Implement start screen #3388
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
SOLR-17658: Implement start screen #3388
Conversation
@epugh Even though this PR is not ready to be merged yet, but maybe you still want to take a look? |
3d9121c
to
c881a66
Compare
solr/ui/src/wasmJsMain/kotlin/org/apache/solr/ui/errors/parseError.kt
Outdated
Show resolved
Hide resolved
I gave it a quick spin and it worked! Gave a reasonable error message back with no solr running, and then I also tried it with a old 8.11 Solr at http://quepid-solr.dev.o19s.com:8985 and it worked! The resulting pages didn't render the environment settings which I would assume is because of the age of the solr! I suppose the follow on to this would be a logout button ;-) |
This is actually the next step together with the logout button. :)
Well, at first sight I thought this is expected behavior, but it was actually a bug. I forgot to update the URL in the HTTP client, so the request was sent to localhost instead. I fixed it and now it should work. As long as the API endpoints exist in the older versions, the UI should be able to render it correctly. We will add error handlings so that compatibility to older versions may be preserved to a certain degree. But one step at a time. |
…rror.kt Co-authored-by: Eric Pugh <epugh@opensourceconnections.com>
It worked with http://quepid-solr.dev.o19s.com:8987! |
https://issues.apache.org/jira/browse/SOLR-17658
Description
The start screen is the entry point where a connection to a Solr instance is established.
Solution
The implementation adds a new start screen where the user can insert a Solr URL (or use the default) and connect to a Solr instance. In case authentication is enabled for that specific Solr instance, it properly redirects to an authentication component (dummy component).
Tests
The PR adds a bucnh of tests that test both the new components and existing ones.
Checklist
Please review the following and check all that apply:
main
branch../gradlew check
.