Skip to content
This repository has been archived by the owner on Sep 10, 2024. It is now read-only.

Add GraphQL mutation to do self-service user registration #3050

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

reivilibre
Copy link
Contributor

Part of #172 — the registration form is being migrated to React, so the current server-side rendered registration page will be removed soon.

@reivilibre reivilibre requested a review from sandhose August 1, 2024 15:23
Copy link

cloudflare-workers-and-pages bot commented Aug 1, 2024

Deploying matrix-authentication-service-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: fdb4cad
Status: ✅  Deploy successful!
Preview URL: https://07819a51.matrix-authentication-service-docs.pages.dev
Branch Preview URL: https://rei-be-registration.matrix-authentication-service-docs.pages.dev

View logs

crates/handlers/src/graphql/mutations/user.rs Outdated Show resolved Hide resolved
crates/handlers/src/graphql/mutations/user.rs Outdated Show resolved Hide resolved
crates/handlers/src/graphql/mutations/user.rs Outdated Show resolved Hide resolved
crates/handlers/src/graphql/mutations/user.rs Show resolved Hide resolved
.record_browser_session(&clock, &session)
.await;

RegisterUserStatus::Allowed.into()
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure if possible, but this should set the session cookie, else the user won't be authenticated after this

Copy link
Contributor Author

@reivilibre reivilibre Aug 9, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not exactly proud of it, but have added a workaround for this now in fdb4cad

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.

2 participants