Merge pull request #229 from differentreality/create_conf_indicator

remove readonly option for start_date and end_date of conference

app/controllers/admin/conference_controller.rb

@@ -67,9 +67,7 @@ class Admin::ConferenceController < ApplicationController
       redirect_to(admin_conference_path(id: @conference.short_title),
                   notice: 'Conference was successfully created.')
     else
-      redirect_to(new_admin_conference_path,
+      render action: 'new'
-                  alert: 'Creating the Conference failed.' \
-                          "#{@conference.errors.full_messages.join('. ')}.")
     end
   end
 

app/views/admin/conference/new.html.haml

@@ -11,7 +11,7 @@
         = f.input :contact_email, hint: 'Contact email address for your conference. Will be used as reply-to address in emails sent out by the system.'
       = f.inputs 'Scheduling' do
         = f.input :timezone, as: :time_zone, hint: 'Please select in what time zone your conference will take place.'
-        = f.input :start_date, as: :string, input_html: { id: 'conference-start-datepicker', readonly: 'readonly', required: 'required' }
+        = f.input :start_date, as: :string, input_html: { id: 'conference-start-datepicker', required: 'required' }
-        = f.input :end_date, as: :string, input_html: { id: 'conference-end-datepicker', readonly: 'readonly', required: 'required'  }
+        = f.input :end_date, as: :string, input_html: { id: 'conference-end-datepicker', required: 'required'  }
       = f.actions do
         = f.action :submit, button_html: {class: 'btn btn-success pull-right'}

spec/controllers/admin/conferences_controller_spec.rb

@@ -97,7 +97,7 @@ describe Admin::ConferenceController do
         it 're-renders the new template' do
           post :create, conference:
               attributes_for(:conference, short_title: nil)
-          expect(response).to redirect_to new_admin_conference_path
+          expect(response).to be_success
         end
       end
 
@@ -114,7 +114,7 @@ describe Admin::ConferenceController do
         it 're-renders the new template' do
           conference
           post :create, conference: attributes_for(:conference, short_title: conference.short_title)
-          expect(response).to redirect_to new_admin_conference_path
+          expect(response).to be_success
         end
       end
     end