Merge pull request #456 from commercekitchen/require_program_location…

…_name Require program location name
commercekitchen · Jun 12, 2020 · d8d4e50 · d8d4e50
2 parents 7aa4c28 + 568bee1
commit d8d4e50
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/app/views/admin/program_locations/create.js.erb b/app/views/admin/program_locations/create.js.erb
@@ -1,2 +1,4 @@
-$("#program_location_rows").append("<%= escape_javascript(render(partial: 'location_row', locals: { program_location: @program_location })) %>");
-$("#program_location_location_name").val('');
+<% if @program_location.valid? %>
+  $("#program_location_rows").append("<%= escape_javascript(render(partial: 'location_row', locals: { program_location: @program_location })) %>");
+  $("#program_location_location_name").val('');
+<% end %>
diff --git a/app/views/admin/programs/_program_locations_form.html.erb b/app/views/admin/programs/_program_locations_form.html.erb
@@ -2,7 +2,7 @@
 <div class='form-row'>
   <fieldset>
     <div class='inline-form'>
-      <%= f.text_field :location_name %>
+      <%= f.text_field :location_name, required: true %>
       <%= f.submit "Add", id: "addLocation" %>
       <%= f.hidden_field :program_id, :value => @program.id %>
     </div>