Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#22809 closed Cleanup/optimization (fixed)

Refactor model field references

Reported by: jorgecarleitao Owned by: jorgecarleitao
Component: Documentation Version: dev
Severity: Normal Keywords:
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

At the moment, the API references of the model Field are scattered in docs/howto/custom-model-field.txt and docs/ref/models/fields.txt.

I propose to move all the references to docs/ref/models/fields.txt and keep the howto part of the documentation more focused on the subject, e.g. avoiding defining

.. method:: Field.get_prep_value(value)

there.

Change History (10)

comment:1 by Tim Graham, 10 years ago

Triage Stage: UnreviewedAccepted

Yes, probably at the bottom of the page in their own section sounds good.

comment:2 by jorgecarleitao, 10 years ago

Has patch: set

Pull request: https://github.com/django/django/pull/2796

Both sections of the docs still require a major editorial process on wording and references to models,
either to be addressed in this ticket or another.

comment:3 by Tim Graham, 10 years ago

Patch needs improvement: set

Please uncheck "Patch needs improvement" when you want me to review it.

comment:4 by jorgecarleitao, 10 years ago

Patch needs improvement: unset

comment:5 by Tim Graham, 10 years ago

Patch needs improvement: set

Reviewed.

comment:6 by jorgecarleitao, 10 years ago

Patch needs improvement: unset
Status: newassigned

comment:7 by Tim Graham, 10 years ago

Patch needs improvement: set

Still has some problems and doesn't merge cleanly.

comment:8 by jorgecarleitao, 10 years ago

Patch needs improvement: unset

Fixed the problems and rebased against master.

comment:9 by Tim Graham <timograham@…>, 10 years ago

Resolution: fixed
Status: assignedclosed

In e1fa7dffdc1b141cb858160890d448ca778366e4:

Fixed #22809 -- Added model Field API reference.

Thanks to @timgraham for the review.

comment:10 by Tim Graham <timograham@…>, 10 years ago

In bddea532f76dd59b54aede74fc7aac0d535b367b:

[1.7.x] Fixed #22809 -- Added model Field API reference.

Thanks to @timgraham for the review.

Backport of e1fa7dffdc from master

Note: See TracTickets for help on using tickets.
Back to Top