It's because we use the same form on the create and edit page, without the
check the errors will show on the initial rendering of the create page.
On May 26, 2014 8:04 PM, "Patrick Van Stee" <notifications@github.com>
wrote:

> In app/views/canned_responses/edit.html.erb:
>
> > @@ -1,6 +1,9 @@
> >  <%= render 'shared/navbar' %>
> >
> >  <div class="container">
> > +  <% if params.include?(:canned_response) && @canned_response.invalid? %>
>
> I think we might be able to remove the conditional here and just render
> the partial. Was there a specific case that
> params.include?(:canned_response) is used for? Couldn't figure that part
> out.
>
> Here's the check in the error_messages partial:
> https://github.com/asm-helpful/helpful-web/blob/master/app/views/shared/_error_messages.html.erb#L1
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/asm-helpful/helpful-web/pull/217/files#r13053936>
> .
>