nick-boss-tech opened a new pull request, #5269:
URL: https://github.com/apache/solr/pull/5269

   🤖 *AI text below* 🤖 *(reviewed by Nick Shanin)*
   
   Jira: [SOLR-17849](https://issues.apache.org/jira/browse/SOLR-17849)
   
   ## What happens today
   
   **The Index & Query header exists as a component, but no screen in the new 
UI hosts it.**
   
   PR #5257 adds the header (the collection and request handler dropdowns) at 
Christos Malliaridis's direction, with this ticket's scope reduced to the 
header only and the content as a separate change, so the two can iterate 
independently. Until the content lands, the header cannot be reached or tried 
in the running UI.
   
   ## What this change does
   
   **The Index & Query screen is hosted on the Configsets Index / Query tab, 
wired to the header's state holder.**
   
   - 
[`IndexQueryContent`](https://github.com/nick-boss-tech/solr/blob/dc8c1c75c678d5923f4825398df04b83cc35685f/ui/shared/src/commonMain/kotlin/org/apache/solr/ui/views/indexquery/IndexQueryContent.kt)
 renders the header from the current dropdown-state API and shows the section's 
error state.
   - 
[`IndexQueryViewModel`](https://github.com/nick-boss-tech/solr/blob/dc8c1c75c678d5923f4825398df04b83cc35685f/ui/shared/src/commonMain/kotlin/org/apache/solr/ui/components/indexquery/viewmodel/IndexQueryViewModel.kt)
 owns the state holder and starts the collection load.
   - The Configsets component, route view model, and content host the screen on 
the tab, and the preview covers the hosting.
   
   This branch is stacked on PR #5257's branch; until that merges, this PR's 
diff includes the header commits. The request form is not part of this change: 
the form is SOLR-17850's task and plugs in below the header when it lands.
   
   ## Proof
   
   **Verified on Linux at `dc8c1c75c6` on 2026-10-30: spotless passes and the 
shared module's JVM suite passes.**
   
   <details>
   <summary>Linux verification at dc8c1c75c6</summary>
   
   - `:shared:spotlessCheck`: passes.
   - `:shared:jvmTest`: 59 tests, 0 failures, 2 skipped (both skips 
pre-existing: UserAuthenticationContentTest, StartContentTest). 
IndexQueryStateHolderTest 3/3, HttpIndexQueryRepositoryTest 3/3, 
IndexQueryHeaderTest 6/6.
   - The wasm browser suite was not run locally for this branch (this VM cannot 
fetch its npm dependencies); the UI check workflow runs that suite in CI.
   
   </details>
   
   ## Limits
   
   The screen hosts the header only; with no form yet, selecting a handler 
shows no further UI. That is the intended intermediate state under the split 
Christos Malliaridis set out on the ticket.
   
   *AI-generated description. Nick Shanin directed the work and takes 
responsibility for this contribution.*
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to