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

Invite a teacher #300

Closed
sra405 opened this issue May 29, 2024 · 4 comments · May be fixed by #304
Closed

Invite a teacher #300

sra405 opened this issue May 29, 2024 · 4 comments · May be fixed by #304
Assignees

Comments

@sra405
Copy link
Contributor

sra405 commented May 29, 2024

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 and school-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:

  • Mechanisms for inviting teachers/owners via the frontend (see design)
  • Storing the email/role (it was suggested we hash the email to avoid raw PII in editor-api for comparison when users login i.e. is there an outstanding invitation with the matching hash)
  • Transactional emails to let them know they have an invited (design)
  • Accepting invites (initial profile design)
  • Retrieving the initial invite to set the role (unless we store the original invite we won't know at this stage what role to apply) - design

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.

@sra405 sra405 added this to the Classroom Management MVP milestone May 29, 2024
@sra405
Copy link
Contributor Author

sra405 commented Jun 3, 2024

Current questions:

  • we'd need to queue these transactional emails so emails will need to be stored in the backend
  • how long will these role invitations last?
    • expire invites and delete PII (e.g. invite only lasts 30 days)

@sra405 sra405 removed the sprint goal label Jun 3, 2024
@chrislo chrislo self-assigned this Jun 3, 2024
Copy link
Contributor

Branch issues/300-Scope_invite_a_teacher/owner_flow created!

create-issue-branch bot added a commit that referenced this issue Jun 3, 2024
@chrislo chrislo changed the title Scope invite a teacher/owner flow Invite a teacher/owner Jun 3, 2024
@chrislo
Copy link
Contributor

chrislo commented Jun 5, 2024

I've started work adding the invitation of teachers via editor-api in #312

@MFarringtonRPF MFarringtonRPF changed the title Invite a teacher/owner Invite a teacher Jun 11, 2024
@chrislo
Copy link
Contributor

chrislo commented Jun 12, 2024

Closed in favour of this epic card. I've moved the relevant detail from this card to there.

@chrislo chrislo closed this as completed Jun 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants