react-native-get-random-values icon indicating copy to clipboard operation
react-native-get-random-values copied to clipboard

Published 20 hours ago • verified publisher icon LinusU

Add support for Windows

Open dstaley opened this issue 5 years ago • 3 comments

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)

dstaley avatar Feb 27 '20 05:02 dstaley

Is there any update to this? I think there has been a resolution in those referenced issues.

ajnozari avatar Feb 04 '21 19:02 ajnozari

Definitely adding a +1 to this. Any updates? The blocks on the final step appear to have been resolved.

alexanderblackh avatar Mar 31 '21 02:03 alexanderblackh

It would be great to see progress on this issue.

Does look like #15 does try to address this.

adamgorMSFT avatar Mar 17 '22 22:03 adamgorMSFT