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.

Sign up for GitHub

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

Jump to bottom

Django integration utterly broken #31

Closed
FelixSchwarz opened this issue Oct 27, 2014 · 2 comments
Closed

Django integration utterly broken #31

FelixSchwarz opened this issue Oct 27, 2014 · 2 comments
Labels
bug patches welcome
Milestone
0.6.0

Comments

@FelixSchwarz
Copy link
Member

The Django integration contains bad imports and undefined variable references so it can't possibly work. I think the major regression was added in 0ba2d8a but there might have been errors before.

I assume we need a "unit" test for an extremly simple Django SOAP app and then fix all the issues which crept up.
@FelixSchwarz FelixSchwarz added this to the 0.6 milestone Oct 28, 2014
@FelixSchwarz
Copy link
Member Author

I think it is important to have a simple Django integration to hook up new users. Django is arguably the most popular Python web framework so it should be supported.

As I don't use soapfish with Django personally we need some unit tests so we don't break the example code accidentally.

@FelixSchwarz
Copy link
Member Author

fixed in edbb4a0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug patches welcome
Milestone
0.6.0
Development

No branches or pull requests
1 participant
@FelixSchwarz