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

Fixes bug in readAsText() and adds readAsDataURL() in File plugin #346

Merged
merged 5 commits into from
Jul 22, 2016

Conversation

amuramoto
Copy link
Contributor

No description provided.

-DEVELOPER.md: Corrects 'npm run tslint' instruction to 'npm run lint'
-CameraPreview: Fixes syntax error in hide
-Base64ToGallery: Adds desc of options obj props
…e.readAsDataUrl()

- File.readAsText(): window.resolveLocalFileSystemURL() was incorrectly implemented
Callback arg of window.resolveLocalFileSystemURL was fileSystem, but it actually returns fileEntry
@ihadeed
Copy link
Collaborator

ihadeed commented Jul 21, 2016

We have a new file plugin in the PR section. #283

@ihadeed ihadeed closed this Jul 21, 2016
- Repo prop and link in desc were not pointing to the same gh repo as the plugin
- setFlashMode() is not currently available in the repo that gets pulled by the corodova plugin
@amuramoto
Copy link
Contributor Author

@ihadeed Would it not be advisable to merge so that the current version works properly while the new plugin is being finished?

@ihadeed
Copy link
Collaborator

ihadeed commented Jul 22, 2016

Sure, you're right.

@ihadeed ihadeed reopened this Jul 22, 2016
@ihadeed ihadeed merged commit 77d31cd into danielsogl:master Jul 22, 2016
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