asifpy / django-crudbuilder

Generic CRUD implementation in Django
https://django-crudbuilder.readthedocs.org/en/latest/index.html
Apache License 2.0
193 stars 67 forks source link

fixed cruds view app and model columns order using namedtuples #64

Closed hsum closed 6 years ago

hsum commented 6 years ago

I noticed the cruds index page had 'app' and 'model' columns swapped. This pull request fixes this. I found regular tuple indexing (0, 1, 2) hard to follow and namedtuples provide semantics making it an easy fix.

Please let me know if it looks good or anything needs further tweaking.

coveralls commented 6 years ago

Coverage Status

Coverage increased (+0.03%) to 79.346% when pulling c9ae939f9abeec99cc7611b2e55a56a371864980 on hsum:master into cc10abe93d3054104ac63fc264334b5896918659 on asifpy:master.

coveralls commented 6 years ago

Coverage Status

Coverage increased (+0.03%) to 79.346% when pulling c9ae939f9abeec99cc7611b2e55a56a371864980 on hsum:master into cc10abe93d3054104ac63fc264334b5896918659 on asifpy:master.