From 45a13428f01a4db058049b740c31ae8daf8d4ed3 Mon Sep 17 00:00:00 2001
From: kosiakkatrina <54268893+kosiakkatrina@users.noreply.github.com>
Date: Fri, 4 Feb 2022 12:41:47 +0000
Subject: [PATCH 1/7] CLDC-967: count inferred check answers answer as
 completed (#268)

* Count inferred check answers answer as an answered answer :)
and remove la unknown reason field

* rename the method and add a test for gdpr completed section

* remove commented code
---
 app/models/form/question.rb                   |  6 ++-
 config/forms/2021_2022.json                   |  9 -----
 ...20203145845_remove_why_dont_you_know_la.rb |  9 +++++
 db/schema.rb                                  |  5 +--
 spec/features/form/check_answers_page_spec.rb |  2 +-
 spec/features/form/tasklist_page_spec.rb      |  4 +-
 spec/fixtures/complete_case_log.json          |  1 -
 spec/fixtures/forms/2021_2022.json            | 38 +++++++++++++------
 spec/helpers/tasklist_helper_spec.rb          |  4 +-
 spec/models/form/question_spec.rb             | 26 +++++++++++++
 spec/models/form_spec.rb                      |  9 ++++-
 spec/requests/case_logs_controller_spec.rb    |  4 +-
 12 files changed, 82 insertions(+), 35 deletions(-)
 create mode 100644 db/migrate/20220203145845_remove_why_dont_you_know_la.rb

diff --git a/app/models/form/question.rb b/app/models/form/question.rb
index 1ed53b2de..5282bae71 100644
--- a/app/models/form/question.rb
+++ b/app/models/form/question.rb
@@ -83,11 +83,15 @@ class Form::Question
     return false if id == "gdpr_acceptance" && case_log[id] == "No"
     return answer_options.keys.any? { |key| case_log[key] == "Yes" } if type == "checkbox"
 
-    case_log[id].present? || !case_log.respond_to?(id.to_sym)
+    case_log[id].present? || !case_log.respond_to?(id.to_sym) || has_inferred_display_value?(case_log)
   end
 
 private
 
+  def has_inferred_display_value?(case_log)
+    inferred_check_answers_value.present? && case_log[inferred_check_answers_value["condition"].keys.first] == inferred_check_answers_value["condition"].values.first
+  end
+
   def checkbox_answer_label(case_log)
     answer = []
     answer_options.each { |key, value| case_log[key] == "Yes" ? answer << value : nil }
diff --git a/config/forms/2021_2022.json b/config/forms/2021_2022.json
index 2c7ce0eb5..7ebdfdfef 100644
--- a/config/forms/2021_2022.json
+++ b/config/forms/2021_2022.json
@@ -1716,16 +1716,7 @@
                   "answer_options": {
                     "1": "Yes",
                     "0": "No"
-                  },
-                  "conditional_for": {
-                    "why_dont_you_know_la": ["No"]
                   }
-                },
-                "why_dont_you_know_la": {
-                  "check_answer_label": "Reason you don’t know the postcode or local authority",
-                  "header": "Give a reason why you don’t know the postcode or local authority",
-                  "hint_text": "",
-                  "type": "textarea"
                 }
               },
               "depends_on": [{ "is_la_inferred": false }]
diff --git a/db/migrate/20220203145845_remove_why_dont_you_know_la.rb b/db/migrate/20220203145845_remove_why_dont_you_know_la.rb
new file mode 100644
index 000000000..6da34c0a7
--- /dev/null
+++ b/db/migrate/20220203145845_remove_why_dont_you_know_la.rb
@@ -0,0 +1,9 @@
+class RemoveWhyDontYouKnowLa < ActiveRecord::Migration[7.0]
+  def up
+    remove_column :case_logs, :why_dont_you_know_la
+  end
+
+  def down
+    add_column :case_logs, :why_dont_you_know_la, :string
+  end
+end
diff --git a/db/schema.rb b/db/schema.rb
index b3f0fca4d..46cf0417d 100644
--- a/db/schema.rb
+++ b/db/schema.rb
@@ -10,7 +10,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 2022_02_03_104800) do
+ActiveRecord::Schema.define(version: 2022_02_03_145845) do
 
   # These are extensions that must be enabled in order to support this database
   enable_extension "plpgsql"
