Join link available to logged-in users

VERIFIED FIXED in 1.0

Status

Websites
creative.mozilla.org
--
major
VERIFIED FIXED
9 years ago
9 years ago

People

(Reporter: krupa, Assigned: ryansnyder)

Tracking

unspecified
x86
All

Details

(URL)

Attachments

(2 attachments)

(Reporter)

Description

9 years ago
Created attachment 392598 [details]
Join link

steps to reproduce:
1.Log in to your MCC account
2.In the "Thanks for Visiting" section,notice the messaging and the availability of Join link

expected behavior:
-Join link is not available for logged in users
-"Thanks for visiting" section is not suitable for logged-in users.
(Reporter)

Comment 1

9 years ago
Created attachment 392618 [details]
join link in About page

steps to reproduce:
1.Login
2.go to http://mcc.stage.mozilla.com/about

Notice the join us link
http://screencast.com/t/1iTLFH1vj5v

After clicking the link on the about page it shows this error message which is fine in my opinion
(Reporter)

Comment 3

9 years ago
Raymond,

Having an error message is good in such a situation.But providing the link to logged in users and compensating that error with an error message is not ideal.We should retain the error message for edge-case scenarios and remove the link for logged-in users.
Assignee: nobody → ryan
Removed Join link from homepage, slide 1 of the slideshow:

Sending        application/controllers/home.php
Sending        application/views/home/home_index.php
Sending        modules/auth/controllers/auth.php
Sending        modules/auth/i18n/en_US/auth.php
Transmitting file data ....
Committed revision 48819.

==

Removed Join link from the About us page.  Verified /register page redirects user and throws appropriate error if they are logged in.

Sending        application/controllers/about.php
Sending        application/views/about/about_faq.php
Sending        application/views/about/about_index.php
Transmitting file data ...
Committed revision 48820.
Status: NEW → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED

Comment 5

9 years ago
Join link removal looks good but per comment #1 "Thanks for visiting" section is not suitable for logged-in users." Do we want the whole "thanks for visiting" slide gone or just the link?
(Reporter)

Comment 6

9 years ago
reopening as per comment #5
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Per Tara:
I would say lets keep the slide, and the text... but change the title so that it's not a link when you're logged in.  
Slater? Jeff?

Per Jeff:
Yes, having the "join" link taken away when logged in makes sense to me, if that is possible. I think it's still ok to have a thanks for visiting tab there, even for users that are logged in

==

Made the title not be a link if the user is loggedin.

Sending        application/views/home/home_index.php
Transmitting file data .
Committed revision 48901.
Status: REOPENED → RESOLVED
Last Resolved: 9 years ago9 years ago
Resolution: --- → FIXED
(Reporter)

Comment 8

9 years ago
verified
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.