fix: use useQuery #87
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
Collaborator
mosoriob
commented
May 24, 2025
mosoriob
commented
- refactor: migrate hooks to use react-query for data fetching
- feat: add SensorStatistics model and integrate into GetSensorResponse and SensorItem
- feat: update MeasurementsSummary and SensorTable to utilize SensorStatistics
- fix: update MeasurementSummary to utilize statistics for displaying measurement data
- feat: integrate @tanstack/react-query for improved data management
- Replaced useState and useEffect with useQuery from react-query in multiple hooks, including useDetail, useList, and useProcessedMeasurements, to streamline data fetching and state management. - Improved error handling and loading states across hooks, enhancing overall performance and user experience. - Updated query keys and functions to ensure consistent data retrieval and handling.
… and SensorItem - Introduced a new SensorStatistics model to encapsulate statistical data related to sensors. - Updated GetSensorResponse and SensorItem models to include statistics as a property, enhancing data representation. - Removed unused UploadfileApi from the API exports for cleaner code structure. - Adjusted index files to reflect the addition of the new SensorStatistics model.
…tistics - Enhanced MeasurementsSummary component to display sensor statistics (count, average, min, max) with proper null checks, ensuring accurate data representation. - Modified SensorTable to include new columns for sensor statistics, improving data visibility and organization. - Updated DataTable to format statistics values correctly, providing a consistent user experience across components.
…easurement data - Enhanced MeasurementSummary component to use statistics for first and last measurement times, min, max, average values, and count, ensuring accurate data representation. - Added null checks for statistics properties to handle cases where data may be unavailable, improving user experience by displaying 'N/A' when necessary.
- Added @tanstack/react-query as a dependency to enhance data fetching and state management across components. - Wrapped the application in QueryClientProvider to enable query management. - Simplified data fetching in CampaignList component by removing unnecessary destructuring. - Updated LineConfidenceContext to handle sensor data more robustly with improved type definitions. - Enhanced SensorDashboard to conditionally render MeasurementsSummary based on data availability.
Closed
❌ Deploy Preview for dso-tacc failed.
|
… handling - Added optional showPoints prop to RouterMap for conditional rendering of measurement points. - Updated StatsSection to fetch measurements using useList hook, improving data management. - Integrated QueryWrapper in StatsSection for better loading and error handling during data fetching.
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.