Provide csv export option for folio_set view

DDMAL / cantus

:mag: Searching with Optical Music Recognition technology and the Cantus Database

http://cantus.simssa.ca

MIT License

15 stars 4 forks source link

Provide csv export option for folio_set view #784

Closed dchiller closed 1 year ago

dchiller commented 1 year ago

This PR:

adds CSVRenderer for use in Django Rest Framework responses.
implements the renderer at the existing /folio-set/manuscript/<manuscript-id> endpoint. Requests to this endpoint with a text/csv accept header will return a csv.

dchiller commented 1 year ago

Looking at the code, I see hints that this might happen (lines 13-14 in csv_renderer.py), but I don't really understand how this behavior is being effected. Is it a bit of Django REST Framework magic?

Yes, it is ... I had to learn more about how Django REST Framework does content negotiation, and it's one of those things where it's really nice that it works, but also really opaque. See additional comments on specific lines.