Skip to content

RI-7197 Rework the manage index section #4785

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

Conversation

valkirilov
Copy link
Member

Description

Rework the table that should show details about the available indexes in the Manage Indexes drawer, part of the new Vector Search feature flow.

  • remove the custom styles from the table
  • update the table columns to include "Identifier" and "No index", and remove "Separator"
  • added additional index summary details below the table
Old New
image image

How to test it

These are base components that will be included in the new wizard for the Vector Search, but currently, you can find them integrated on a temporary page, for the sake of testing them.

Simply flip this temporary switch in redisinsight/ui/src/pages/vector-search/VectorSearchPage.tsx and then click on the "Search" icon in the left side nav to open the page:

export const VectorSearchPage = () => {
  const hasIndexes = true // Toggle between "wizard" and "search" view
  1. Go to the new Vector Search page (use the magnifier icon from the left side nav)
  2. In the top right corner, click on the Manage Indexes button
  3. A drawer will open, with information about the existing indexes
image

- remove the custom styles from the table
- update the table columns to include "identifier" and "no index", and remove "separator"
- added index summary details bellow tyhe table

re #RI-7197
@valkirilov valkirilov changed the title feat(ui): rework the manage index section RI-7197 Rework the manage index section Aug 1, 2025
@valkirilov valkirilov force-pushed the fe/feature/RI-7197_vector-search-manage-indexes-rework branch from 5c0e5a8 to 4589777 Compare August 1, 2025 10:55
expect(summaryInfo).toBeInTheDocument()
})

it('should render loader when index info is not provided', () => {
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Image

expect(loader).toBeInTheDocument()
})

it('should render index attributes in the table', () => {
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Image

Copy link
Contributor

github-actions bot commented Aug 1, 2025

Code Coverage - Frontend unit tests

St.
Category Percentage Covered / Total
🟢 Statements 81.48% 19152/23504
🟡 Branches 66.86% 8314/12435
🟡 Functions 75.28% 5031/6683
🟢 Lines 81.89% 18744/22888

Test suite run success

4917 tests passing in 649 suites.

Report generated by 🧪jest coverage report action from 4589777

@valkirilov valkirilov marked this pull request as ready for review August 1, 2025 11:22
@valkirilov valkirilov self-assigned this Aug 1, 2025
@valkirilov valkirilov merged commit a2e14f3 into feature/RI-6855/vector-search Aug 4, 2025
17 checks passed
@valkirilov valkirilov deleted the fe/feature/RI-7197_vector-search-manage-indexes-rework branch August 4, 2025 05:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants