Browse Source

Merge pull request #202 from communitiesuk/create-log-no-js

Don’t require JS to create new log
pull/205/head
Paul Robert Lloyd 3 years ago committed by GitHub
parent
commit
c7363a5500
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
  1. 2
      app/views/case_logs/index.html.erb
  2. 2
      spec/features/form/form_navigation_spec.rb

2
app/views/case_logs/index.html.erb

@ -6,7 +6,7 @@
<%= content_for(:title) %> <%= content_for(:title) %>
</h1> </h1>
<%= govuk_button_link_to "Create new log", case_logs_path, method: :post %> <%= govuk_button_to "Create new log", case_logs_path %>
<% if @in_progress_case_logs.present? %> <% if @in_progress_case_logs.present? %>
<%= render partial: "log_list", locals: { case_logs: @in_progress_case_logs, title: "Logs you need to complete", date_title: "Last Changed" } %> <%= render partial: "log_list", locals: { case_logs: @in_progress_case_logs, title: "Logs you need to complete", date_title: "Last Changed" } %>

2
spec/features/form/form_navigation_spec.rb

@ -31,7 +31,7 @@ RSpec.describe "Form Navigation" do
describe "Create new log" do describe "Create new log" do
it "redirects to the task list for the new log" do it "redirects to the task list for the new log" do
visit("/logs") visit("/logs")
click_link("Create new log") click_button("Create new log")
id = CaseLog.order(created_at: :desc).first.id id = CaseLog.order(created_at: :desc).first.id
expect(page).to have_content("Log #{id}") expect(page).to have_content("Log #{id}")
end end

Loading…
Cancel
Save