New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Followup on removing last chart price overrides #4506
Conversation
fe29023
to
4fbf335
Compare
944a8d9
to
86ab846
Compare
APP-301 Remove the override of the last price in chart data
Commits can be read in order. We had an override of the last price point in chart data to match it with the "current price" of an asset. However, sometimes the last price point we have is stale data, and it caused weird inconsistent issues with charts. This PR removes that manual override and just uses the default chart data as is. In the course of doing this PR, I noticed quite a few other params that were unused or unnecessary. In order to test this PR, charts should behave as expected, and we should no longer see any stale price chart data or weird lines on charts. |
fetchingCharts: false, | ||
fetchingCharts2: false, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🙃
4fbf335
to
bef893a
Compare
The relevant hooks and components use chartType passed along in params when necessary - they do not read the redux state for it.
This is just to help simplify as there is another useChartData from the reanimated chart package.
bef893a
to
bbdd3d5
Compare
…cations-onboarding-improvements * origin/develop: (102 commits) Ledger: Match "Looking for devices" screen to spec + animated device image (#4222) feat: add logger.log, send log/warn to Sentry (#4544) Remove generic asset fallback (#4527) Remove optimism explorer redux (#4524) Fix L2 transaction badges in transactions history (#4539) Change copy from View Contact to Edit Contact to avoid confusion (#4538) Fix ENS transactions bug, missing network parameter in RainbowTransaction (#4543) Update original asset address (#4540) Update remote config (#4535) rc v1.8.6 fixes (#4523) ledger: init ledgerSigner + logger context (#4515) load the correct provider (#4536) audit: ignore debug (#4531) feat: add walletconnect debug context, use new logger (#4514) ledger: holdToAuthorizeButton (#4516) feat: align approval sheet navigation params (#4517) chore: remove com.polidea.reactnativeble.BlePackage import (#4518) Remove transaction details playground from dev menu (#4519) Add scroll enabled back and disable status icon for short phones (#4521) Followup on removing last chart price overrides (#4506) ...
Follow up on Fixes APP-301