From 279093ad98d96106c60de9bfea3192c9088f0aec Mon Sep 17 00:00:00 2001
From: Manuel Bustillo <bustikiller@bustikiller.com>
Date: Sun, 1 Dec 2024 14:03:06 +0100
Subject: [PATCH] Configure registration endpoint to create a wedding as well

---
 app/controllers/application_controller.rb     |  4 ++++
 .../users/registrations_controller.rb         | 20 +++++++++++++++++++
 app/models/wedding.rb                         |  2 +-
 .../mailer/confirmation_instructions.html.erb |  5 +++++
 app/views/users/mailer/email_changed.html.erb |  7 +++++++
 .../users/mailer/password_change.html.erb     |  3 +++
 .../reset_password_instructions.html.erb      |  8 ++++++++
 .../users/mailer/unlock_instructions.html.erb |  7 +++++++
 config/initializers/devise.rb                 |  2 +-
 config/routes.rb                              |  2 +-
 spec/requests/users/registrations_spec.rb     |  7 +++++++
 11 files changed, 64 insertions(+), 3 deletions(-)
 create mode 100644 app/views/users/mailer/confirmation_instructions.html.erb
 create mode 100644 app/views/users/mailer/email_changed.html.erb
 create mode 100644 app/views/users/mailer/password_change.html.erb
 create mode 100644 app/views/users/mailer/reset_password_instructions.html.erb
 create mode 100644 app/views/users/mailer/unlock_instructions.html.erb

diff --git a/app/controllers/application_controller.rb b/app/controllers/application_controller.rb
index 13400e3..f163c3e 100644
--- a/app/controllers/application_controller.rb
+++ b/app/controllers/application_controller.rb
@@ -30,6 +30,10 @@ class ApplicationController < ActionController::Base
 
   private
 
+  def default_url_options(options = {})
+    options.merge(path_params: { slug: ActsAsTenant.current_tenant&.slug })
+  end
+
   def set_tenant
     ActsAsTenant.current_tenant = Wedding.find_by(slug: params[:slug])
   end
diff --git a/app/controllers/users/registrations_controller.rb b/app/controllers/users/registrations_controller.rb
index 09ffe33..24d8eca 100644
--- a/app/controllers/users/registrations_controller.rb
+++ b/app/controllers/users/registrations_controller.rb
@@ -3,4 +3,24 @@
 class Users::RegistrationsController < Devise::RegistrationsController
   clear_respond_to 
   respond_to :json
+
+  def create
+    wedding = Wedding.create(wedding_params)
+    unless wedding.persisted?
+      render json: { errors: wedding.errors.full_messages }, status: :unprocessable_entity
+      return
+    end
+
+    ActsAsTenant.with_tenant(wedding) do
+      super do |user|
+        wedding.destroy unless user.persisted?
+      end
+    end
+  end
+
+  private
+
+  def wedding_params
+    { slug: params[:slug], **params.expect(wedding: :date) }
+  end
 end
\ No newline at end of file
diff --git a/app/models/wedding.rb b/app/models/wedding.rb
index fa6c6e4..abc0999 100644
--- a/app/models/wedding.rb
+++ b/app/models/wedding.rb
@@ -16,5 +16,5 @@
 #
 class Wedding < ApplicationRecord
   validates :date, presence: true
-  validates :slug, presence: true, uniqueness: true
+  validates :slug, presence: true, uniqueness: true, format: { with: /\A[a-z]+\z/ }
 end
diff --git a/app/views/users/mailer/confirmation_instructions.html.erb b/app/views/users/mailer/confirmation_instructions.html.erb
new file mode 100644
index 0000000..7b71e44
--- /dev/null
+++ b/app/views/users/mailer/confirmation_instructions.html.erb
@@ -0,0 +1,5 @@
+<p>Welcome <%= @email %>!</p>
+
+<p>You can confirm your account email through the link below:</p>
+
+<p><%= link_to 'Confirm my account', confirmation_url(slug: ActsAsTenant.current_tenant&.slug, confirmation_token: @token) %></p>
diff --git a/app/views/users/mailer/email_changed.html.erb b/app/views/users/mailer/email_changed.html.erb
new file mode 100644
index 0000000..32f4ba8
--- /dev/null
+++ b/app/views/users/mailer/email_changed.html.erb
@@ -0,0 +1,7 @@
+<p>Hello <%= @email %>!</p>
+
+<% if @resource.try(:unconfirmed_email?) %>
+  <p>We're contacting you to notify you that your email is being changed to <%= @resource.unconfirmed_email %>.</p>
+<% else %>
+  <p>We're contacting you to notify you that your email has been changed to <%= @resource.email %>.</p>
+<% end %>
diff --git a/app/views/users/mailer/password_change.html.erb b/app/views/users/mailer/password_change.html.erb
new file mode 100644
index 0000000..b41daf4
--- /dev/null
+++ b/app/views/users/mailer/password_change.html.erb
@@ -0,0 +1,3 @@
+<p>Hello <%= @resource.email %>!</p>
+
+<p>We're contacting you to notify you that your password has been changed.</p>
diff --git a/app/views/users/mailer/reset_password_instructions.html.erb b/app/views/users/mailer/reset_password_instructions.html.erb
new file mode 100644
index 0000000..010a079
--- /dev/null
+++ b/app/views/users/mailer/reset_password_instructions.html.erb
@@ -0,0 +1,8 @@
+<p>Hello <%= @resource.email %>!</p>
+
+<p>Someone has requested a link to change your password. You can do this through the link below.</p>
+
+<p><%= link_to 'Change my password', edit_password_url(slug: ActsAsTenant.current_tenant&.slug, reset_password_token: @token) %></p>
+
+<p>If you didn't request this, please ignore this email.</p>
+<p>Your password won't change until you access the link above and create a new one.</p>
diff --git a/app/views/users/mailer/unlock_instructions.html.erb b/app/views/users/mailer/unlock_instructions.html.erb
new file mode 100644
index 0000000..313fe8e
--- /dev/null
+++ b/app/views/users/mailer/unlock_instructions.html.erb
@@ -0,0 +1,7 @@
+<p>Hello <%= @resource.email %>!</p>
+
+<p>Your account has been locked due to an excessive number of unsuccessful sign in attempts.</p>
+
+<p>Click the link below to unlock your account:</p>
+
+<p><%= link_to 'Unlock my account', unlock_url(slug: ActsAsTenant.current_tenant&.slug, unlock_token: @token) %></p>
diff --git a/config/initializers/devise.rb b/config/initializers/devise.rb
index 77d542f..5f631ef 100644
--- a/config/initializers/devise.rb
+++ b/config/initializers/devise.rb
@@ -246,7 +246,7 @@ Devise.setup do |config|
   # Turn scoped views on. Before rendering "sessions/new", it will first check for
   # "users/sessions/new". It's turned off by default because it's slower if you
   # are using only default views.
-  # config.scoped_views = false
+  config.scoped_views = true
 
   # Configure the default scope given to Warden. By default it's the first
   # devise role declared in your routes (usually :user).
diff --git a/config/routes.rb b/config/routes.rb
index b407e01..9659d05 100644
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -2,7 +2,7 @@
 
 Rails.application.routes.draw do
   mount LetterOpenerWeb::Engine, at: "/letter_opener" if Rails.env.development?
-  scope ":slug", constraints: {slug: /[a-z]+/} do
+  scope ":slug", constraints: { slug: /[a-z]+/ } do
     devise_for :users, skip: [:registration, :session, :confirmation]
     devise_scope :user do
       post 'users', to: 'users/registrations#create'
diff --git a/spec/requests/users/registrations_spec.rb b/spec/requests/users/registrations_spec.rb
index aa7562f..821db72 100644
--- a/spec/requests/users/registrations_spec.rb
+++ b/spec/requests/users/registrations_spec.rb
@@ -23,6 +23,13 @@ RSpec.describe 'users/registrations', type: :request do
               password: SwaggerResponseHelper::PASSWORD,
               password_confirmation: SwaggerResponseHelper::PASSWORD
             }
+          },
+          wedding: {
+            type: :object,
+            required: %i[date],
+            properties: {
+              date: { type: :string, format: :date},
+            }
           }
         }
       }