@@ -93,7 +93,6 @@ ActiveRecord::Schema.define(version: 2022_02_03_104800) do
     t.integer "beds"
     t.integer "offered"
     t.integer "wchair"
-    t.integer "earnings"
     t.integer "incfreq"
     t.integer "benefits"
     t.integer "period"
@@ -162,7 +161,6 @@ ActiveRecord::Schema.define(version: 2022_02_03_104800) do
     t.datetime "startdate"
     t.integer "armedforces"
     t.integer "first_time_property_let_as_social_housing"
-    t.string "why_dont_you_know_la"
     t.integer "unitletas"
     t.integer "builtype"
     t.datetime "property_void_date"
@@ -186,6 +184,7 @@ ActiveRecord::Schema.define(version: 2022_02_03_104800) do
     t.integer "is_carehome"
     t.integer "letting_in_sheltered_accomodation"
     t.integer "household_charge"
+    t.integer "earnings"
     t.integer "referral"
     t.decimal "brent", precision: 10, scale: 2
     t.decimal "scharge", precision: 10, scale: 2
diff --git a/spec/features/form/check_answers_page_spec.rb b/spec/features/form/check_answers_page_spec.rb
index cde579bf4..c51ddfa48 100644
--- a/spec/features/form/check_answers_page_spec.rb
+++ b/spec/features/form/check_answers_page_spec.rb
@@ -209,7 +209,7 @@ RSpec.describe "Form Check Answers Page" do
       end
 
       it "they can click a button to cycle around to the next incomplete section" do
-        visit("/logs/#{cycle_sections_case_log.id}/local-authority/check-answers")
+        visit("/logs/#{cycle_sections_case_log.id}/setup/check-answers")
         click_link("Save and go to next incomplete section")
         expect(page).to have_current_path("/logs/#{cycle_sections_case_log.id}/tenant-code")
       end
diff --git a/spec/features/form/tasklist_page_spec.rb b/spec/features/form/tasklist_page_spec.rb
index ba731e05e..12a6afd6f 100644
--- a/spec/features/form/tasklist_page_spec.rb
+++ b/spec/features/form/tasklist_page_spec.rb
@@ -33,12 +33,12 @@ RSpec.describe "Task List" do
 
   it "shows the number of completed sections if no sections are completed" do
     visit("/logs/#{empty_case_log.id}")
-    expect(page).to have_content("You have completed 0 of 9 sections.")
+    expect(page).to have_content("You have completed 0 of 10 sections.")
   end
 
   it "shows the number of completed sections if one section is completed" do
     answer_all_questions_in_income_subsection(empty_case_log)
     visit("/logs/#{empty_case_log.id}")
-    expect(page).to have_content("You have completed 1 of 9 sections.")
+    expect(page).to have_content("You have completed 1 of 10 sections.")
   end
 end
diff --git a/spec/fixtures/complete_case_log.json b/spec/fixtures/complete_case_log.json
index eb5d70eb9..577523dc9 100644
--- a/spec/fixtures/complete_case_log.json
+++ b/spec/fixtures/complete_case_log.json
@@ -136,7 +136,6 @@
     "postcod2": "w3",
     "ppostc1": "w3",
     "ppostc2": "w3",
-    "why_dont_you_know_la": "Forgot",
     "first_time_property_let_as_social_housing": "No",
     "unitletas": "Affordable rent basis",
     "builtype": "Purpose built",
diff --git a/spec/fixtures/forms/2021_2022.json b/spec/fixtures/forms/2021_2022.json
index dac2b94d3..3a322369e 100644
--- a/spec/fixtures/forms/2021_2022.json
+++ b/spec/fixtures/forms/2021_2022.json
@@ -593,18 +593,6 @@
                   }
                 }
               },
-              "why_dont_you_know_la": {
-                "header": "",
-                "description": "",
-                "questions": {
-                  "reason": {
-                    "check_answer_label": "Reason for not knowing local authority",
-                    "header": "Give a reason why you don’t know the postcode or local authority",
-                    "hint_text": "",
-                    "type": "textarea"
-                  }
-                }
-              },
               "property_postcode": {
                 "questions": {
                   "property_postcode": {
@@ -638,6 +626,32 @@
           }
         }
       },
