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

Added pickDirectory() implementation for Windows (proper PR) #444

Conversation

qmatteoq
Copy link
Collaborator

@qmatteoq qmatteoq commented Aug 2, 2021

Summary

This PR adds Windows support for the pickDirectory() function.

Test Plan

image

What's required for testing (prerequisites)?

See the official prerequisites for React Native for Windows here

What are the steps to reproduce (after prerequisites)?

Move to the root of the project and run:

yarn example windows

Compatibility

OS Implemented
iOS
Android
Windows

Checklist

  • [x ] 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)

@qmatteoq qmatteoq requested a review from vonovak August 2, 2021 21:15
@qmatteoq qmatteoq changed the title Added pickDirectory() implementation for Windows Added pickDirectory() implementation for Windows (proper PR) Aug 2, 2021
@vonovak vonovak merged commit 8af5f9e into react-native-documents:feat/use-bob-with-history Aug 2, 2021
@vonovak
Copy link
Member

vonovak commented Aug 2, 2021

thank you @qmatteoq! :)

vonovak added a commit that referenced this pull request Aug 9, 2021
* feat: rework module using react native bob

* chore: rebase

* Added windows example (not working for the moment)

* Fixed implementation of the Windows example

* Fixed Lint issues

* Fixed a missing Lint issue

* Fixed another missing Lint issue

* fix: fixed all the pending Lint issues

* Added pickDirectory() implementation for Windows (proper PR) (#444)

* feat: added pickDirectory implementation for Windows

* fix: fixed typo in property name

Co-authored-by: Matteo Pagani <matteo.pagani@microsoft.com>
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