diff --git a/app/models/user.rb b/app/models/user.rb index f64ddfa..cd72292 100644 --- a/app/models/user.rb +++ b/app/models/user.rb @@ -19,6 +19,8 @@ class User < ApplicationRecord validates :password, allow_nil: true, length: { minimum: 12 } # validates :password, not_pwned: { message: "might easily be guessed" } + validates_presence_of :firstname, :lastname + normalizes :email, with: -> { _1.strip.downcase } enum :role, { diff --git a/app/views/registrations/new.html.erb b/app/views/registrations/new.html.erb index a966be8..dde52d2 100644 --- a/app/views/registrations/new.html.erb +++ b/app/views/registrations/new.html.erb @@ -1,10 +1,8 @@

Sign up

- <%= form_with(url: sign_up_path) do |form| %> <% if @user.errors.any? %>

<%= pluralize(@user.errors.count, "error") %> prohibited this user from being saved:

-
<% end %> - +
+ <%= form.label :firstname, style: "display: block" %> + <%= form.text_field :firstname, value: @user.firstname, required: true, autofocus: true %> +
+
+ <%= form.label :lastname, style: "display: block" %> + <%= form.text_field :lastname, value: @user.lastname, required: true %> +
<%= form.label :email, style: "display: block" %> - <%= form.email_field :email, value: @user.email, required: true, autofocus: true, autocomplete: "email" %> + <%= form.email_field :email, value: @user.email, required: true, autocomplete: "email" %>
-
<%= form.label :password, style: "display: block" %> <%= form.password_field :password, required: true, autocomplete: "new-password" %>
12 characters minimum.
-
<%= form.label :password_confirmation, style: "display: block" %> <%= form.password_field :password_confirmation, required: true, autocomplete: "new-password" %>
-
<%= form.submit "Sign up" %>