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)

Django New Ticket.jpg (191.1 KB ) - added by Margaret Fero, CodePath Instructor 3 weeks ago.
Screenshot of the Forbidden error with warning banners

Download all attachments as: .zip

Change History (8)

by Margaret Fero, CodePath Instructor, 3 weeks ago

Attachment: Django New Ticket.jpg added

Screenshot of the Forbidden error with warning banners

comment:1 by Jacob Walls, 2 weeks ago

Owner: set to Margaret Fero
Status: newassigned
Triage Stage: UnreviewedAccepted

Thanks, I think this was simply missed when implementing #26701.

We need to:

  • remove /newticket from 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 Jacob Walls, 2 weeks ago

Summary: New Contributor Instructions Mildly DiscouragingREADME links to /newticket on Trac, which requires being logged in

in reply to:  1 comment:3 by Margaret Fero, 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 /newticket from 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.

in reply to:  1 comment:4 by Margaret Fero, 2 weeks ago

Has patch: set

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 /newticket from 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.

Version 0, edited 2 weeks ago by Margaret Fero (next)

comment:5 by Jacob Walls, 2 weeks ago

Patch needs improvement: set

comment:6 by Jacob Walls, 8 days ago

Patch needs improvement: unset
Triage Stage: AcceptedReady for checkin

comment:7 by Jacob Walls <jacobtylerwalls@…>, 8 days ago

Resolution: fixed
Status: assignedclosed

In aa0efc9e:

Fixed #37146 -- Improved new-user experience of README's Trac link.

Trac link in README was throwing a bunch of errors at newcomers.

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