search

dle07 / cointoss

0 stars 0 forks source link

Heatmap multi stock fix #1

Closed hzco closed 1 year ago

hzco commented 1 year ago

I decoupled the heatmap from the tracker. Heatmap now doesn't rely on Tracker to send the request to the backend. Right now I set it to the name of the stock and the last close price of the stock.

There is still a visual bug where it shows undefined for a brief second before filling in the close price. Have to move the fetching of data and filling out the data array to outside of the Heatmap function so that it renders with the data already loaded.

hzco commented 1 year ago

Also added the ability to request a how far back in time you want data for to the backend

hzco commented 1 year ago

"There is still a visual bug where it shows undefined for a brief second before filling in the close price. Have to move the fetching of data and filling out the data array to outside of the Heatmap function so that it renders with the data already loaded."

Basically lines 19-38 in Heatmap.js needs to be moved before the the Heatmap function somehow.