Skip to content

Adds Hooks API on parse/node #243

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

Merged
merged 1 commit into from
Jun 30, 2016

Conversation

flovilmart
Copy link
Contributor

  • Adds hooks API to parse/node in perspective to remove the hooks API from experimental on parse-server
  • Will remove Parse.Hooks from parse-server once merged
  • Will keep E2E tests in parse-server to make sure the API is compliant
  • Tested on parse.com and create/update/remove works perfectly without any changes

@codecov-io
Copy link

Current coverage is 80.14%

Merging #243 into master will increase coverage by +0.13% as of 2bb5520

@@            master    #243   diff @@
======================================
  Files           40      41     +1
  Stmts         3272    3349    +77
  Branches       773     783    +10
  Methods          0       0       
======================================
+ Hit           2618    2684    +66
- Partial        224     232     +8
- Missed         430     433     +3

Review entire Coverage Diff as of 2bb5520

Powered by Codecov. Updated on successful CI builds.

@andrewimm
Copy link
Contributor

If we move to 1.9.0, I'd be happy to merge this in. I just want to take another look at it before we do so.

@flovilmart
Copy link
Contributor Author

That would be nice to remove it from parse-server. I'll update it when you release

@andrewimm andrewimm merged commit aabbfcd into parse-community:master Jun 30, 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.

4 participants