Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Add useAccessibilityInfo tests #264

Merged
merged 1 commit into from
Oct 18, 2021
Merged

Add useAccessibilityInfo tests #264

merged 1 commit into from
Oct 18, 2021

Conversation

retyui
Copy link
Contributor

@retyui retyui commented Oct 18, 2021

Summary

  • Add useAccessibilityInfo(...) tests

It was the last PR from hooks testing PRs. useCameraRoll and useClipboard hooks I skip for testing it because it uses deprecated API

Test Plan

yarn test

What's required for testing (prerequisites)?

What are the steps to reproduce (after prerequisites)?

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

LinusU
LinusU approved these changes Oct 18, 2021
View reviewed changes
Copy link
Member

@LinusU LinusU left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great 👍

@LinusU LinusU added the skip-release Preserve the current version when merged label Oct 18, 2021
@LinusU LinusU merged commit b4eb2c2 into react-native-community:master Oct 18, 2021
@retyui retyui deleted the add-use-accessibility-info-test branch October 18, 2021 14:52
@pvinis
Copy link
Member

pvinis commented Oct 18, 2021

🚀 PR was released in v2.7.0 🚀

@pvinis pvinis added the released This issue/pull request has been released. label Oct 18, 2021
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@LinusU LinusU LinusU approved these changes

Assignees
No one assigned
Labels
released This issue/pull request has been released. skip-release Preserve the current version when merged
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@retyui @pvinis @LinusU