Skip to content

Commit b46823a

Browse files
Render form errors inline
The original guide uses a partial to render form errors, but it's easier to add this inline in our templates. Issues ------ - Closes #33
1 parent 945eb08 commit b46823a

File tree

2 files changed

+14
-2
lines changed

2 files changed

+14
-2
lines changed

lib/generators/rails_mvp_authentication/templates/views/users/edit.html.erb.tt

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,11 @@
11
<%%= form_with model: @user, url: account_path, method: :put do |form| %>
2-
<%%= render partial: "shared/form_errors", locals: { object: form.object } %>
2+
<%% if form.object.errors.any? %>
3+
<ul>
4+
<%% form.object.errors.full_messages.each do |message| %>
5+
<li><%%= message %></li>
6+
<%% end %>
7+
</ul>
8+
<%% end %>
39
<div>
410
<%%= form.label :email, "Current Email" %>
511
<%%= form.email_field :email, disabled: true %>

lib/generators/rails_mvp_authentication/templates/views/users/new.html.erb.tt

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,11 @@
11
<%%= form_with model: @user, url: sign_up_path do |form| %>
2-
<%%= render partial: "shared/form_errors", locals: { object: form.object } %>
2+
<%% if form.object.errors.any? %>
3+
<ul>
4+
<%% form.object.errors.full_messages.each do |message| %>
5+
<li><%%= message %></li>
6+
<%% end %>
7+
</ul>
8+
<%% end %>
39
<div>
410
<%%= form.label :email %>
511
<%%= form.email_field :email, required: true %>

0 commit comments

Comments
 (0)