-
Notifications
You must be signed in to change notification settings - Fork 410
[RI-7058] Replace EuiInMemoryTable with Table #4640
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
Merged
KrumTy
merged 20 commits into
fe/feature/RI-7039-replace-eui
from
fe/feature/RI-7058-replace-EuiInMemoryTable
Jun 30, 2025
Merged
Changes from all commits
Commits
Show all changes
20 commits
Select commit
Hold shift + click to select a range
6339aa8
RI-7058: expose redis ui Table component
KrumTy 9673114
update Table for TopKeys
KrumTy afa5cce
use Table for TopNamespaces
KrumTy 815e860
use Table for UserApiKeysTable
KrumTy 60d490e
use Table for TestConnectionsTable
KrumTy b84435a
use Table for TableResult
KrumTy a6afb0f
use Table for ClusterNodesTable
KrumTy e5939c9
update Table for TableInfoResult
KrumTy d2418e9
use Table for TableResult
KrumTy 2a670e0
use Table for ShortcutsTable
KrumTy edb93b5
use Table for RedisClusterDatabasesPage
KrumTy b1bd65a
use Table for SentinelDatabasesResultPage
KrumTy 382a2dc
use Table for SentinelDatabasesPage
KrumTy cb00a6c
use Table for RedisCloudSubscriptionsPage
KrumTy 1961ab8
use Table for RedisCloudDatabasesResultPage
KrumTy 1e1f704
use Table for RedisCloudDatabasesPage
KrumTy 0e64e60
use Table for redisgraph
KrumTy 0d70f81
use Table for TableView
KrumTy e57df2c
use Table for rdi tables
KrumTy c5aa10b
Merge remote-tracking branch 'origin/fe/feature/RI-7039-replace-eui' …
KrumTy File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1 @@ | ||
| export * from '@redis-ui/table' |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
38 changes: 0 additions & 38 deletions
38
redisinsight/ui/src/components/shortcuts-flyout/styles.module.scss
This file was deleted.
Oops, something went wrong.
28 changes: 8 additions & 20 deletions
28
redisinsight/ui/src/packages/clients-list/src/components/table-view/TableView.tsx
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file was deleted.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
From what I see in tanstack table docs,
accessorKeyis optional, and from what I see in usages here, it is 100% of the time the same asidfield. Have you tested if the tables work with just id?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.
I've tested without and cell items disappear in columns
it's probably optional since you can also have accessorFn
anyway - doesn't seem to default to "access the field that corresponds to the id"