thinkingmachines / linksight-2018

LinkSight is a web app for applying the Philippine Standard Geographic Code to messy and misspelled barangay, municipality, city, and province names.
https://linksight.thinkingmachin.es
GNU General Public License v3.0
11 stars 1 forks source link

Change user account access to one step Google Sign-in #167

Closed piafaustino closed 6 years ago

piafaustino commented 6 years ago

Because LinkSight is still in alpha phase and needs active feedback, we need to limit access:

Log In or Sign Up

Approving Users

piafaustino commented 6 years ago

I got feedback from mgmt team confirming that current sign-up process is too tedious. Let's please implement instead the original treatment described above:

Changes:

piafaustino commented 6 years ago

Complexity: 1/4 of a day

piafaustino commented 6 years ago

See new wireframes for simplified log-in:

screen shot 2018-09-17 at 1 29 42 pm

If user's email is not authorized:

screen shot 2018-09-17 at 1 29 47 pm

@daniellegayares please provide @marksteve with link to Google Form

piafaustino commented 6 years ago

Reopening this issue until it's confirmed working on deployed front-end.

marksteve commented 6 years ago

@pepe @flo

Proposed execution:

  1. Just add to https://github.com/thinkingmachines/linksight/pull/225
  2. Update SOCIAL_AUTH_PIPELINE, replace social_core.pipeline.social_auth.auth_allowed with our own auth check (put in linksight.accounts.pipeline) that downloads the sheet from Drive and reads the email as a whitelist. You can use the compute engine service account for auth.
  3. Update UI to catch the auth error and show Bianca's error message
piafaustino commented 6 years ago

This works! Closing