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

ResourceApiBase#body_to_json overwrite #19

Open
lifeBCE opened this issue Jun 17, 2014 · 0 comments
Open

ResourceApiBase#body_to_json overwrite #19

lifeBCE opened this issue Jun 17, 2014 · 0 comments

Comments

@lifeBCE
Copy link

lifeBCE commented Jun 17, 2014

Seems ResourceApiBase class inherits from ApiBase class and both have a body_to_json() method. The methods do the same thing and therefore ResourceApiBase does not need its own body_to_json() method. Especially considering that the one in ResourceApiBase is susceptible to an unhandled exception condition.

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 a pull request may close this issue.

1 participant