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 some missing cozy.files.*() options #78

Merged
merged 1 commit into from
Feb 21, 2017
Merged

Add some missing cozy.files.*() options #78

merged 1 commit into from
Feb 21, 2017

Conversation

sebn
Copy link
Contributor

@sebn sebn commented Feb 17, 2017

  • Make lastModifiedDate option of updateById() official
    (only updated tests and doc since it already worked)
  • Add checksum option to create() and updateById()
  • Add ifMatch option to updateById() and updateAttributesBy*()
  • Fixed path parameter typo in updateAttributesById() doc

Needs cozy/cozy-stack#305

@sebn sebn requested review from nono and aenario February 17, 2017 14:01
@sebn
Copy link
Contributor Author

sebn commented Feb 20, 2017

Please wait before merging, I'm encountering some strange issues while using it with yarn link.
I'm currently updated some integration tests to make sure it actually works.

- Make `lastModifiedDate` option of `updateById()` official
  (only updated tests and doc since it already worked)
- Add `checksum` option to `create()` and `updateById()`
- Add `ifMatch` option to `updateById()` and `updateAttributesBy*()`
- Fixed `path` parameter typo in `updateAttributesById()` doc
@sebn
Copy link
Contributor Author

sebn commented Feb 21, 2017

Issues resolved on the cozy-desktop side 🚢

@nono
Copy link
Member

nono commented Feb 21, 2017

@aenario I let you review amd merge if OK this PR

@aenario aenario merged commit 03cb11f into master Feb 21, 2017
@aenario aenario deleted the files-options branch February 21, 2017 13:07
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.

3 participants