Opened 17 years ago

Closed 17 years ago

#4987 closed (fixed)

DecimalField not documented

Reported by: James <james_027@…> Owned by: anonymous
Component: Documentation Version: dev
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: no UI/UX: no

Description

Hi,

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.

james

Attachments (1)

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

Download all attachments as: .zip

Change History (7)

comment:1 by Chris Beaven, 17 years ago

Summary: built in DecimalField documentationDecimalField not documented
Triage Stage: UnreviewedAccepted

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

comment:2 by James <james_027@…>, 17 years ago

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 by Chris Beaven, 17 years ago

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 by James <james_027@…>, 17 years ago

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

Hi,

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.

by James <james_027@…>, 17 years ago

Attachment: my_patch.diff added

comment:5 by Chris Beaven, 17 years ago

Patch needs improvement: unset
Triage Stage: AcceptedReady for checkin

Thanks James, looks good.

comment:6 by Malcolm Tredinnick, 17 years ago

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