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.

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

Adds react-native-windows support for UWP #337

Merged
merged 1 commit into from
Oct 5, 2017

Conversation

rozele
Copy link
Contributor

@rozele rozele commented Aug 24, 2017

This changeset includes most of the react-native-fs functionality working for UWP. Some notable missing features are:

  • all of the Assets APIs (e.g., readDirAssets, existsAssets, etc.) that are Android specific
  • the pathForBundle and pathForGroup APIs that are iOS specific
  • the uploadFiles API, which just has not yet been implemented.

This changeset includes most of the react-native-fs functionality working for UWP. Some notable missing features are:
* all of the `Assets` APIs (e.g., `readDirAssets`, `existsAssets`, etc.) that are Android specific
* the `pathForBundle` and `pathForGroup` APIs that are iOS specific
* the `uploadFiles` API, which just has not yet been implemented.
@rozele
Copy link
Contributor Author

rozele commented Oct 5, 2017

@itinance any chance we can get this merged?

@itinance
Copy link
Owner

itinance commented Oct 5, 2017

Hell yes, thank you :) My issue was that i haven't any windows system here to test it. Since it doesnt effect any existing code for iOS and Android i will merge now.

@itinance itinance merged commit 7f222be into itinance:master Oct 5, 2017
@itinance
Copy link
Owner

itinance commented Oct 5, 2017

@rozele Would you mind providing a section in README.md for UWP-support? Thank you!

@rozele
Copy link
Contributor Author

rozele commented Oct 5, 2017

@itinance of course! I'll try to write something up this week! Thanks for merging!

@itinance
Copy link
Owner

itinance commented Oct 5, 2017

Great. It would be part then of next release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants