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

Add FileSystemAdapter file adapter #1098

Merged
merged 1 commit into from
Mar 21, 2016

Conversation

dtsolis
Copy link
Contributor

@dtsolis dtsolis commented Mar 19, 2016

Adapter for saving files to the server's file system (previous PR https://github.com/ParsePlatform/parse-server/pull/716/files).

Can be used while starting the parse-server

var api = new ParseServer({
  databaseURI: 'mongodb://localhost:52853/dev',
  appId: 'appId',
  masterKey: "masterKey",
  serverURL: "http://localhost:1337/parse",
  filesAdapter: new FileSystemAdapter({
    filesSubDirectory: 'path/under/files/directory'
  })
});

@facebook-github-bot
Copy link

@dtsolis updated the pull request.

@codecov-io
Copy link

Current coverage is 91.26%

Merging #1098 into master will decrease coverage by -0.66% as of 7f00fce

@@            master   #1098   diff @@
======================================
  Files           91      92     +1
  Stmts         5736    5791    +55
  Branches      1043    1054    +11
  Methods          0       0       
======================================
+ Hit           5273    5285    +12
  Partial         11      11       
- Missed         452     495    +43

Review entire Coverage Diff as of 7f00fce

Powered by Codecov. Updated on successful CI builds.

@gfosco
Copy link
Contributor

gfosco commented Mar 21, 2016

@flovilmart this is okay with me, can you also take a look?

@flovilmart
Copy link
Contributor

That look good to me, and it passes the factory tests.

flovilmart added a commit that referenced this pull request Mar 21, 2016
@flovilmart flovilmart merged commit 134b6de into parse-community:master Mar 21, 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.

None yet

5 participants