bitsongofficial / wallet-mobile

1 stars 2 forks source link

Tightly coupled components refactory #42

Open EnricoBarbieri1997 opened 2 years ago

EnricoBarbieri1997 commented 2 years ago

Components (in particular the ones inside BottomSheet) needs to be refactored because some are tightly coupled with stores, outer most components and BottomSheet. For the ones tightly coupled with stores or controllers is enough to have a version where everything is a prop and a version that calculates the current logic and passes it down as prop. Not every component that access stores or models directly needs to be changed but there are a lot of general components that can't be reused because instead of having generic event handlers they have a very specific model passed to it

EnricoBarbieri1997 commented 2 years ago

I will provide an initial list with examples of which ones are fine and which ones are not as soon as possible

EnricoBarbieri1997 commented 2 years ago

Examples of it are: