Add rich text field to school profile

Author: richardpattinson

app/models/organisation.rb

@@ -1,11 +1,15 @@
 require "digest"
 
 class Organisation < ApplicationRecord
+  self.ignored_columns += %w[description]
+
   before_save :update_searchable_content
 
   include PgSearch::Model
   extend FriendlyId
 
+  has_rich_text :description
+
   SPECIAL_SCHOOL_TYPES = ["Community special school", "Foundation special school", "Non-maintained special school", "Academy special converter", "Academy special sponsor led", "Free schools special"].freeze
   NON_FAITH_RELIGIOUS_CHARACTER_TYPES = ["", "None", "Does not apply", "null"].freeze
   OUT_OF_SCOPE_DETAILED_SCHOOL_TYPES = ["Further education", "Other independent school", "Miscellaneous", "Special post 16 institution", "Other independent special school", "Higher education institutions", "Welsh establishment"].freeze
@@ -141,7 +145,7 @@ def has_ofsted_report?
   end
 
   def profile_complete?
-    %i[email description safeguarding_information logo photo url].all? { |attribute| send(attribute).present? }
+    %i[email safeguarding_information logo photo url].all? { |attr| send(attr).present? } && description.body.present?
   end
 
   def self.update_all_searchable_content!

app/views/publishers/organisations/_organisation.html.slim

@@ -44,7 +44,7 @@
   - summary_list.with_row(html_attributes: { id: "description" }) do |row|
     - row.with_key text: t("publishers.organisations.organisation.description.label.#{organisation.school? ? :school : :organisation}")
     - row.with_value do
-      = required_profile_info(value: truncate(organisation.description, length: 130), missing_prompt: t("publishers.organisations.organisation.description.missing_prompt.#{organisation.school? ? :school : :organisation}"))
+      = required_profile_info(value: truncate(organisation.description.to_plain_text, length: 130), missing_prompt: t("publishers.organisations.organisation.description.missing_prompt.#{organisation.school? ? :school : :organisation}"))
     - row.with_action text: t("buttons.change"), href: edit_publishers_organisation_description_path(organisation), classes: "govuk-link--no-visited-state", visually_hidden_text: t("publishers.organisations.organisation.description.label.#{organisation.school? ? :school : :organisation}")
 
   - summary_list.with_row(html_attributes: { id: "safeguarding_information" }) do |row|

app/views/publishers/organisations/description/edit.html.slim

@@ -12,9 +12,16 @@
   - if params["vacancy_id"]
     = f.hidden_field "vacancy_id", value: params["vacancy_id"]
 
-  = f.govuk_text_area :description,
-    label: { text: t("helpers.label.publishers_organisation_form.description", organisation_type: organisation_type_basic(@organisation)), size: "s" },
-    hint: { text: t("helpers.hint.publishers_organisation_form.description", organisation_type: organisation_type_basic(@organisation)) },
-    rows: 10
+  .govuk-form-group
+    = f.label :description, t("helpers.label.publishers_organisation_form.description", organisation_type: organisation_type_basic(@organisation)), class: "govuk-label govuk-label--s"
+    
+    .govuk-hint
+      = t("helpers.hint.publishers_organisation_form.description", organisation_type: organisation_type_basic(@organisation))
 
-  = f.govuk_submit t("buttons.save_changes")
+    div class="school-description-form"
+      = f.rich_text_area :description, aria: { label: t("helpers.label.publishers_organisation_form.description"), required: true }, class: "govuk-!-margin-bottom-6"
+
+      noscript
+        = f.govuk_text_area :description, rows: 10, class: "govuk-!-margin-bottom-6"
+
+  = f.govuk_submit t("buttons.save_changes")

config/analytics.yml

@@ -242,7 +242,6 @@ shared:
     - id
     - type
     - name
-    - description
     - safeguarding_information
     - urn
     - uid

db/migrate/20260129140738_migrate_school_description_to_rich_text.rb

@@ -0,0 +1,18 @@
+class MigrateSchoolDescriptionToRichText < ActiveRecord::Migration[8.0]
+  def up
+    Organisation.find_each do |org|
+      old_text = org.read_attribute(:description)
+      next if old_text.blank?
+
+      org.update!(description: old_text)
+    end
+  end
+
+  def down
+    Organisation.find_each do |org|
+      if org.description.body.present?
+        org.update_column(:description, org.description.body.to_html)
+      end
+    end
+  end
+end

db/schema.rb

@@ -10,7 +10,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema[8.0].define(version: 2026_01_20_090246) do
+ActiveRecord::Schema[8.0].define(version: 2026_01_29_140738) do
   # These are extensions that must be enabled in order to support this database
   enable_extension "btree_gist"
   enable_extension "citext"

spec/views/organisations/show.html.slim_spec.rb

@@ -78,7 +78,7 @@
     end
 
     it "displays the organisation's description" do
-      expect(show_view).to have_content(organisation.description)
+      expect(show_view).to have_content(organisation.description.to_plain_text)
     end
 
     it "displays the organisation's safeguarding information" do

spec/views/publishers/organisations/preview.html.slim_spec.rb

@@ -16,7 +16,7 @@
     end
 
     it "displays the organisation's description" do
-      expect(rendered).to have_content(organisation.description)
+      expect(rendered).to have_content(organisation.description.to_plain_text)
     end
 
     it "displays the organisation's safeguarding information" do
@@ -48,7 +48,7 @@
     end
 
     it "displays the organisation's description" do
-      expect(rendered).to have_content(organisation.description)
+      expect(rendered).to have_content(organisation.description.to_plain_text)
     end
 
     it "displays the organisation's safeguarding information" do
@@ -82,7 +82,7 @@
       end
 
       it "displays the organisation's description" do
-        expect(rendered).to have_content(school_one.description)
+        expect(rendered).to have_content(school_one.description.to_plain_text)
       end
 
       it "displays the organisation's safeguarding information" do