#19243 closed Cleanup/optimization (fixed)
Edit GeoDjango Tutorial for consistency and style.
Reported by: | shepdl | Owned by: | |
---|---|---|---|
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 , 12 years ago
Owner: | changed from | to
---|
comment:2 by , 12 years ago
comment:3 by , 12 years ago
Resolution: | → needsinfo |
---|---|
Status: | new → closed |
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 , 12 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 , 12 years ago
Resolution: | needsinfo |
---|---|
Status: | closed → reopened |
comment:6 by , 12 years ago
Resolution: | → fixed |
---|---|
Status: | reopened → closed |
comment:7 by , 12 years ago
Has patch: | set |
---|---|
Resolution: | fixed |
Status: | closed → reopened |
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.
comment:8 by , 12 years ago
Owner: | removed |
---|---|
Status: | reopened → new |
Triage Stage: | Unreviewed → Accepted |
comment:9 by , 12 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:11 by , 12 years ago
Owner: | set to |
---|---|
Resolution: | → fixed |
Status: | new → closed |
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?