Skip to content
This repository has been archived by the owner on Feb 8, 2018. It is now read-only.

Identity form #4034

Merged
merged 15 commits into from
May 12, 2016
Merged

Identity form #4034

merged 15 commits into from
May 12, 2016

Conversation

chadwhitacre
Copy link
Contributor

@chadwhitacre chadwhitacre commented May 11, 2016

Remake of #4008.

Punchlist

  • saving a verified identity should also be in the danger zone since it resets verification bumping to Step 4.ii.
  • consider preventing even admins from seeing identity info
  • write a test suite for %countries.spt

@chadwhitacre
Copy link
Contributor Author

consider preventing even admins from seeing identity info

@kaguillera and I talked about this a bit. When we go to expand takes beyond Gratipay we'll definitely want to think this through. For now the admins are more or less those who would likely be able to access the information on behalf of the Gratipay team anyway. Bumping.

@chadwhitacre
Copy link
Contributor Author

On third thought ...

@chadwhitacre
Copy link
Contributor Author

Is 8335b3b too ridiculous?

@chadwhitacre
Copy link
Contributor Author

Alright, ready.

@chadwhitacre chadwhitacre merged commit da4cace into master May 12, 2016
@chadwhitacre chadwhitacre deleted the identity-form branch December 2, 2016 21:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant