-
Notifications
You must be signed in to change notification settings - Fork 5
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
Invite a teacher #300
Milestone
Comments
Current questions:
|
Branch issues/300-Scope_invite_a_teacher/owner_flow created! |
I've started work adding the invitation of teachers via editor-api in #312 |
Closed in favour of this epic card. I've moved the relevant detail from this card to there. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
As an Owner
I want to invite teachers at the school level by email
So that educators at my school can start to use the classroom management features
Context
Now we know the roles are modelled in
editor-api
we'll need to handle the invite flow for roles associated with Raspberry Pi Accounts (i.e.school-owner
andschool-teacher
roles).Related work
Notes
Due to the disconnect between Pi Accounts and Editor API + the complication users being invited might not have an account there are several things that need to be considered, these include:
editor-api
for comparison when users login i.e. is there an outstanding invitation with the matching hash)design
PII concerns
An earlier version of this card suggested creating a technical implementation ADR in our internal docs here. Mostly this was motiviated by a concern that we may end up needing to store PII (e.g. an email address to send an invitation email to, and possibly other information from the invitation) and we wanted to be sure that we were making a concious decision about that.
We've decided to keep this issue in mind as we work on this story and when we get that far, to look at what, if any, problems we're creating and make sure we discuss them.
The text was updated successfully, but these errors were encountered: