determined icon indicating copy to clipboard operation
determined copied to clipboard

fix: Unable to fetch data after sorting or filtering items or filter

Open keita-determined opened this issue 2 years ago • 1 comments

Description

DET-7930

Test Plan

  • Check if we can able to fetch data after clicking sort many times in few seconds (ideally click 10 times in a second and table should have at least 50 items) Check out Jira ticket for the reproducing video

Commentary (optional)

We use polling every 5 sec so I removed fetching experiments when sorting or filtering are triggered because it causes the issue. I tried using throttle and debouce but it's not working well due to useEffect. Is there any better approach besides removing the useEffect?

Checklist

  • [ ] User-facing API changes need the "User-facing API Change" label.
  • [ ] Release notes should be added as a separate file under docs/release-notes/. See Release Note for details.
  • [ ] Licenses should be included for new code which was copied and/or modified from any external code.
  • [ ] If modifying /webui/react/src/shared/ verify make -C webui/react test-shared passes.

keita-determined avatar Aug 02 '22 04:08 keita-determined

Deploy Preview for determined-ui ready!

Name Link
Latest commit affa3e6d4163b8f03c947b81d134bd8153aa8b6b
Latest deploy log https://app.netlify.com/sites/determined-ui/deploys/62f3fd0b4ee180000976eb30
Deploy Preview https://deploy-preview-4696--determined-ui.netlify.app/
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

netlify[bot] avatar Aug 02 '22 04:08 netlify[bot]

this behavior is resolved, close!!

keita-determined avatar Oct 17 '22 19:10 keita-determined