react-native-bottom-sheet icon indicating copy to clipboard operation
react-native-bottom-sheet copied to clipboard

Add callback to interrupt the closing of the bottom sheet component

Open AlexanderEggers opened this issue 2 years ago • 24 comments

Please provide enough information so that others can review your pull request:

Motivation

In certain cases we want to conditionally prevent the bottom sheet component from closing and show a warning modal to the user instead.

AlexanderEggers avatar May 01 '23 04:05 AlexanderEggers

This PR is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 10 days.

github-actions[bot] avatar Jun 03 '23 09:06 github-actions[bot]

This PR is still revelant..

AlexanderEggers avatar Jun 03 '23 09:06 AlexanderEggers

@gorhom When you have a moment, can you please review this PR?

AlexanderEggers avatar Jun 14 '23 13:06 AlexanderEggers

@gorhom I think this PR is a relevant use case. Found myself, having to folk and implement something similar

emmsdan avatar Jun 21 '23 07:06 emmsdan

Will look into this this weekend

gorhom avatar Jun 21 '23 08:06 gorhom

Related to https://github.com/gorhom/react-native-bottom-sheet/pull/1320?

karimcambridge avatar Jun 25 '23 18:06 karimcambridge

@gorhom Do you have any updates around this PR?

AlexanderEggers avatar Jul 08 '23 07:07 AlexanderEggers

This PR is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 10 days.

github-actions[bot] avatar Aug 07 '23 09:08 github-actions[bot]

@gorhom Any updates?

AlexanderEggers avatar Aug 07 '23 11:08 AlexanderEggers

This PR is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 10 days.

github-actions[bot] avatar Sep 08 '23 09:09 github-actions[bot]

This PR is still relevant

AlexanderEggers avatar Sep 08 '23 15:09 AlexanderEggers

This PR is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 10 days.

github-actions[bot] avatar Oct 09 '23 09:10 github-actions[bot]

@gorhom I made some minor fixes to the changes and updated the PR with the latest master. Any chance that we can get this one merged soon?

AlexanderEggers avatar Oct 09 '23 10:10 AlexanderEggers

This PR is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 10 days.

github-actions[bot] avatar Nov 09 '23 09:11 github-actions[bot]

@gorhom Any updates?

AlexanderEggers avatar Nov 09 '23 10:11 AlexanderEggers

This PR is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 10 days.

github-actions[bot] avatar Dec 10 '23 09:12 github-actions[bot]

This PR is still relevant.

AlexanderEggers avatar Dec 10 '23 10:12 AlexanderEggers

This PR is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 10 days.

github-actions[bot] avatar Feb 03 '24 09:02 github-actions[bot]

@gorhom When do you think we can have this PR finally merged?

AlexanderEggers avatar Feb 05 '24 00:02 AlexanderEggers

This PR is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 10 days.

github-actions[bot] avatar Mar 20 '24 09:03 github-actions[bot]

This PR is still relevant.

AlexanderEggers avatar Mar 20 '24 11:03 AlexanderEggers

@gorhom any updates?

dima-vantage avatar Apr 15 '24 12:04 dima-vantage

Any updates on this?

edwardanthony avatar May 02 '24 18:05 edwardanthony

Need it too !!

gaedlld avatar May 11 '24 12:05 gaedlld

This PR is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 10 days.

github-actions[bot] avatar Jun 11 '24 09:06 github-actions[bot]