digidem / comapeo-mobile

The next version of Mapeo mobile
GNU General Public License v3.0
5 stars 0 forks source link

fix: use onDismiss() prop in BottomSheetModal component #389

Closed achou11 closed 1 month ago

achou11 commented 1 month ago

the prop was defined but never used in the component when it should be passed to the library's BottomSheetModal onDismiss prop, like it's done in Mapeo Mobile.

There are a few consumers of this component that specify the prop for things like cleanup, which would've led to some unexpected behavior without this fix.