+      "setup": {
+        "label": "Before you start",
+        "subsections": {
+          "setup": {
+            "label": "Set up your lettings log",
+            "pages": {
+              "gdpr_acceptance": {
+                "header": "",
+                "description": "",
+                "questions": {
+                  "gdpr_acceptance": {
+                    "check_answer_label": "Privacy notice seen",
+                    "header": "Has the tenant or buyer seen the Department for Levelling Up, Housing and Communities (DLUHC) privacy notice?",
+                    "hint_text": "You must <a class=\"govuk-link\" href=\"/files/privacy-notice.pdf\">show the privacy notice</a> to the tenant or buyer before you can use this service.",
+                    "type": "radio",
+                    "answer_options": {
+                      "0": "Yes",
+                      "1": "No"
+                    }
+                  }
+                }
+              }
+            }
+          }
+        }
+      },
       "submission": {
         "label": "Submission",
         "subsections": {
diff --git a/spec/helpers/tasklist_helper_spec.rb b/spec/helpers/tasklist_helper_spec.rb
index 18b436a8a..578edf7c6 100644
--- a/spec/helpers/tasklist_helper_spec.rb
+++ b/spec/helpers/tasklist_helper_spec.rb
@@ -22,7 +22,7 @@ RSpec.describe TasklistHelper do
 
   describe "get sections count" do
     it "returns the total of sections if no status is given" do
-      expect(get_subsections_count(empty_case_log)).to eq(9)
+      expect(get_subsections_count(empty_case_log)).to eq(10)
     end
 
     it "returns 0 sections for completed sections if no sections are completed" do
@@ -30,7 +30,7 @@ RSpec.describe TasklistHelper do
     end
 
     it "returns the number of not started sections" do
-      expect(get_subsections_count(empty_case_log, :not_started)).to eq(8)
+      expect(get_subsections_count(empty_case_log, :not_started)).to eq(9)
     end
 
     it "returns the number of sections in progress" do
diff --git a/spec/models/form/question_spec.rb b/spec/models/form/question_spec.rb
index d3062a7f0..2d166b73b 100644
--- a/spec/models/form/question_spec.rb
+++ b/spec/models/form/question_spec.rb
@@ -156,4 +156,30 @@ RSpec.describe Form::Question, type: :model do
       end
     end
   end
+
+  describe ".completed?" do
+    context "when the question has inferred value only for check answers display" do
+      let(:section_id) { "tenancy_and_property" }
+      let(:subsection_id) { "property_information" }
+      let(:page_id) { "property_postcode" }
+      let(:question_id) { "property_postcode" }
+
+      it "returns true" do
+        case_log["postcode_known"] = "No"
+        expect(question.completed?(case_log)).to be(true)
+      end
+    end
+
+    context "when the gdpr acceptance is No" do
+      let(:section_id) { "setup" }
+      let(:subsection_id) { "setup" }
+      let(:page_id) { "gdpr_acceptance" }
+      let(:question_id) { "gdpr_acceptance" }
+
+      it "returns false" do
+        case_log["gdpr_acceptance"] = "No"
+        expect(question.completed?(case_log)).to be(false)
+      end
+    end
+  end
 end
diff --git a/spec/models/form_spec.rb b/spec/models/form_spec.rb
index 67269e78e..4ab1b6ada 100644
--- a/spec/models/form_spec.rb
+++ b/spec/models/form_spec.rb
@@ -36,7 +36,7 @@ RSpec.describe Form, type: :model do
   describe "next_incomplete_section_redirect_path" do
     let(:case_log) { FactoryBot.build(:case_log, :in_progress) }
     let(:subsection) { form.get_subsection("household_characteristics") }
-    let(:later_subsection) { form.get_subsection("local_authority") }
+    let(:later_subsection) { form.get_subsection("setup") }
 
     context "when a user is on the check answers page for a subsection" do
       def answer_household_needs(case_log)
@@ -85,6 +85,10 @@ RSpec.describe Form, type: :model do
         case_log.mrcdate = Time.zone.parse("03/11/2019")
       end
 
+      def answer_local_gdpr_acceptance(case_log)
+        case_log.gdpr_acceptance = "Yes"
+      end
+
       before do
         case_log.tenant_code = "123"
         case_log.age1 = 35
@@ -108,7 +112,7 @@ RSpec.describe Form, type: :model do
       end
 
       it "returns the next incomplete section by cycling back around if next subsections are completed" do
-        answer_local_authority(case_log)
+        answer_local_gdpr_acceptance(case_log)
         expect(form.next_incomplete_section_redirect_path(later_subsection, case_log)).to eq("armed-forces")
       end
 
@@ -124,6 +128,7 @@ RSpec.describe Form, type: :model do
         answer_income_and_benefits(case_log)
         answer_rent_and_charges(case_log)
         answer_local_authority(case_log)
+        answer_local_gdpr_acceptance(case_log)
 
         expect(form.next_incomplete_section_redirect_path(subsection, case_log)).to eq("declaration")
       end
diff --git a/spec/requests/case_logs_controller_spec.rb b/spec/requests/case_logs_controller_spec.rb
index 09f14089e..9fef47719 100644
--- a/spec/requests/case_logs_controller_spec.rb
+++ b/spec/requests/case_logs_controller_spec.rb
@@ -200,7 +200,7 @@ RSpec.describe CaseLogsController, type: :request do
             end
 
             it "displays a section status for a case log" do
-              assert_select ".govuk-tag", text: /Not started/, count: 8
+              assert_select ".govuk-tag", text: /Not started/, count: 9
               assert_select ".govuk-tag", text: /Completed/, count: 0
               assert_select ".govuk-tag", text: /Cannot start yet/, count: 1
             end
@@ -222,7 +222,7 @@ RSpec.describe CaseLogsController, type: :request do
             end
 
             it "displays a section status for a case log" do
-              assert_select ".govuk-tag", text: /Not started/, count: 7
+              assert_select ".govuk-tag", text: /Not started/, count: 8
               assert_select ".govuk-tag", text: /Completed/, count: 1
               assert_select ".govuk-tag", text: /Cannot start yet/, count: 1
             end

From 5e3631bdedb7c9748c0ce4a2ce190969946c6bfa Mon Sep 17 00:00:00 2001
From: baarkerlounger <db@slothlife.xyz>
Date: Fri, 4 Feb 2022 13:23:36 +0000
Subject: [PATCH 2/7] 2FA translations

---
 config/locales/en.yml | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/config/locales/en.yml b/config/locales/en.yml
index 9b0789828..0aabc8da4 100644
--- a/config/locales/en.yml
+++ b/config/locales/en.yml
@@ -129,3 +129,5 @@ en:
       max_login_attempts_reached: "Access completely denied as you have reached your attempts limit"
       contact_administrator: "Please contact your system administrator."
       code_has_been_sent: "Your security code has been sent."
+      code_required: "Security code is required"
+      code_incorrect: "Security code is incorrect"

From f0dedc42c26d9545fa81031cc85a62a2d186d554 Mon Sep 17 00:00:00 2001
From: baarkerlounger <db@slothlife.xyz>
Date: Fri, 4 Feb 2022 13:57:31 +0000
Subject: [PATCH 3/7] Format account locked view

---
 .../max_login_attempts_reached.html.erb               | 11 +++++++++++
 config/locales/en.yml                                 |  4 ++--
 2 files changed, 13 insertions(+), 2 deletions(-)
 create mode 100644 app/views/devise/two_factor_authentication/max_login_attempts_reached.html.erb

diff --git a/app/views/devise/two_factor_authentication/max_login_attempts_reached.html.erb b/app/views/devise/two_factor_authentication/max_login_attempts_reached.html.erb
new file mode 100644
index 000000000..3eae19de3
--- /dev/null
+++ b/app/views/devise/two_factor_authentication/max_login_attempts_reached.html.erb
@@ -0,0 +1,11 @@
+<% content_for :title, I18n.t("devise.two_factor_authentication.max_login_attempts_reached") %>
+
+<div class="govuk-grid-row">
+  <div class="govuk-grid-column-two-thirds">
+    <h1 class="govuk-heading-l">
+      <%= content_for(:title) %>
+    </h1>
+
+    <p><%= I18n.t("devise.two_factor_authentication.contact_administrator") %></p>
+  </div>
+</div>
diff --git a/config/locales/en.yml b/config/locales/en.yml
index 0aabc8da4..cccaed1e0 100644
--- a/config/locales/en.yml
+++ b/config/locales/en.yml
@@ -126,8 +126,8 @@ en:
     two_factor_authentication:
       success: "Two factor authentication successful."
       attempt_failed: "Attempt failed."
-      max_login_attempts_reached: "Access completely denied as you have reached your attempts limit"
-      contact_administrator: "Please contact your system administrator."
+      max_login_attempts_reached: "Too many incorrect attempts, your account has been locked"
+      contact_administrator: "Contact another helpdesk administrator."
       code_has_been_sent: "Your security code has been sent."
       code_required: "Security code is required"
       code_incorrect: "Security code is incorrect"

From 2ece8a45d0ea8b9a6a7cc04d37581151d2d080fb Mon Sep 17 00:00:00 2001
From: baarkerlounger <db@slothlife.xyz>
Date: Fri, 4 Feb 2022 16:21:54 +0000
Subject: [PATCH 4/7] Update content

---
 .../max_login_attempts_reached.html.erb                        | 1 +
 config/locales/en.yml                                          | 3 ++-
 2 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/app/views/devise/two_factor_authentication/max_login_attempts_reached.html.erb b/app/views/devise/two_factor_authentication/max_login_attempts_reached.html.erb
index 3eae19de3..7ff3d2d0e 100644
--- a/app/views/devise/two_factor_authentication/max_login_attempts_reached.html.erb
+++ b/app/views/devise/two_factor_authentication/max_login_attempts_reached.html.erb
@@ -6,6 +6,7 @@
       <%= content_for(:title) %>
     </h1>
 
+    <p><%= I18n.t("devise.two_factor_authentication.account_locked") %></p>
     <p><%= I18n.t("devise.two_factor_authentication.contact_administrator") %></p>
   </div>
 </div>
diff --git a/config/locales/en.yml b/config/locales/en.yml
index cccaed1e0..8664dcf1f 100644
--- a/config/locales/en.yml
+++ b/config/locales/en.yml
@@ -126,7 +126,8 @@ en:
     two_factor_authentication:
       success: "Two factor authentication successful."
       attempt_failed: "Attempt failed."
-      max_login_attempts_reached: "Too many incorrect attempts, your account has been locked"
+      max_login_attempts_reached: "Too many incorrect login attempts"
+      account_locked: "Your account has been locked for security reasons."
       contact_administrator: "Contact another helpdesk administrator."
       code_has_been_sent: "Your security code has been sent."
       code_required: "Security code is required"

From f89c550819e5ef69dc059c4851a5816ed611a147 Mon Sep 17 00:00:00 2001
From: baarkerlounger <db@slothlife.xyz>
Date: Fri, 4 Feb 2022 17:24:52 +0000
Subject: [PATCH 5/7] Wording

---
 config/locales/en.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/config/locales/en.yml b/config/locales/en.yml
index 8664dcf1f..17e5ce3aa 100644
--- a/config/locales/en.yml
+++ b/config/locales/en.yml
@@ -128,7 +128,7 @@ en:
       attempt_failed: "Attempt failed."
       max_login_attempts_reached: "Too many incorrect login attempts"
       account_locked: "Your account has been locked for security reasons."
-      contact_administrator: "Contact another helpdesk administrator."
+      contact_administrator: "Contact another helpdesk administrator for access."
       code_has_been_sent: "Your security code has been sent."
       code_required: "Security code is required"
       code_incorrect: "Security code is incorrect"

From 0db5f3a4dc0fc2297e32ed8956f6b4f0b2b63a00 Mon Sep 17 00:00:00 2001
From: baarkerlounger <5101747+baarkerlounger@users.noreply.github.com>
Date: Fri, 4 Feb 2022 17:41:36 +0000
Subject: [PATCH 6/7] Fix docker compose flow (#275)

* Make docker-compose actually useful

* Add docker-compose debug instructions to readme

* Add db seed for docker-compose instructions to readme

* Update readme to include migrate step

* DB connection
---
 Dockerfile          |  2 +-
 Dockerfile_dev      | 33 +++++++++++++++++++++++++++++++++
 Gemfile.lock        |  4 ++--
 README.md           | 28 ++++++++++++++++++++--------
 config/database.yml |  4 ++--
 docker-compose.yml  |  8 +++-----
 6 files changed, 61 insertions(+), 18 deletions(-)
 create mode 100644 Dockerfile_dev

diff --git a/Dockerfile b/Dockerfile
index b9bfae502..c6bc9d990 100644
--- a/Dockerfile
+++ b/Dockerfile
@@ -16,7 +16,7 @@ RUN apk add --no-cache build-base yarn postgresql-dev git
 
 # Install bundler to run bundle exec
 # This should be the same version as the Gemfile.lock
-RUN gem install bundler:2.2.3 --no-document
+RUN gem install bundler:2.3.4 --no-document
 
 # Install gems defined in Gemfile
 COPY .ruby-version Gemfile Gemfile.lock /app/
diff --git a/Dockerfile_dev b/Dockerfile_dev
new file mode 100644
index 000000000..aac15d8c6
--- /dev/null
+++ b/Dockerfile_dev
@@ -0,0 +1,33 @@
+# Build compilation image
+FROM ruby:3.0.3-alpine3.14 as builder
+
+# The application runs from /app
+WORKDIR /app
+
+# Add the timezone as it's not configured by default in Alpine
+RUN apk add --update --no-cache tzdata && cp /usr/share/zoneinfo/Europe/London /etc/localtime && echo "Europe/London" > /etc/timezone
+
+RUN apk add --no-cache build-base yarn postgresql-dev git bash
+
+# Install bundler to run bundle exec
+# This should be the same version as the Gemfile.lock
+RUN gem install bundler:2.3.4 --no-document
+
+# Install gems defined in Gemfile
+COPY .ruby-version Gemfile Gemfile.lock /app/
+
+ARG BUNDLE_FLAGS="--jobs=4 --no-binstubs --no-cache"
+RUN bundle install ${BUNDLE_FLAGS}
+
+# Install node packages defined in package.json, including webpack
+COPY package.json yarn.lock /app/
+RUN yarn install --frozen-lockfile
+
+# Copy all files to /app (except what is defined in .dockerignore)
+COPY . /app/
+
+ENV PORT=8080
+
+EXPOSE ${PORT}
+
+CMD RAILS_ENV=${RAILS_ENV} bundle exec rake db:migrate && bundle exec rails s -e ${RAILS_ENV} -p ${PORT} --binding=0.0.0.0
diff --git a/Gemfile.lock b/Gemfile.lock
index b985f123e..a4c12148f 100644
--- a/Gemfile.lock
+++ b/Gemfile.lock
@@ -8,7 +8,7 @@ GIT
 
 GIT
   remote: https://github.com/baarkerlounger/devise.git
-  revision: ac956cc2a58daa57e071202967a212a1dac08053
+  revision: 6d0b6b52a9d0e87ae6d9f9acb562169751623078
   branch: dluhc-fixes
   specs:
     devise (4.8.1)
@@ -31,7 +31,7 @@ GIT
 
 GIT
   remote: https://github.com/tagliala/activeadmin.git
-  revision: d1492c54e76871d95f3a7ff20e445b48f455d4cb
+  revision: 920f14393bcf56cea9dc155276d20bcd10f4fa85
   branch: feature/railties-7
   specs:
     activeadmin (2.9.0)
diff --git a/README.md b/README.md
index f75b018c9..47db577a0 100644
--- a/README.md
+++ b/README.md
@@ -29,25 +29,37 @@ Pre-requisites:
 4. Run the database migrations:\
   `rake db:migrate`
 
-5. Install the frontend depenencies:\
+5. Seed the database if required:\
+`rake db:seed`
+
+6. Install the frontend depenencies:\
   `yarn install`
 
-6. Start the Rails server:\
+7. Start the Rails server:\
   `bundle exec rails s`
 
 The Rails server will start on <http://localhost:3000>.
 
 ### Using Docker
 
-```sh
-docker-compose build
-docker-compose run --rm app rails db:create
-docker-compose up
-```
+1. Build the image:\
+`docker-compose build`
+
+2. Run the database migrations:\
+`docker-compose run --rm app /bin/bash -c 'rake db:migrate'`
+
+3. Seed the database if required:\
+`docker-compose run --rm app /bin/bash -c 'rake db:seed'`
+
+4. To be able to debug with Pry run the app using:\
+`docker-compose run --service-ports app`
+
+If this is not needed you can run `docker-compose up` as normal
+
 
 The Rails server will start on <http://localhost:8080>.
 
-Note `docker-compose` runs the production docker image (`RAILS_ENV=production`) as the Dockerfile doesn’t include development gems to keep the image size down.
+
 
 ## Infrastructure
 
diff --git a/config/database.yml b/config/database.yml
index f04b2a776..74e6632ef 100644
--- a/config/database.yml
+++ b/config/database.yml
@@ -28,8 +28,8 @@ default: &default
 
 development:
   <<: *default
-  database: data_collector_development
-  host: localhost
+  database: <%= ENV['DB_DATABASE'] || 'data_collector_development' %>
+  host: <%= ENV['DB_HOST'] || 'localhost' %>
 
 staging:
   <<: *default
diff --git a/docker-compose.yml b/docker-compose.yml
index 0ca8930a1..fc50f354e 100644
--- a/docker-compose.yml
+++ b/docker-compose.yml
@@ -5,7 +5,7 @@ volumes:
 
 services:
   db:
-    image: postgres:13.3-alpine
+    image: postgres:13.5-alpine
     restart: always
     # To preserve data between runs of docker-compose, we mount a folder from the host machine.
     volumes:
@@ -20,9 +20,7 @@ services:
   app:
     build:
       context: .
-      dockerfile: ./Dockerfile
-      args:
-        GIT_COMMIT_SHA: DOCKER_COMPOSE_GIT_COMMIT_SHA
+      dockerfile: ./Dockerfile_dev
     ports:
       - 8080:8080
     depends_on:
@@ -32,7 +30,7 @@ services:
       - DB_DATABASE=data_collector
       - DB_USERNAME=postgres
       - DB_PASSWORD=password
-      - RAILS_ENV=production
+      - RAILS_ENV=development
       - RAILS_SERVE_STATIC_FILES=true
       - RAILS_LOG_TO_STDOUT=true
       - SECRET_KEY_BASE=abcd1234

From ef2e6ff9981faf6e90ea7a4e2711dbf037951a85 Mon Sep 17 00:00:00 2001
From: baarkerlounger <db@slothlife.xyz>
Date: Mon, 7 Feb 2022 09:36:41 +0000
Subject: [PATCH 7/7] Bump active admin

---
 Gemfile.lock | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/Gemfile.lock b/Gemfile.lock
index a4c12148f..06bea744e 100644
--- a/Gemfile.lock
+++ b/Gemfile.lock
@@ -31,7 +31,7 @@ GIT
 
 GIT
   remote: https://github.com/tagliala/activeadmin.git
-  revision: 920f14393bcf56cea9dc155276d20bcd10f4fa85
+  revision: 5d6dfbe086331ce0538cc726abfe2eebe0260f02
   branch: feature/railties-7
   specs:
     activeadmin (2.9.0)
@@ -116,7 +116,7 @@ GEM
       public_suffix (>= 2.0.2, < 5.0)
     ast (2.4.2)
     aws-eventstream (1.2.0)
-    aws-partitions (1.552.0)
+    aws-partitions (1.553.0)
     aws-sdk-core (3.126.0)
       aws-eventstream (~> 1, >= 1.0.2)
       aws-partitions (~> 1, >= 1.525.0)
@@ -327,7 +327,7 @@ GEM
       activerecord (>= 5.2.4)
       activesupport (>= 5.2.4)
       i18n
-    rb-fsevent (0.11.0)
+    rb-fsevent (0.11.1)
     rb-inotify (0.10.1)
       ffi (~> 1.0)
     regexp_parser (2.2.0)