Login-by-email feature and supporting infrastructure#23
Merged
tomast1337 merged 118 commits intodevelopfrom Mar 10, 2025
Merged
Login-by-email feature and supporting infrastructure#23tomast1337 merged 118 commits intodevelopfrom
tomast1337 merged 118 commits intodevelopfrom
Conversation
…nd add user registration logic
…oteBlockWorld into feature/login-by-email
…remove forRoot method
…e useFactory for configuration values
Member
Author
|
Just a reminder
|
… template inclusion
…pdating button styles
…mprove email context handling
Member
Author
…for improved rate limiting
…k for Axios errors
Member
|
Thanks for implementing these changes! The email template looks great now. I'll still review the frontend and likely do some small changes, though I'm only able to do that after about the 22nd this month. Let me know when you're done with your changes :) |
Member
Author
…nto feature/login-by-email
Member
|
This PR is waiting on us setting up an email server to be merged. :) |
Member
Author
I feel it would be good to merge and maybe disable the email server-related feature until it's available. |
…nto feature/login-by-email
Bentroen
added a commit
that referenced
this pull request
May 5, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.



This pull request implements email-based authentication and adds necessary supporting services and infrastructure.
Authentication & Email Features:
passport-magic-loginMagicLinkEmailStrategyfor email authenticationMailServicefor SMTP email handlingInfrastructure Updates:
EnvironmentVariableswith new configs and validationTechnical Improvements:
useFactoryandinject(Nest.js upgrade compatibility)Documentation Updates:
CONTRIBUTING.md: The setup instructions have been updated to usedocker-compose-dev.ymlinstead ofdocker-compose.yml. Additional environment variables and detailed setup instructions for both the backend and frontend have been added. New sections for running only the backend or frontend, populating the database with test data, and running tests have been included.