#7977: add geo support to admindocs
----------------------------------------------+-----------------------------
Reporter: [email protected] | Owner: jbronn
Status: assigned | Milestone:
Component: GIS | Version: gis
Resolution: | Keywords:
Stage: Accepted | Has_patch: 1
Needs_docs: 0 | Needs_tests: 0
Needs_better_patch: 1 |
----------------------------------------------+-----------------------------
Comment (by jcd):
A couple comments on my patch:
1. I'm not sure I like that I changed the docstring on OneToOneField to a
comment. It was the only field/relationship that had a docstring at all.
Alternatively, the patch could be modified so that
get_readable_field_data_type() could just take the first line of the
docstring, and strip off the rest. That might be unexpected behavior,
though.
2. admindocs didn't have any associated unit tests that I could find, so
I created some, but I only wrote tests for the function I modified. Some
tests are better than none, right?
--
Ticket URL: <http://code.djangoproject.com/ticket/7977#comment:10>
Django <http://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"Django updates" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to
[email protected]
For more options, visit this group at
http://groups.google.com/group/django-updates?hl=en
-~----------~----~----~----~------~----~------~--~---
