[Suggestion] [v3] Do not emit warning for undefined hook properties

[unendingblue]

Description

In the current v3 implementation, setting a hook property as undefined emits the following error:

WARN  [react-native-gesture-handler] <property> is not a valid property for <handler> and will be ignored.

It should instead silently ignore those properties.

Steps to reproduce

const panGesture = usePanGesture({
  [standard configuration here],
  activeOffsetY: Platform.OS === 'android' ? [-75, 75] : undefined,
});

A link to a Gist, an Expo Snack or a link to a repository based on this template that reproduces the bug.

Gesture Handler version

3.0.0-beta.1

React Native version

0.83.2

Platforms

iOS

JavaScript runtime

None

Workflow

None

Architecture

None

Build type

None

Device

None

Device model

No response

Acknowledgements

Yes

[github-actions]

Hey! 👋

The issue doesn't seem to contain a minimal reproduction.

Could you provide a snack or a link to a GitHub repository under your username that reproduces the problem?

[github-actions]

Hey! 👋

It looks like you've omitted a few important sections from the issue template.

Please complete A link to a Gist, an Expo Snack or a link to a repository based on this template that reproduces the bug. section.

[m-bert]

Hi @unendingblue! Thank you for spotting this issue! Could you confirm if #4026 fixes it?

[unendingblue]

hey @m-bert, that fixes it yes, thank you!

[unendingblue]

@m-bert quick question - the reason why i'm using activeOffsetY on android is because of the system's gesture logic when ScrollView is a child of a parent with a pan gesture (think of a global bottom sheet component where arbitrary views are placed inside). without activeOffsetY, finger gestures inside the ScrollView are detected by the PanGesture first, blocking the child ScrollView from scrolling. is there an easier way to fix this without passing useNativeGesture around?

related issue from v2: #3522

[m-bert]

Do you use ScrollView from React Native? If so, have you tried using ScrollView from Gesture Handler? Or is this combination also broken?

[unendingblue]

@m-bert i didn't think of trying that at all... it does seem to work correctly with the ScrollView the gesture handler exports. thank you!