Skip to content
This repository has been archived by the owner on Jan 15, 2021. It is now read-only.

Fix localhost version of Mobile.GetMobileDocumentsPath #233

Closed
deadlyfingers opened this issue Oct 26, 2015 · 3 comments
Closed

Fix localhost version of Mobile.GetMobileDocumentsPath #233

deadlyfingers opened this issue Oct 26, 2015 · 3 comments
Assignees

Comments

@deadlyfingers
Copy link
Contributor

Postcard app is now generating string error with Mobile.GetMobileDocumentsPath in 'mockmobile.js'

TypeError: Arguments to path.join must be strings
@deadlyfingers deadlyfingers self-assigned this Oct 26, 2015
@vjrantal
Copy link
Member

For cross-platform, you could do something like:

var os = require('os'); // This to the top of the file
return os.tmpdir();

@yaronyg
Copy link
Member

yaronyg commented Oct 28, 2015

This is supposed to be resolved by #63

@deadlyfingers
Copy link
Contributor Author

@vjrantal - I submitted the update as a PR

deadlyfingers added a commit that referenced this issue Jan 19, 2016
Localhost GetDocumentsPath method update in mockmobile.js - fixes #233
@vjrantal vjrantal assigned vjrantal and unassigned deadlyfingers Jan 27, 2016
vjrantal added a commit that referenced this issue Jan 27, 2016
Revert "Localhost GetDocumentsPath method update in mockmobile.js - fixes #233"
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants