-
-
Notifications
You must be signed in to change notification settings - Fork 5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: add ScrollView
type to 'getScrollResponder'
#11112
fix: add ScrollView
type to 'getScrollResponder'
#11112
Conversation
✅ Deploy Preview for react-navigation-example ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
@TomCorvus I recreated a pull request (the same diff). Could you try yarn add ken0nek/react-navigation.git#react-navigation-native-v6.1.1-gitpkg |
@satya164 I recreated #10775 as you suggested. Please review this when you have a chance. Thank you! ref: #10775 (comment) |
It seems to be fixed 👍 |
@TomCorvus Thank you for the update! 🤝 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the PR
Codecov ReportBase: 74.11% // Head: 74.11% // No change to project coverage 👍
📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more Additional details and impacted files@@ Coverage Diff @@
## main #11112 +/- ##
=======================================
Coverage 74.11% 74.11%
=======================================
Files 176 176
Lines 5602 5602
Branches 2197 2197
=======================================
Hits 4152 4152
Misses 1401 1401
Partials 49 49
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
Recreation of #10775 due to the git changes. **Motivation** Close #10722 `getScrollResponder` has a type error around `SectionList` due to missing type for `SectionList` ----- FlatList - https://github.com/DefinitelyTyped/DefinitelyTyped/blob/master/types/react-native/index.d.ts#L4057 ```ts getScrollResponder: () => JSX.Element | null | undefined; ``` SectionList - https://github.com/DefinitelyTyped/DefinitelyTyped/blob/master/types/react-native/index.d.ts#L4294 ```ts getScrollResponder(): ScrollView | undefined; ``` **Test plan** Use - `@types/react`: `^18.0.15` - `@react-navigation/native`: `ken0nek/react-navigation.git#react-navigation-native-v6.1.1-gitpkg` Check no errors for `useScrollToTop` with `SectionList`. ```ts const sectionList = useRef<SectionList>(null); useScrollToTop(sectionList); ``` Co-authored-by: Satyajit Sahoo <satyajit.happy@gmail.com>
Recreation of #10775 due to the git changes. **Motivation** Close #10722 `getScrollResponder` has a type error around `SectionList` due to missing type for `SectionList` ----- FlatList - https://github.com/DefinitelyTyped/DefinitelyTyped/blob/master/types/react-native/index.d.ts#L4057 ```ts getScrollResponder: () => JSX.Element | null | undefined; ``` SectionList - https://github.com/DefinitelyTyped/DefinitelyTyped/blob/master/types/react-native/index.d.ts#L4294 ```ts getScrollResponder(): ScrollView | undefined; ``` **Test plan** Use - `@types/react`: `^18.0.15` - `@react-navigation/native`: `ken0nek/react-navigation.git#react-navigation-native-v6.1.1-gitpkg` Check no errors for `useScrollToTop` with `SectionList`. ```ts const sectionList = useRef<SectionList>(null); useScrollToTop(sectionList); ``` Co-authored-by: Satyajit Sahoo <satyajit.happy@gmail.com>
Recreation of #10775 due to the git changes. **Motivation** Close #10722 `getScrollResponder` has a type error around `SectionList` due to missing type for `SectionList` ----- FlatList - https://github.com/DefinitelyTyped/DefinitelyTyped/blob/master/types/react-native/index.d.ts#L4057 ```ts getScrollResponder: () => JSX.Element | null | undefined; ``` SectionList - https://github.com/DefinitelyTyped/DefinitelyTyped/blob/master/types/react-native/index.d.ts#L4294 ```ts getScrollResponder(): ScrollView | undefined; ``` **Test plan** Use - `@types/react`: `^18.0.15` - `@react-navigation/native`: `ken0nek/react-navigation.git#react-navigation-native-v6.1.1-gitpkg` Check no errors for `useScrollToTop` with `SectionList`. ```ts const sectionList = useRef<SectionList>(null); useScrollToTop(sectionList); ``` Co-authored-by: Satyajit Sahoo <satyajit.happy@gmail.com>
Recreation of #10775 due to the git changes.
Motivation
Close #10722
getScrollResponder
has a type error aroundSectionList
due to missing type forSectionList
FlatList - https://github.com/DefinitelyTyped/DefinitelyTyped/blob/master/types/react-native/index.d.ts#L4057
SectionList - https://github.com/DefinitelyTyped/DefinitelyTyped/blob/master/types/react-native/index.d.ts#L4294
Test plan
Use
@types/react
:^18.0.15
@react-navigation/native
:ken0nek/react-navigation.git#react-navigation-native-v6.1.1-gitpkg
Check no errors for
useScrollToTop
withSectionList
.