Opened 9 years ago

Closed 9 years ago

#4987 closed (fixed)

DecimalField not documented

Reported by: James <james_027@…> Owned by: anonymous
Component: Documentation Version: master
Severity: Keywords: newforms DecimalField
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:



under the field type of generating forms for models there is the decimalField by for newforms, but it was not documented on the newforms built in types.


Attachments (1)

my_patch.diff (1019 bytes) - added by James <james_027@…> 9 years ago.

Download all attachments as: .zip

Change History (7)

comment:1 Changed 9 years ago by Chris Beaven

Needs documentation: unset
Needs tests: unset
Patch needs improvement: unset
Summary: built in DecimalField documentationDecimalField not documented
Triage Stage: UnreviewedAccepted

Thanks James. Perhaps you'd like to attach a patch? :)

comment:2 Changed 9 years ago by James <james_027@…>

Owner: changed from Jacob to anonymous
Status: newassigned

Sorry Chris,

I am very new to Django and python programming I might write it wrong.

comment:3 Changed 9 years ago by Chris Beaven

Don't let being new stop you writing patches, especially documentation ones.

Assuming you've done a full SVN checkout, just edit the txt file (copying the existing style you find in there) and then produce an svn diff. If you need some more help than that, visit the IRC channel - we're a helpful bunch. Alternatively, don't fret - someone else will probably fix it eventually. :)

comment:4 Changed 9 years ago by James <james_027@…>

Has patch: set
Keywords: newforms added
Patch needs improvement: set


This is my patch for the DecimalField of newforms with the help of looking into the source code itself. plus the optional arguments for the IntegerField.

Changed 9 years ago by James <james_027@…>

Attachment: my_patch.diff added

comment:5 Changed 9 years ago by Chris Beaven

Patch needs improvement: unset
Triage Stage: AcceptedReady for checkin

Thanks James, looks good.

comment:6 Changed 9 years ago by Malcolm Tredinnick

Resolution: fixed
Status: assignedclosed

(In [5855]) Fixed #4987 -- Added documentation for newforms.DecimalField. Based on a patch from james_027@….

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