belaczek / hodlwatch

Simple, yet powerful web app for automatic cryptocurrency portfolio tracking
https://hodl.watch
MIT License
121 stars 26 forks source link

Add TimeFrame ALL #51

Open LucBerge opened 2 years ago

LucBerge commented 2 years ago

In the chart section, it is possible to select 1D, 1W, 1M, 6M, 1Y and 2Y. My portfolios are more than 2 years old. I would like to be able to see the evolution of my portfolios since the beginning.

We can add a "ALL" button next to the "2Y" button.

LucBerge commented 2 years ago

In the appConstants.js file, add a TF_ALL value.

In the price priceDataService.js file, add a TF_ALL value with the limit value set to -1 or Infinite.

(By the way, this is not normal to have timeframe constants defined twice. Why don't you merge rhem and use only one ?)

In the Chart/index.js file, add something more to handle the -1 or Infinite value.