determined icon indicating copy to clipboard operation
determined copied to clipboard

perf: improve too many user api call

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

Description

DET-7962

Test Plan

  • Check if user api call is reduced (user api call less than 5 times before polling starts in network tab)
    • user api used to be called more than 5 times before polling starts
  • Check if user data is fetched appropriately instead of no user data

Commentary (optional)

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 10 '22 22:08 keita-determined

Deploy Preview for determined-ui canceled.

Name Link
Latest commit c8ef5086f12490892f87f03199b61f0260c73539
Latest deploy log https://app.netlify.com/sites/determined-ui/deploys/62f4949cad45440008fb2c4d

netlify[bot] avatar Aug 10 '22 22:08 netlify[bot]

test-cli is failed. I believe that it should fail cuz the code change is not related to what ci tests

keita-determined avatar Aug 11 '22 05:08 keita-determined