search

jpadilla / django-rest-framework-xml

XML support for Django REST Framework
http://jpadilla.github.io/django-rest-framework-xml
BSD 3-Clause "New" or "Revised" License
87 stars 59 forks source link

Can this use lxml? #6

Closed savingschampion closed 7 years ago

savingschampion commented 9 years ago

For large responses, using something like lxml to render the xml could cut rendering time.

jpadilla commented 9 years ago

@savingschampion I don't see why we couldn't use lxml if it was available falling back to default implementation. Want to tackle it and submit a PR?

kyleobrien91 commented 7 years ago

@savingschampion - if you find yourself inclined to contribute a PR for this, please reopen an issue with a spec at that point. Thanks!