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

Render function instead of render_to_response #111

Merged
merged 9 commits into from
Jun 10, 2016
Merged

Render function instead of render_to_response #111

merged 9 commits into from
Jun 10, 2016

Conversation

auvipy
Copy link
Contributor

@auvipy auvipy commented May 29, 2016

this is the preferred one

@avelis
Copy link
Collaborator

avelis commented May 31, 2016

@auvipy After reading the following stackoverflow question I am ok with merging this in. If you could resolve the failing unit tests In this PR that would be greatly appreciated.

@auvipy
Copy link
Contributor Author

auvipy commented May 31, 2016

I will try to fix the tests failing first then.

@avelis
Copy link
Collaborator

avelis commented May 31, 2016

@auvipy I appreciate the effort. I also understand that there might be a failing test not associated to this change. I can be reasonable and expect that test to not get addressed in this PR.

@auvipy
Copy link
Contributor Author

auvipy commented Jun 1, 2016

@avelis failed tests are a lot which take time to fix. so i would suggest to merge for now as is. i'm promising to comeback soon with more patches.

@avelis avelis merged commit 268eed8 into jazzband:master Jun 10, 2016
@avelis
Copy link
Collaborator

avelis commented Jun 10, 2016

@auvipy I have merged in this commit.

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.

2 participants