Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#19243 closed Cleanup/optimization (fixed)

Edit GeoDjango Tutorial for consistency and style.

Reported by: shepdl Owned by: Tim Graham <timograham@…>
Component: Documentation Version: 1.4
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

There are a few grammatical problems with consistency and style in the GeoDjango Tutorial. Nothing factually wrong, but some changes would make it easier to read, I think. Also, there is at least one empty section ("Google.")

Change History (12)

comment:1 by shepdl, 11 years ago

Owner: changed from nobody to shepdl

comment:2 by Claude Paroz, 11 years ago

We welcome any improvements to the docs, but please then provide a patch with your suggestions. It is worth nothing having open tickets with too general statements like this. Do you plan to provide a patch soon?

comment:3 by Claude Paroz, 11 years ago

Resolution: needsinfo
Status: newclosed

Once again, we'd really appreciate any patch aimed to improve GeoDjango docs. I'm just setting this ticket as needinfo because the current description is too vague in its current state.

comment:4 by shepdl, 11 years ago

My apologies. I added this as part of something I was working on at a hackathon, had a bunch of trouble with Git (my own fault) so I couldn't commit the patch, and then went out of town for a week. I'll provide this soon.

comment:5 by anonymous, 11 years ago

Resolution: needsinfo
Status: closedreopened

comment:6 by shepdl, 11 years ago

Resolution: fixed
Status: reopenedclosed

comment:7 by Tim Graham, 11 years ago

Has patch: set
Resolution: fixed
Status: closedreopened

Thanks for this, however, don't close a ticket until a core developer commits it. You can put a link to your pull request in the ticket and check the "Has Patch" box so people can find it for review.

https://github.com/django/django/pull/531

comment:8 by Claude Paroz, 11 years ago

Owner: shepdl removed
Status: reopenednew
Triage Stage: UnreviewedAccepted

comment:9 by Claude Paroz, 11 years ago

Thanks, this looks very good (as seen by a non-native speaker). I'd still like a quick review by a native English speaker before the commit, if possible.

comment:10 by Tim Graham, 11 years ago

Left a few trivial comments, English looks fine to me.

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

Owner: set to Tim Graham <timograham@…>
Resolution: fixed
Status: newclosed

In 8e5573e99abd5351dfbd6de1214662ecf7f6c9ba:

Fixed #19243 - Edited GeoDjango Tutorial for consistency and style.

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

In 187441602c992fe8e97e32a7a53cc433cf06dd46:

[1.5.X] Fixed #19243 - Edited GeoDjango Tutorial for consistency and style.

Backport of 8e5573e99a from master

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