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

[FEATURE] [MER-4068] Reimplement user invite functionality #5322

Open
wants to merge 85 commits into
base: master
Choose a base branch
from

Conversation

nicocirio
Copy link
Contributor

Link to the ticket

As discussed in the ticket, this PR also introduced the :pending_confirmation and :rejected statuses for and enrollment, to allow the user to decide what to do with an invitation (in the previous version the user was enrolled and the email invitation ended up being an information email)

So, the student's overview page (in the instructor dashboard) was expanded to allow the instructor to filter by these two new values.
On the other hand, now the student workspace won't show the course until the student ends up accepting the invitation (in a future iteration we might want to show a pending confirmation course as "grayed" with a link to the UserInvite liveview)

A non-existing user invitation flow

Screen.Recording.2024-12-20.at.7.09.27.PM.mov

An existing user invitation flow (not logged in -> password must be provided)

Screen.Recording.2024-12-20.at.7.11.24.PM.mov

An existing user invitation flow (logged in with the same account as the invitation)

Since the user is already logged in, the user invitation is confirmed and the user gets redirected to the course as soon as it clicks the "Accept" button

Screen.Recording.2024-12-20.at.7.17.39.PM.mov

An existing user invitation flow (logged in with a different account than the invitation)

The user sees a warning and will be logged in with the account corresponding to the invitation

Screen.Recording.2024-12-20.at.7.24.57.PM.mov

Rejecting an invitation flow

Screen.Recording.2024-12-20.at.7.13.33.PM.mov

An already rejected invitation

image

An already accepted invitation

image

A non-existing invitation

image

Background color matches the invitation role

Student invitation

image

Instructor invitation

image

convert author accounts
remove Pow dependency
separate live_socket_id for user and author accounts
add auth provider links to other login pages
remove remaining references to pow
eliknebel and others added 30 commits December 9, 2024 10:10
…ontroller + create new route for that post action
…ted_by_id (in the future an invitation could be sent by an author)
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 this pull request may close these issues.

2 participants