Opened 3 weeks ago
Closed 8 days ago
#37146 closed Cleanup/optimization (fixed)
README links to /newticket on Trac, which requires being logged in
| Reported by: | Margaret Fero, CodePath Instructor | Owned by: | Margaret Fero |
|---|---|---|---|
| Component: | Documentation | Version: | 6.0 |
| Severity: | Normal | Keywords: | |
| Cc: | Triage Stage: | Ready for checkin | |
| Has patch: | yes | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | yes | UI/UX: | no |
Description
One of the first things I saw in the Readme was the instruction "Docs are updated rigorously. If you find any problems in the docs, or think they should be clarified in any way, please take 30 seconds to fill out a ticket here: https://code.djangoproject.com/newticket". I clicked it just to see what was on the page.
However, coming in as a new person, that page displays a giant "Error: Forbidden" with two different warning banners on it, which would likely be discouraging if I weren't already an experienced contributor to other projects.
Adding that you can log in with GitHub (where you're being sent from) and that you must log in somehow would alleviate the "oh no, the page is yelling at me" feeling there. Feel free to assign to @MaggieFero (my primary account) if you'd like me to add such a mention.
Attachments (1)
Change History (8)
by , 3 weeks ago
| Attachment: | Django New Ticket.jpg added |
|---|
follow-ups: 3 4 comment:1 by , 2 weeks ago
| Owner: | set to |
|---|---|
| Status: | new → assigned |
| Triage Stage: | Unreviewed → Accepted |
Thanks, I think this was simply missed when implementing #26701.
We need to:
- remove
/newticketfrom the URL - consider adding "(login required)" to help users navigate that dense page, since we have yet to make the "New Ticket" button appear for logged-out users per this issue.
Thanks for offering a tweak.
comment:2 by , 2 weeks ago
| Summary: | New Contributor Instructions Mildly Discouraging → README links to /newticket on Trac, which requires being logged in |
|---|
comment:3 by , 2 weeks ago
You're welcome! Getting started on this now.
Replying to Jacob Walls:
Thanks, I think this was simply missed when implementing #26701.
We need to:
- remove
/newticketfrom the URL- consider adding "(login required)" to help users navigate that dense page, since we have yet to make the "New Ticket" button appear for logged-out users per this issue.
Thanks for offering a tweak.
comment:4 by , 2 weeks ago
| Has patch: | set |
|---|
[I can't delete this seemingly, but it was an accidental duplicate]
comment:5 by , 2 weeks ago
| Patch needs improvement: | set |
|---|
comment:6 by , 8 days ago
| Patch needs improvement: | unset |
|---|---|
| Triage Stage: | Accepted → Ready for checkin |
Screenshot of the Forbidden error with warning banners