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

feat: verify signup #39

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

feat: verify signup #39

wants to merge 16 commits into from

Conversation

scmmishra
Copy link
Owner

@scmmishra scmmishra commented Jan 24, 2025

This pull request introduces significant updates to user verification, email handling, gem configuration, and database structure. Here's a summary:

Key Changes:

User Verification and Email Handling:

  • User Model: Added methods for sending verification emails, handling tokens, and verifying users.
  • UsersController: Updated to include actions for verification (e.g., showing verification page, resending emails, verifying users).
  • UserMailer: Created for sending verification emails, with corresponding HTML and text views.

Gem Configuration:

  • Replaced the annotate gem with annotaterb and updated related configuration files and tasks.

Database Changes:

  • Removed the active column and added a blocked_at column in the users table to support user blocking functionality.

Controller and View Updates:

  • Enhanced ApplicationController to include user verification checks in redirection logic.
  • Added a new view to guide users through the email verification process.

Copy link

deepsource-io bot commented Jan 24, 2025

Here's the code health analysis summary for commits f816ba4..aa3c154. View details on DeepSource ↗.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource JavaScript LogoJavaScript✅ SuccessView Check ↗
DeepSource Ruby LogoRuby✅ SuccessView Check ↗

💡 If you’re a repository administrator, you can configure the quality gates from the settings.

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.

1 participant