Show 'action' links even if user is not logged in
Bug #624676 reported by
Daniel Holbach
This bug affects 2 people
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| LoCo Team Portal |
Fix Released
|
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
lp://staging/~dholbach/loco-team-portal/624676
- Chris Johnston: Approve
-
Diff: 12 lines (+1/-1)1 file modifiedloco_directory/events/views.py (+1/-1)
| Changed in loco-directory: | |
| importance: | Undecided → High |
| status: | New → Triaged |
Revision history for this message
| Dave Walker (davewalker) wrote | #1 |
Revision history for this message
| Penelope Stowe (pendulum) wrote | #2 |
| Changed in loco-directory: | |
| assignee: | nobody → Daniel Holbach (dholbach) |
| status: | Triaged → In Progress |
Revision history for this message
| Chris Johnston (cjohnston) wrote | #3 |
| Changed in loco-directory: | |
| status: | In Progress → Fix Committed |
| milestone: | none → 0.2.21 |
Revision history for this message
| Chris Johnston (cjohnston) wrote | #4 |
| Changed in loco-directory: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
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.