Support for Sectionlist

[alexco2]

Hello, first of all, amazing package. This is something the rn community definetly needs! One question though. Do you plan to support Sectionlist, or is there a way to make it work with sectionlist?

[andreialecu]

The refactoring made as part of #93 should make this very easy to add.

[andreialecu]

This will actually not be easily possible to do on iOS because of issues described in this comment: #136 (comment)

[RoscoeZhao]

Hi @andreialecu , firstly I want to say thank you for this great component!
The PR which fixs the iOS offset issue, facebook/react-native#30737 , has been merged to RN. Would you please add support for SectionList ? Thank you very much!

[andreialecu]

@RoscoeZhao unfortunately even though the PR was merged, RN0.64 does not include it.

You can verify by comparing https://cdn.jsdelivr.net/npm/react-native@0.64.0/React/Views/RefreshControl/RCTRefreshControl.m with the changes in the PR.

The release workflow of RN diverges from what is in master. Instead, they cherry pick commits:
react-native-community/releases#221

RN 0.64.1 is not scheduled to have it either, as per the thread linked above.

[alexco2]

I asked for a cherrypick. If you give the comment some attention, perhaps It'll work ;)

[davidbiedenbach]

I guess 30737 finally made it into 0.65 =)

[alexco2]

@andreialecu since the PR is now included in RN 0.65, is there a plan to support sectionlist as well? :)

[andreialecu]

Sure, we can consider a PR that adds section list support if anyone is willing to implement it. 🙂

[alexco2]

Are there some considerations that come into your mind regarding the implementation of SectionList? @andreialecu

[alexco2]

@andreialecu I was able to make SectionList work. I will test it further and commit a PR if everything works fine. Also, I will look into bumping reanimated to version 2.2.2 for an additional PR. #164

[alexco2]

I commited the PR. Please let me know if anything is missing. Would be happy if the feature could be merged and shipped :)
#201