Allow me to log in/log out

VERIFIED FIXED in 2012-08-23

Status

Marketplace
Consumer Pages
P1
normal
VERIFIED FIXED
6 years ago
6 years ago

People

(Reporter: krupa, Assigned: spasovski)

Tracking

2012-08-23
Points:
---

Details

(Whiteboard: [janus], URL)

(Reporter)

Description

6 years ago
We should allow users to Log out. Settings is probably a nice place to have this fetaure.
(Reporter)

Updated

6 years ago
Whiteboard: [janus]
(Reporter)

Updated

6 years ago
Target Milestone: --- → 2012-08-23
(Reporter)

Comment 1

6 years ago
We should also allow users to log in.
Summary: Allow me to log out → Allow me to log in/log out

Updated

6 years ago
Assignee: nobody → dspasovski
(In reply to krupa raj 82[:krupa] from comment #0)
> We should allow users to Log out. Settings is probably a nice place to have
> this fetaure.

This will depend on how the details on identity and payments shake out. It might be the case that there will be no log in/log out. Happy to talk about it more if and when we decide to go that way.
(In reply to Maria Sandberg [:mushi] from comment #2)
> (In reply to krupa raj 82[:krupa] from comment #0)
> > We should allow users to Log out. Settings is probably a nice place to have
> > this fetaure.
> 
> This will depend on how the details on identity and payments shake out. It
> might be the case that there will be no log in/log out. Happy to talk about
> it more if and when we decide to go that way.

Users with extra privileges absolutely need to be able to log in and out.  (And by extension everyone)
(In reply to Andrew Williamson [:eviljeff] from comment #3)
> (In reply to Maria Sandberg [:mushi] from comment #2)
> > (In reply to krupa raj 82[:krupa] from comment #0)
> > > We should allow users to Log out. Settings is probably a nice place to have
> > > this fetaure.
> > 
> > This will depend on how the details on identity and payments shake out. It
> > might be the case that there will be no log in/log out. Happy to talk about
> > it more if and when we decide to go that way.
> 
> Users with extra privileges absolutely need to be able to log in and out. 
> (And by extension everyone)

Yes, we need this.  Putting under settings is fine for the short term.
The reason why I am being hesitant about this is that we don't know how sign in will work yet. (Unless there is new information from Dan Mills since last week that I haven't heard). 

We might end up in a situation where users on B2G sign in with their phone number and we do not have a password connected to it. In that case all a sign out / sign in button would do would be to toggle back and forth - not a very good experience and more confusing than useful. If you still want to go ahead and implement things for B2G before we know more, just be aware that these details are not yet finalized :)
Priority: -- → P1
(In reply to Maria Sandberg [:mushi] from comment #5)
> The reason why I am being hesitant about this is that we don't know how sign
> in will work yet. (Unless there is new information from Dan Mills since last
> week that I haven't heard). 

Authentication on desktop is still through Persona - and on B2G it's still through Persona for today (though that's of course changing). So yeah for now we still need people to log in for reviewing/submitting/approving (and hopefully purchasing soon!) apps.
(In reply to Chris Van Wiemeersch [:cvan] from comment #6)
> Authentication on desktop is still through Persona - and on B2G it's still
> through Persona for today (though that's of course changing). 

On non-B2G mobile (Android) its through Persona also.
(Assignee)

Comment 8

6 years ago
done: https://github.com/mozilla/zamboni/commit/da4b035
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
(Reporter)

Comment 9

6 years ago
Log in/Log out works, yay!

verified at https://marketplace-dev.allizom.org/settings
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.