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

Makes sure the location in results has the proper objectId #1065

Merged
merged 1 commit into from
Mar 17, 2016

Conversation

flovilmart
Copy link
Contributor

No description provided.

@nlutsenko
Copy link
Contributor

A test would be great, but i think it's ok without it.

@flovilmart
Copy link
Contributor Author

Yeah I'll add one, good point

@codecov-io
Copy link

Current coverage is 91.64%

Merging #1065 into master will not affect coverage as of a0d9836

@@            master   #1065   diff @@
======================================
  Files           73      73       
  Stmts         4442    4442       
  Branches       889     889       
  Methods          0       0       
======================================
  Hit           4071    4071       
  Partial         10      10       
  Missed         361     361       

Review entire Coverage Diff as of a0d9836

Powered by Codecov. Updated on successful CI builds.

@flovilmart flovilmart force-pushed the flovilmart.hotfix-oauth-login branch from 59b5d07 to 461ddcb Compare March 17, 2016 00:38
@facebook-github-bot
Copy link

@flovilmart updated the pull request.

flovilmart added a commit that referenced this pull request Mar 17, 2016
…ogin

Makes sure the location in results has the proper objectId
@flovilmart flovilmart merged commit 7346ff2 into master Mar 17, 2016
@flovilmart flovilmart deleted the flovilmart.hotfix-oauth-login branch March 17, 2016 12:13
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