-
Notifications
You must be signed in to change notification settings - Fork 45
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Filing a new bug when you're not logged in is *extremely* difficult #100
Comments
Hi @mlissner I remember this being non-obvious to me also. Skimmers gonna skim. We do have this link in the README, which has a blaring warning to sign in. In the wiki article at code.djangoproject.com we have:
Is your suggestion just to add a New Ticket link right after that paragraph? Could be solved with a wiki edit if so. |
Ah, there was a change to point folks away from /newticket and just to the homepage in django/django#6700, so maybe increasing redundancy is the answer. Put a more obvious "new ticket" link in the wiki article so that when we send you to the wiki, if you're a skimmer you (ideally) struggle less. The link could read New ticket (login required). |
I guess my solution would be to make the New Ticket link always appear. If you click it when you're logged out, you go to the login page with a redirect to the new ticket page. No idea if that's practical. |
The nav buttons come from Trac, so based on this (very old) comment I was able to get close. Haven't ported CSS, etc. I think the missing extra buttons could be because I haven't granted my anonymous user permissions for viewing the wiki. All to say -- before going further would be nice to get a reaction from somebody who's worked more with Trac. diff --git a/DjangoPlugin/tracdjangoplugin/__init__.py b/DjangoPlugin/tracdjangoplugin/__init__.py
index d4845e1..41d2010 100644
--- a/DjangoPlugin/tracdjangoplugin/__init__.py
+++ b/DjangoPlugin/tracdjangoplugin/__init__.py
@@ -52,6 +52,12 @@ class CustomNavigationBar(Component):
return ''
def get_navigation_items(self, req):
+ # Trac disables the New Ticket button in the main nav for logged out users, so create our own.
+ if 'TICKET_CREATE' not in req.perm:
+ return [
+ ('mainnav', 'new_ticket', tag.a('New Ticket', href=req.href.newticket())),
+ ('mainnav', 'custom_reports', Markup('<a href="%s">Reports</a>' % req.href.wiki('Reports'))),
+ ]
return [
('mainnav', 'custom_reports', Markup('<a href="%s">Reports</a>' % req.href.wiki('Reports'))),
] And at least that would take you to the current /newticket with a blaring warning to sign in. There are two login forms so I don't know about choosing one and adding a redirect. |
Fixes django#95, django#61 and django#60 Also possibly django#100 and django#64
Fixes django#95, django#61 and django#60 Also possibly django#100 and django#64
Fixes django#95, django#61 and django#60 Also possibly django#100 and django#64
Fixes django#95, django#61 and django#60 Also possibly django#100 and django#64
I just spent ten minutes trying to figure out where the "New Ticket" button is. Alas, it only shows up if you're logged in and I didn't realize I wasn't logged in. If you're logged out you'll look around forever and ever just hoping to figure it out.
There's a lot of documentation about how to file new bugs, but at a skim it just talks about things to do first, how to handle security bugs, etc. Where's the link‽ :)
If there's a way to add a "New Ticket" link that redirects you to the login page, that'd be very helpful.
The text was updated successfully, but these errors were encountered: