Skip to content
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.

Sign up for GitHub

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

Jump to bottom

feat: add useRefresh hook #320

Merged
merged 7 commits into from
Feb 1, 2023
Merged

feat: add useRefresh hook #320

merged 7 commits into from
Feb 1, 2023

Conversation

kanelloc
Copy link
Contributor

@kanelloc kanelloc commented Jan 14, 2023 •
edited
Loading

Summary

  • Added useRefresh hook.
  • It can be used with <Flatlist/>, <SectionList/>, <VirtualizedList/> and <ScrollView/>

Test Plan

What's required for testing (prerequisites)?

yarn
yarn test

What are the steps to reproduce (after prerequisites)?

Something like what is shown in the readme.

Compatibility

OS Implemented
iOS ✅
Android ✅
Web ❌

Checklist

  • I have tested this on a device and a simulator
  • I added the documentation in README.md
  • I updated the typed files (TS and Flow)
  • I've created a snack to demonstrate the changes: LINK HERE

@kanelloc
Copy link
Contributor Author

Hey @LinusU @pvinis 👋 !!
What do you think about this useRefresh hook?

@pvinis
Copy link
Member

pvinis commented Jan 31, 2023

nice! I can take a look later today, if that's alright 👍

@pvinis pvinis added minor Increment the minor version when merged prerelease This change is available in a prerelease. release Create a release when this pr is merged labels Feb 1, 2023
@pvinis pvinis merged commit b060899 into react-native-community:main Feb 1, 2023
@github-actions
Copy link

github-actions bot commented Feb 1, 2023

🚀 PR was released in v3.0.0 🚀

@github-actions github-actions bot added released This issue/pull request has been released. and removed prerelease This change is available in a prerelease. labels Feb 1, 2023
@pvinis
Copy link
Member

pvinis commented Feb 1, 2023

try it, and let me know if its alright!

@kanelloc kanelloc mentioned this pull request Feb 1, 2023
Merged
13 tasks
@kanelloc
Copy link
Contributor Author

kanelloc commented Feb 1, 2023

try it, and let me know if its alright!

Tested it and works as expected 🎉

@kanelloc kanelloc deleted the feat/useRefresh_hook branch February 1, 2023 08:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pvinis pvinis pvinis approved these changes

Assignees
No one assigned
Labels
minor Increment the minor version when merged release Create a release when this pr is merged released This issue/pull request has been released.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@kanelloc @pvinis