LinusU / LinusU/react-native-get-random-values
Add support for Windows
Nobody has claimed this yet.
- Dominant language
- JavaScript
- Stars
- 415
- Forks
- 63
- PR merge metrics
- No merged PRs in 30d
Description
With React Native for Windows becoming a viable target, enabling this package for Windows would be a great benefit to the community! (Especially if this package backs `uuid`, since a library I plan on using in RN uses it.)
I'm willing to take a stab at an initial implementation, and wanted to post this issue as a place to track progress.
Todo:
- [x] ~~Ensure RN for Windows supports synchronous methods~~ It does!
- [x] ~~Decide if C++ or C# would be the most appropriate~~ C# would be the most approachable, and would be just as performant as the C++ version since they're both just calling UWP APIs
- [x] ~~Determine if RN prefers Win32, UWP, or both for third party modules~~ RN for Windows only works on Windows 10, so there's no sense in using the Win32 APIs.
- [x] Setup project (blocked by https://github.com/microsoft/react-native-windows/issues/4211)
- [ ] Rewrite module in C++ (blocked by https://github.com/microsoft/react-native-windows/issues/4215)
Contributor guide
No contributing guide indexed for this repository
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
No files, tests, or entry points are named. Review the completed setup decisions and the linked react-native-windows blockers first; the issue is done when the module has a working C++ implementation supporting Windows.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp, csharp, react-native
- Domain
- desktop-dev, mobile-dev
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 25/100