search

django-commons / drf-excel

An XLSX spreadsheet renderer for Django REST Framework.
BSD 3-Clause "New" or "Revised" License
217 stars 40 forks source link

Refactoring #67

Closed thesaintraphael closed 1 year ago

thesaintraphael commented 2 years ago

I simplified the code in some places and tried to make it more readable.

FlipperPA commented 2 years ago

@thesaintraphael Thanks, I've started reviewing this and it looks good but was to take a day or two since it touches a lot of edge cases. We really need to build a test suite!

FlipperPA commented 1 year ago

I'm going to merge this to main for testing, before we do #71 and #72.