microsoft / microsoft/react-native-windows
Remove FlatList*.windows.js RNTester Page Forks
Open
Nobody has claimed this yet.
Area: Fabric
Area: FlatList
Area: Tests
enhancement
Integration Follow-up
New Architecture
Workstream: Test Coverage
- Dominant language
- C++
- Stars
- 17.3k
- Forks
- 1.2k
- Avg merge
- 1d 13h
- Merged PRs (30d)
- 33
Description
I've forked several of the RNTester example pages for the FlatList component in order to add FlatList tests to the repo. The example pages originally did not have testID's and so we were unable to collect testing data on these pages. We can remove these forks once we upstream the testID addition.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Locate the FlatList*.windows.js forks among the RNTester example pages and compare them with the original FlatList pages. Once the testID addition has been upstreamed, remove the Windows forks and confirm the RNTester pages still provide the intended testing data.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- react-native
- Domain
- testing
- Issue type
- Refactor
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100