Ruthvik-1411 / Finance_Tracker_app

Basic finance tracker for personal use
0 stars 0 forks source link

Fix: Latency issue in investments screen #23

Closed Ruthvik-1411 closed 3 months ago

Ruthvik-1411 commented 3 months ago

Resolves Issues #21 , #22

Latency:

  1. The latency observed when switching from investment screen to any other screen is identified due to too many rendering components. The pie chart render has not been optimized similar to tracker screen as done in this #14 .
  2. The idea of auto refresh data in the screen also seemed unnecessary and too many state keepers. So removed that and added initial data loading similar to banking screen.

Expo go build issue:

  1. Made changes as commented here, https://github.com/Ruthvik-1411/Finance_Tracker_app/issues/22#issuecomment-2267436956

Others:

  1. prettify bottomtab.js and renamed the label for screen summary and changed it to tracker screen, to keep track and simpler.
  2. Update date picker in add expenses screen to limit date selection to past dates only and not future dates.

The latency issue has been resolved and works as expected. The expo development has also been reverted back to original state.