Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Stop setting an email address on tutorial accounts #164

Closed
ahelsing opened this issue Jul 18, 2016 · 3 comments
Closed

Stop setting an email address on tutorial accounts #164

ahelsing opened this issue Jul 18, 2016 · 3 comments
Assignees
Milestone

Comments

@ahelsing
Copy link
Member

Currently we create email accounts for tutorial accounts. These emails do not go to the experimenter.
Going forward, we won't be creating any such email accounts. So emails to these addresses would just bounce.

Instead, leave the email address blank. Then the experimenter can self-assert their own real email address when they get to the portal.

@ahelsing
Copy link
Member Author

Note that this requires:

  • ensure no geni-ar code dies if an account has no email address
  • clearinghouse database queries that try to ID tutorials by email address must do something different (use EPPN and/or affiliation)

Also note that these users, as before, cannot use our IdP to reset their IdP password or recover their username, as they have no email. (Before there was an email address they couldn't read.)

ahelsing added a commit to ahelsing/geni-ar that referenced this issue Jul 19, 2016
ahelsing added a commit to ahelsing/geni-ar that referenced this issue Jul 19, 2016
@ahelsing ahelsing added this to the 1.9 milestone Jul 20, 2016
ahelsing added a commit to ahelsing/geni-ar that referenced this issue Jul 21, 2016
…ts. Users will need to self-assert their address at the Portal.
ahelsing added a commit to ahelsing/geni-ar that referenced this issue Jul 22, 2016
ahelsing added a commit to ahelsing/geni-ar that referenced this issue Jul 22, 2016
ahelsing added a commit to ahelsing/geni-ar that referenced this issue Jul 22, 2016
ahelsing added a commit to ahelsing/geni-ar that referenced this issue Jul 22, 2016
@ahelsing ahelsing self-assigned this Jul 22, 2016
ahelsing added a commit to ahelsing/geni-ar that referenced this issue Jul 22, 2016
ahelsing added a commit to ahelsing/geni-ar that referenced this issue Jul 22, 2016
@ahelsing
Copy link
Member Author

Note this requires a schema change to allow the email address field to be null on account requests.

ahelsing added a commit to ahelsing/geni-ar that referenced this issue Jul 25, 2016
ahelsing added a commit to ahelsing/geni-ar that referenced this issue Jul 25, 2016
ahelsing added a commit to ahelsing/geni-ar that referenced this issue Jul 28, 2016
…nstructions on modifying attendee's profile
ahelsing added a commit to ahelsing/geni-ar that referenced this issue Jul 28, 2016
… that in the confirmation email, and the comment on creating the account.
@ahelsing
Copy link
Member Author

Will also include the name of the tutorial organizer in the email to the organizer (cc'ed to admins).

ahelsing added a commit to ahelsing/geni-ar that referenced this issue Jul 28, 2016
ahelsing added a commit to ahelsing/geni-ar that referenced this issue Jul 28, 2016
…e (non empty) and email (pass PHP FILTER_VALIDATE_EMAIL)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant