Show 'action' links even if user is not logged in

Bug #624676 reported by Daniel Holbach on 2010-08-26
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
LoCo Team Portal
High
Daniel Holbach

Bug Description

Right now it's too hard to figure out how to

 - comment on an event
 - attend an event
 - join a team
 - add an event
 - <maybe others>

(People need to know they have to log in before even knowing what they could do on those pages.)

All these links should be shown if a user is logged in or not. Also do we need to make sure that the requires_login decorator is used for all those views.

Related branches

Changed in loco-directory:
importance: Undecided → High
status: New → Triaged
Dave Walker (davewalker) wrote :

I believe all actions require authenticated user are currently hidden if the user isn't logged in. I agree this is an accessibility issue.

Using the @requires_login decorator will initiate a login process, which the user may not want to do. I propose both slighter pale href colour if the user isn't authenticated and a JS popup area that says, "to perform this action you need to sign in" etc.

Penelope Stowe (pendulum) wrote :

Most accessibility software doesn't read javascript so a javascript pop-up won't be visible to users of screen readers and similar accessibility software.

Changed in loco-directory:
assignee: nobody → Daniel Holbach (dholbach)
status: Triaged → In Progress
Chris Johnston (cjohnston) wrote :

The "Add Comment" link does not redirect to login if the user is not logged in. Also need to show the 'Add Event' link on the teams page.

Changed in loco-directory:
status: In Progress → Fix Committed
milestone: none → 0.2.21
Chris Johnston (cjohnston) wrote :

We had talked about unhiding the link on the teams page as well. I have attached a merge for unhiding that link. The way that it works is if you are a member of the team, you are directed to the add event page, if you are not a team member, you get a message saying you are not a member of the team or on the LC.

Changed in loco-directory:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Duplicates of this bug

Other bug subscribers