Mozillians Authentication

RESOLVED FIXED

Status

--
major
RESOLVED FIXED
6 years ago
4 years ago

People

(Reporter: richard, Assigned: peterbe)

Tracking

Trunk
x86
Mac OS X
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(URL)

(Reporter)

Description

6 years ago
Changes required to allow Mozillians privileged access to Air Mozilla.
(Assignee)

Comment 1

6 years ago
1. Start working on it Nov 5. Talk to Ben and Giorgos
2. Change Persona button to "Sign-in" or something
3. Every event should have a dropdown instead of a checkbox:
  * Public, Company only, Company + Contributors
(Assignee)

Updated

6 years ago
Severity: normal → major
(Assignee)

Comment 3

6 years ago
The code is complete but for this to work we need to set up some keys on the dev server. 
I will file a new bug.
(Assignee)

Comment 4

6 years ago
Come to think of it, perhaps I should write a little something into the code so that this is allowed to be disabled. E.g. if no MOZILLIANS_API_KEY is set, instead of raising that as an error it can be used to simply mean that it can't use Mozillians to authenticate. 

However, I've already changed how the sign in button looks. ("Sign in" instead of "Employee Sign in")
(Assignee)

Updated

6 years ago
Depends on: 809289
(Assignee)

Comment 5

6 years ago
BTW, here's what it can look like: http://cl.ly/KhtK
Peter: 

Looks good! Great that you're the first to use the API ;)

I 'd go ahead and implement a graceful fallback in the case that the key doesn't exists or you get a "unauthorized" message, as you suggest. Obviously neither should happen but we better be safe than sorry.

Comment 7

6 years ago
Commit pushed to master at https://github.com/mozilla/airmozilla

https://github.com/mozilla/airmozilla/commit/62829ba254f9558a79a50275f3d9df598c0a9826
bug 805092 - full ability to log in as vouched contributor and new privacy field
(Assignee)

Comment 8

6 years ago
This is now complete and has been deployed on air-dev. 

Test away!
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
(Assignee)

Updated

4 years ago
Assignee: nobody → peterbe
You need to log in before you can comment on or make changes to this bug.