fix(FE): fix the timestamp reduction issue when attempting to reduce the timestamp via drag

[Aarif5435]

Fix: Timestamp Update on Zoom for Graph Data Refresh

Description

This PR resolves an issue with the timestamp update on zoom for graph data in the application. Previously, when attempting to zoom in on the graph by dragging, the timeRange was not included in the requestedData, causing the useMemo hook to recognize the requestedData as unchanged. As a result, the post-request function to fetch new data was not triggered, and the graph failed to refresh. The graph data would only update when the timestamp was manually adjusted from the dropdown selection.

Changes Made

• Added timeRange to the requestedData when zooming in on the graph.
• Ensured that every time the graph is zoomed, the updated timestamp range is dispatched and triggers a re-fetch of data.

Issue

In the attached video, the timestamp reduction on the graph does not refresh correctly upon zoom. The graph only reloads when using the station selection, leading to a poor user experience when trying to zoom. This issue persisted for three days, where initial adjustments worked, but further attempts failed.

Testing

• Verified that zooming via drag now includes timeRange in requestedData.
• Confirmed that the graph data refreshes as expected on each zoom action without requiring a dropdown timestamp change.

Screenshot

Screen.Recording.2024-11-13.at.5.mp4

Notes

This fix will improve the responsiveness of the graph by ensuring real-time data updates on zoom, providing a more seamless user experience.

---

Important

Fixes graph data refresh issue on zoom by adding timeRange to requestedData in NewWidget.

• Behavior:
  • Fixes issue where graph data did not refresh on zoom by adding timeRange to requestedData in NewWidget.
  • Ensures useMemo triggers data re-fetch on zoom by including minTime and maxTime.
• Testing:
  • Verified timeRange is included in requestedData on zoom.
  • Confirmed graph refreshes correctly on zoom without dropdown adjustment.

^{This description was created by for b177594. It will automatically update as commits are pushed.}

[CLAassistant]

All committers have signed the CLA.

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to b177594 in 36 seconds

More details

• Looked at 30 lines of code in 1 files
• Skipped 0 files when reviewing.
• Skipped posting 2 drafted comments based on config settings.

1. frontend/src/container/NewWidget/index.tsx:350

• Draft comment:
  Consider adding 'selectedGraph' to the dependency array of this useMemo to ensure it updates when 'selectedGraph' changes.
• Reason this comment was not posted:
  Comment was on unchanged code.

2. frontend/src/container/NewWidget/index.tsx:88

• Draft comment:
  Avoid using the component/index.tsx file structure approach, as it makes it difficult to debug and find components using global search tools like VS Code.
• Reason this comment was not posted:
  Comment was on unchanged code.

Workflow ID: wflow_0jsQBXB261kpGDoD

---

You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.