From f5e3b16d58f4dc4d678b3f43cc14affa8daf9814 Mon Sep 17 00:00:00 2001 From: Paul Robert Lloyd Date: Tue, 15 Mar 2022 15:34:02 +0000 Subject: [PATCH] Add helper to render Markdown content pages --- Gemfile | 2 ++ Gemfile.lock | 5 +++++ app/controllers/content_controller.rb | 11 +++++++++++ app/helpers/content_helper.rb | 14 ++++++++++++++ app/views/content/accessibility_statement.md | 0 app/views/content/page.html.erb | 9 +++++++++ app/views/content/privacy_notice.md | 0 config/routes.rb | 4 ++++ spec/helpers/content_helper_spec.rb | 13 +++++++++++++ 9 files changed, 58 insertions(+) create mode 100644 app/controllers/content_controller.rb create mode 100644 app/helpers/content_helper.rb create mode 100644 app/views/content/accessibility_statement.md create mode 100644 app/views/content/page.html.erb create mode 100644 app/views/content/privacy_notice.md create mode 100644 spec/helpers/content_helper_spec.rb diff --git a/Gemfile b/Gemfile index 9ce05d2b5..442986219 100644 --- a/Gemfile +++ b/Gemfile @@ -19,6 +19,8 @@ gem "bootsnap", ">= 1.4.4", require: false gem "govuk-components" # GOV UK component form builder DSL gem "govuk_design_system_formbuilder" +# Convert Markdown into GOV.UK frontend-styled HTML +gem "govuk_markdown" # GOV UK Notify gem "notifications-ruby-client" # Turbo and Stimulus diff --git a/Gemfile.lock b/Gemfile.lock index 1c5f83389..512665f41 100644 --- a/Gemfile.lock +++ b/Gemfile.lock @@ -181,6 +181,9 @@ GEM activemodel (>= 6.1) activesupport (>= 6.1) deep_merge (~> 1.2.1) + govuk_markdown (1.0.0) + activesupport + redcarpet has_scope (0.8.0) actionpack (>= 5.2) activesupport (>= 5.2) @@ -328,6 +331,7 @@ GEM rb-fsevent (0.11.1) rb-inotify (0.10.1) ffi (~> 1.0) + redcarpet (3.5.1) redis (4.6.0) regexp_parser (2.2.1) request_store (1.5.1) @@ -476,6 +480,7 @@ DEPENDENCIES factory_bot_rails govuk-components govuk_design_system_formbuilder + govuk_markdown hotwire-rails json-schema listen (~> 3.3) diff --git a/app/controllers/content_controller.rb b/app/controllers/content_controller.rb new file mode 100644 index 000000000..1ee9f55d4 --- /dev/null +++ b/app/controllers/content_controller.rb @@ -0,0 +1,11 @@ +class ContentController < ApplicationController + include ContentHelper + + def accessibility_statement + render_content_page :accessibility_statement + end + + def privacy_notice + render_content_page :privacy_notice, page_title: "Privacy notice for tenants and buyers of new social housing" + end +end diff --git a/app/helpers/content_helper.rb b/app/helpers/content_helper.rb new file mode 100644 index 000000000..08dc0b0cd --- /dev/null +++ b/app/helpers/content_helper.rb @@ -0,0 +1,14 @@ +module ContentHelper + def render_content_page(page_name, page_title: nil, locals: {}) + raw_content = File.read("app/views/content/#{page_name}.md") + content_with_erb_tags_replaced = ApplicationController.renderer.render( + inline: raw_content, + locals: locals + ) + + @page_title = page_title || page_name.to_s.humanize + @page_content = GovukMarkdown.render(content_with_erb_tags_replaced).html_safe + + render "content/page" + end +end diff --git a/app/views/content/accessibility_statement.md b/app/views/content/accessibility_statement.md new file mode 100644 index 000000000..e69de29bb diff --git a/app/views/content/page.html.erb b/app/views/content/page.html.erb new file mode 100644 index 000000000..39acc46f9 --- /dev/null +++ b/app/views/content/page.html.erb @@ -0,0 +1,9 @@ +<%= content_for :title, @page_title %> + +
+
+

<%= @page_title %>

+ + <%= @page_content %> +
+
diff --git a/app/views/content/privacy_notice.md b/app/views/content/privacy_notice.md new file mode 100644 index 000000000..e69de29bb diff --git a/config/routes.rb b/config/routes.rb index 2b6b4aa7d..867980bc1 100644 --- a/config/routes.rb +++ b/config/routes.rb @@ -33,6 +33,10 @@ Rails.application.routes.draw do root to: "start#index" + # Content pages + get "/accessibility-statement", to: "content#accessibility_statement" + get "/privacy-notice", to: "content#privacy_notice" + resources :users do member do get "password/edit", to: "users#edit_password" diff --git a/spec/helpers/content_helper_spec.rb b/spec/helpers/content_helper_spec.rb new file mode 100644 index 000000000..8c5172226 --- /dev/null +++ b/spec/helpers/content_helper_spec.rb @@ -0,0 +1,13 @@ +require "rails_helper" + +RSpec.describe ContentHelper do + let(:page_name) { "privacy_notice" } + + describe "render content page" do + it "returns the page" do + expected_html = "Privacy notice" + expect(render_content_page(page_name)).to match(expected_html) + expect(page).to have_title("Privacy notice") + end + end +end