FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

IOS Known folders are not created if do not exist by zh-m · Pull Request #2699 · NativeScript/NativeScript · GitHub

IOS Known folders are not created if do not exist - #2699

Merged
zh-m merged 3 commits into
masterfrom
ios-folders
Sep 15, 2016
Merged

zh-m merged 3 commits into
masterfrom
ios-folders

Conversation

zh-m commented Sep 9, 2016

Copy link
Copy Markdown
Contributor

GetFolder method in file-system-access creates a new folder if the requested one does not exist. This action cannot be performed on IOS device for folders outside the scope of the app, consequently - the failing test on ios device (_testIOSSpecificKnownFolder).

ns-bot added the cla: yes label Sep 9, 2016

Copy link
Copy Markdown
Contributor

👍

Copy link
Copy Markdown
Contributor

👍 After tests pass

vchimev commented Sep 14, 2016

Copy link
Copy Markdown
Contributor

There is a typescript error:

Using the custom compiler : node_modules/typescript/bin/tsc
tns-core-modules/file-system/file-system-access.ios.ts(112,31): error TS2349: Cannot invoke an expression whose type lacks a call signature.

vchimev commented Sep 14, 2016

Copy link
Copy Markdown
Contributor

run ci

zh-m commented Sep 15, 2016

Copy link
Copy Markdown
Contributor Author

run ci

zh-m merged commit e4e5324 into master Sep 15, 2016
zh-m deleted the ios-folders branch September 15, 2016 15:38

lock Bot commented Aug 28, 2019

Copy link
Copy Markdown

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

lock Bot locked and limited conversation to collaborators Aug 28, 2019
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants


Back | FazBrowse Home | New Git URL