-
Notifications
You must be signed in to change notification settings - Fork 20
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
fix(react-components): asset search hook to support more than 1000 results #4655
Conversation
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.
Looks nice, will spend some time testing after you've had a chance to go through the review 👍
react-components/src/query/useSearchMappedEquipmentAssetMappings.tsx
Outdated
Show resolved
Hide resolved
react-components/src/query/useSearchMappedEquipmentAssetMappings.tsx
Outdated
Show resolved
Hide resolved
react-components/src/query/useSearchMappedEquipmentAssetMappings.tsx
Outdated
Show resolved
Hide resolved
react-components/src/query/useSearchMappedEquipmentAssetMappings.tsx
Outdated
Show resolved
Hide resolved
react-components/src/components/RuleBasedOutputs/RuleBasedOutputsSelector.tsx
Outdated
Show resolved
Hide resolved
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.
LGTM
hakon on vacation and have address his review comments
Type of change
Jira ticket 📘
https://cognitedata.atlassian.net/browse/BND3D-4459
Description 📝
Existing search hook support 1000 asset as the end point
asset.search
supports only 1000, in this PR I have changed the way searching of asset work. I have added support for infinite asset search usingasset.list
utilizing advanced filter option.How has this been tested? 🔍
In storybook and fusion
Test instructions ℹ️
SearchHooks
storybook exampleAll asset mappings
,Asset search hook
andLoad more
Checklist ☑️