shapeshift / web

ShapeShift Web
https://app.shapeshift.com
MIT License
173 stars 190 forks source link

Remove requirement for full tx history on app boot #7872

Open woodenfurniture opened 2 months ago

woodenfurniture commented 2 months ago

Overview

On app boot, we currently initiate eager load of tx history. Depending on existing cache this can be:

The main driver of the need for eager tx history loading is the balance history which requires full tx history to be known for the time period being displayed on the balance history chart. If we negate this requirement, we should be able to request a single page of tx history for the specific account(s) a user is viewing, lazily.

References and additional details

This should be don incrementally in case our requirements change:

  1. Hide the balance history chart from the "My Wallet" page to measure users reaction
  2. Remove all code relating to balance history chart
  3. Remove balance history fetch on boot, make specific parts of the app page on-demand (with caching in redux store)
  4. Implement replacement chart/feature not requiring large numbers of requests on boot @twblack88

Acceptance Criteria

App doesn't eagerly load balance history on boot

Need By Date

No response

Screenshots/Mockups

No response

Estimated effort

No response

woodenfurniture commented 1 month ago

Blocked pending discussion with product on long-term goals. There are multiple other parts of the app requiring solution to not rely on tx history: