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

changed iOS recordings folder to app's Documents folder due to prepareToRecord failing and to follow Apple guidelines #17

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ncfx
Copy link

@ncfx ncfx commented Nov 18, 2015

…brary/NoCloud. This is for 2 reasons:

    1) as recommended by Apple, the Documents folder is used to store user generated content.
    2) [recorder prepareToRecord] fails because the Library/NoCloud directory does not exist by default

…brary/NoCloud. This is for 2 reasons:

        1) as recommended by Apple, the Documents folder is used to store user generated content.
        2) [recorder prepareToRecord] fails because the Library/NoCloud directory does not exist by default
@emj365
Copy link
Owner

emj365 commented Nov 30, 2015

http://ngcordova.com/docs/plugins/file/

I wish use path cordova.file.dataDirectory with $cordovaFile.

Any suggestion?

@thomasjonas
Copy link

Why isn't this pull request merged? The plugin isn't working otherwise and apparently it's also recommended by Apple...

@ghost
Copy link

ghost commented May 5, 2016

  1. This plugin creates user generated content, seems like it should belong there...
  2. This is true but if you include cordova-plugin-files, the file plugin will by default ensure the dataDirectory as well as several others exist. So maybe being more explicit on this is a good idea, but behavior on the whole seems appropriate.

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.

4 participants