-
Notifications
You must be signed in to change notification settings - Fork 405
Fe/feature/ri 5681 search and query #4283
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
Merged
Conversation
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
…-search-and-query
…-search-and-query # Conflicts: # redisinsight/ui/src/pages/workbench/utils/searchSuggestions.ts # redisinsight/ui/src/pages/workbench/utils/suggestions.ts
# Conflicts: # redisinsight/ui/src/pages/workbench/utils/searchSuggestions.ts
kchepikava
previously approved these changes
Jan 15, 2025
KIvanow
approved these changes
Feb 3, 2025
ArtemHoruzhenko
added a commit
that referenced
this pull request
Feb 3, 2025
* hide version on Status page * extract rdi version * add default version * add tests * RI-5547 renamed data type, small changes * update message * #RI-6451 - update add database form * RI-6336 - Allow to hide/show columns in Browser - basic functional implementation * RI-6336 - Allow to hide/show columns in Browser - added tooltip * RI-6336 - Allow to hide/show columns in Browser - added column hiding - list view * RI-6336 - Allow to hide/show columns in Browser - added column hiding in the tree view * RI-6336 - Allow to hide/show columns in Browser - refetching of metadata when we reenable one of the checkboxes * RI-6336 - Allow to hide/show columns in Browser - refetching of metadata when we reenable one of the checkboxes * RI-6336 - Allow to hide/show columns in Browser - refetching of metadata when we reenable one of the checkboxes for the tree this time * RI-6336 - Allow to hide/show columns in Browser - telemetry data * #RI-6451 - fix pr comments * RI-6336 - Allow to hide/show columns in Browser - limit for list type * RI-6336 - Allow to hide/show columns in Browser - limit for the hash type * RI-6336 - Allow to hide/show columns in Browser - limit for the set type * RI-6336 - Allow to hide/show columns in Browser - limit for the sorted set type * RI-6336 - Allow to hide/show columns in Browser - limit for the stream type * RI-6336 - Allow to hide/show columns in Browser - limit for the json type * RI-6336 - Allow to hide/show columns in Browser - limit handling in the UI * RI-5547 not show graph key when no graph module * RI-5547 make skip options if no module more generic * RI-6336 converted to an array of columns for better scalability later on * RI-6336 - columns updated * RI-6336 - Allow to hide/show columns in Browser - unit tests pt1 * RI-6336 - Allow to hide/show columns in Browser - unit tests pt2 * RI-6336 - Allow to hide/show columns in Browser - tests for the API * RI-6336 - RI-6336 - Allow to hide/show columns in Browser - cleanup * e2e/feature/RI-6451 * RI-6372 remember context when switching btw rdis * change parallel runs number to 4 * fix for electron ssh test * add e2e for navigation * fix for remove crt * RI-6615 fixed tests and navigation close when switch btw dbs * #RI-6602 - fix scroll * remove comments * update texts * RI-6336 - Allow to hide/show columns in Browser - updating tests based on feedback * #RI-6597 - fix error * RI-6336 - Allow to hide/show columns in Browser - unit tests updated based on feedback from the PR * #RI-6597 - add test * remove comment * RI-6336 - Allow to hide/show columns in Browser - unit tests updated based on feedback from the PR * Unit and Integration tests proposal for change in the workflow (#4284) * remove '+' from suggested branch names as it is treated as a special symbol * testing running all unit tests * RI-6336 - Allow to hide/show columns in Browser - unit tests fixed a broken test * RI-6336 - Allow to hide/show columns in Browser - unit tests fixed integration tests * RI-6336 - Allow to hide/show columns in Browser - unit tests fixed integration tests * RI-6336 - Allow to hide/show columns in Browser - removed unused file * #RI-6554 - update icons and connectivity buttons * #RI-5411 - Optimize builds * #RI-5411 - Optimize builds * #RI-5411 - Optimize builds * #RI-5411 - Optimize builds * RI-5747 filter per RI version in feature config * RI-6629 - TTL and Size are not showing * RI-6630 - Columns icon is not visible for light theme * RI-6631 - There is no info icon next to size to get tooltip * #RI-5411 - fix pr comments * RI-6631 - There is no info icon next to size to get tooltip * RI-6634 - Inconsistent behavior in displaying size * RI-6336 - removed poluted object for testing * RI-6336 - removed poluted object for testing * RI-6336 - removed poluted object for testing * RI-6336 - removed poluted object for testing * RI-6336 - removed poluted object for testing + fixed unit tests * RI-6336 - added cluster strategy implementation. Fixed integration tests * RI-6336 - added cluster strategy implementation. Fixed integration tests * RI-6336 - updated unit tests to match new pattern * fix: redisinsight/api/package.json to reduce vulnerabilities (#4179) The following vulnerabilities are fixed with an upgrade: - https://snyk.io/vuln/SNYK-JS-PATHTOREGEXP-8482416 Co-authored-by: snyk-bot <snyk-bot@snyk.io> * fix: package.json & yarn.lock to reduce vulnerabilities (#4144) The following vulnerabilities are fixed with an upgrade: - https://snyk.io/vuln/SNYK-JS-AXIOS-6671926 Co-authored-by: snyk-bot <snyk-bot@snyk.io> Co-authored-by: Kristiyan Ivanov <kristiyan.ivanov@redis.com> * RI-6547 apply multiline ellipsis to the table tooltip text that overflows, instead of the scroll * RI-6638 - No trash icon to remove key if key size column is hidden. And updated texts for the RI-6336 * adding some clearance for the css styling so it is easier to see to which element(s) is applied * RI-6336 - WIP - moving to per DB config * Bump vite from 5.4.9 to 5.4.12 (#4297) Bumps [vite](https://github.com/vitejs/vite/tree/HEAD/packages/vite) from 5.4.9 to 5.4.12. - [Release notes](https://github.com/vitejs/vite/releases) - [Changelog](https://github.com/vitejs/vite/blob/v5.4.12/packages/vite/CHANGELOG.md) - [Commits](https://github.com/vitejs/vite/commits/v5.4.12/packages/vite) --- updated-dependencies: - dependency-name: vite dependency-type: direct:development ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * add test for RI-6336 * RI-6336 - simplified approach for per db setting * #RI-6628 - adopt profile visualization to work with redis 8+ * remove only * RI-6336 updated unit tests * #RI-6554 - fix pr comments, fix icons color * RI-6336 update naming based on feedback from the PR * RI-6336 - removed unused mocked variables * RI-6336 added recommendation from the PR * #RI-6451 - fix scroll, fix some styles * decrease parallel runs number * #RI-6643 - update link * RI-5335 Add free DB identifier in telemetry events (#4267) - INSIGHTS_PANEL_OPENED - INSIGHTS_PANEL_CLOSED - EXPLORE_PANEL_TUTORIAL_OPENED - EXPLORE_PANEL_COMMAND_COPIED - EXPLORE_PANEL_COMMAND_RUN_CLICKED - EXPLORE_PANEL_LINK_CLICKED - TelemetryPageView.BROWSER_PAGE * updated code owners * RI-5395: Add Nightly Virustotal Analyze (#4305) * RI-000 run tests without e2e by default + add it for bugfix and feature branches * RI-6641 fix css in db nav popover * #RI-6624 - fix slow request for json * #RI-6509 - Create a free Cloud db (#4317) * fix integration tests for RDI after adding "status" call ro obtain version * RI-6527 Resolve an issue on Ubuntu with a space in a folder name && RI-6551 Resolve the app name issue for Debian (#4304) * RI-6527 Resolve an issue on Ubuntu with a space in a folder name * RI-6551 Resolve the app name issue for Debian * RI-6527 Resolve an issue on Ubuntu with a space in a folder name. Updated based on PR feedback * RI-6527 Resolve an issue on Ubuntu with a space in a folder name && RI-6551 Resolve the app name issue for Debian * fixed chrome sandbox permissions * DEV: Add integration tests readme (#4316) * RI-5395: Run VirusTotal Analyze as crone job + support batch validation (#4310) * use upload files instead of url (different VT API) * run array of files * add midnight run * RI-000 run tests without run tests for feature and bugfix branches * RI-6613: Rework redisearch integration tests (#4314) * RI-6613: fixed failing redisearch tests: schema type changes and error message slight change; indexing searching took place before the indexing itself was completed. added waitIndexingToComplete method for waiting before data was searched * add retryLimit=3 and retryInterval=300 as optional parameters in the waitIndexingToComplete function. * separate test cases based on redisearch versions - before 2.8, after 2.8 but before 2.10 and after 2.10 * Fe/feature/ri 5681 search and query (#4283) * #RI-6208 - start refactoring * #RI-5681 - continue refactoring * #RI-6243 - continue refactoring * refactor with count * continue refactoring, add tests * continue refactoring * # continue refactoring * #RI-5681 - fix some commands * #RI-5681 - refactor, code cleanup * #RI-6646 - fix count arguments * #RI-6647 - fix fields suggestions * #RI-6212 - fix ft.create schema * #RI-6212 - add test to cover such case * #RI-6687 - fix count --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: Pavel Angelov <pavel.angelov@redis.com> Co-authored-by: kchepikava <krystsina.chepikava@softeq.com> Co-authored-by: Roman Sergeenko <roman.sergeenko@softeq.com> Co-authored-by: KIvanow <kristiyan.ivanov@redis.com> Co-authored-by: Kristiyan Ivanov <k.ivanow@gmail.com> Co-authored-by: vlad-dargel <u.darhel@aisnovations.com> Co-authored-by: Vlad <vladislav.darhel@softeq.com> Co-authored-by: mariasergeenko <maria.sergeenko@softeq.com> Co-authored-by: zalenskiSofteq <egor.zalenski@softeq.com> Co-authored-by: snyk-bot <snyk-bot@snyk.io> Co-authored-by: Dijana Antovska <dijana.antovska@redis.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: ArtemHoruzhenko <some@user.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
No description provided.