panda-cms 0.7.4 → 0.7.5

data/app/models/panda/cms/menu.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Panda
   module CMS
     class Menu < ApplicationRecord
@@ -5,13 +7,16 @@ module Panda
 
       after_save :generate_auto_menu_items, if: -> { kind == "auto" }
 
-      has_many :menu_items, -> { order(lft: :asc) }, foreign_key: :panda_cms_menu_id, class_name: "Panda::CMS::MenuItem", inverse_of: :menu
-      belongs_to :start_page, class_name: "Panda::CMS::Page", foreign_key: "start_page_id", inverse_of: :page_menu, optional: true
+      has_many :menu_items, lambda {
+        order(lft: :asc)
+      }, foreign_key: :panda_cms_menu_id, class_name: "Panda::CMS::MenuItem", inverse_of: :menu
+      belongs_to :start_page, class_name: "Panda::CMS::Page", foreign_key: "start_page_id", inverse_of: :page_menu,
+        optional: true
 
       accepts_nested_attributes_for :menu_items, reject_if: :all_blank, allow_destroy: true
 
       validates :name, presence: true, uniqueness: {case_sensitive: false}
-      validates :kind, presence: true, inclusion: {in: ["static", "auto"]}
+      validates :kind, presence: true, inclusion: {in: %w[static auto]}
       validate :validate_start_page
 
       def generate_auto_menu_items
@@ -30,9 +35,7 @@ module Panda
       def generate_menu_items(parent_menu_item:, parent_page:)
         parent_page.children.where(status: [:active]).each do |page|
           menu_item = menu_items.create(text: page.title, panda_cms_page_id: page.id, parent: parent_menu_item)
-          if page.children
-            generate_menu_items(parent_menu_item: menu_item, parent_page: page)
-          end
+          generate_menu_items(parent_menu_item: menu_item, parent_page: page) if page.children
         end
       end
 
@@ -43,9 +46,9 @@ module Panda
       # @visibility private
       #
       def validate_start_page
-        if kind == "auto" && start_page.nil?
-          errors.add(:start_page, "can't be blank")
-        end
+        return unless kind == "auto" && start_page.nil?
+
+        errors.add(:start_page, "can't be blank")
       end
     end
   end

data/app/models/panda/cms/menu_item.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "awesome_nested_set"
 
 module Panda
@@ -8,8 +10,10 @@ module Panda
       self.implicit_order_column = "lft"
       self.table_name = "panda_cms_menu_items"
 
-      belongs_to :menu, foreign_key: :panda_cms_menu_id, class_name: "Panda::CMS::Menu", inverse_of: :menu_items, touch: true
-      belongs_to :page, foreign_key: :panda_cms_page_id, class_name: "Panda::CMS::Page", inverse_of: :menu_items, optional: true
+      belongs_to :menu, foreign_key: :panda_cms_menu_id, class_name: "Panda::CMS::Menu", inverse_of: :menu_items,
+        touch: true
+      belongs_to :page, foreign_key: :panda_cms_page_id, class_name: "Panda::CMS::Page", inverse_of: :menu_items,
+        optional: true
 
       validates :text, presence: true, uniqueness: {scope: :panda_cms_menu_id, case_sensitive: false}
       validates :page, presence: true, unless: -> { external_url.present? }
@@ -48,10 +52,10 @@ module Panda
           errors.add(:external_url, "must be a valid page or external link, neither are set")
         end
 
-        if !page.nil? && !external_url.nil?
-          errors.add(:page, "must be a valid page or external link, both are set")
-          errors.add(:external_url, "must be a valid page or external link, both are set")
-        end
+        return unless !page.nil? && !external_url.nil?
+
+        errors.add(:page, "must be a valid page or external link, both are set")
+        errors.add(:external_url, "must be a valid page or external link, both are set")
       end
     end
   end

data/app/models/panda/cms/page.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "awesome_nested_set"
 
 module Panda
@@ -8,7 +10,8 @@ module Panda
       self.implicit_order_column = "lft"
 
       belongs_to :template, class_name: "Panda::CMS::Template", foreign_key: :panda_cms_template_id
-      has_many :block_contents, class_name: "Panda::CMS::BlockContent", foreign_key: :panda_cms_page_id, dependent: :destroy
+      has_many :block_contents, class_name: "Panda::CMS::BlockContent", foreign_key: :panda_cms_page_id,
+        dependent: :destroy
       has_many :blocks, through: :block_contents
       has_many :menu_items, foreign_key: :panda_cms_page_id, class_name: "Panda::CMS::MenuItem", inverse_of: :page
       has_many :menus, through: :menu_items
@@ -19,7 +22,7 @@ module Panda
 
       validates :path,
         presence: true,
-        format: {with: /\A\/.*\z/, message: "must start with a forward slash"}
+        format: {with: %r{\A/.*\z}, message: "must start with a forward slash"}
 
       validate :validate_unique_path_in_scope
 
@@ -62,12 +65,11 @@ module Panda
         # Find any other pages with the same path
         other_page = self.class.where(path: path).where.not(id: id).first
 
-        if other_page
-          # If there's another page with the same path, check if it has a different parent
-          if other_page.parent_id == parent_id
-            errors.add(:path, "has already been taken in this section")
-          end
-        end
+        return unless other_page
+        # If there's another page with the same path, check if it has a different parent
+        return unless other_page.parent_id == parent_id
+
+        errors.add(:path, "has already been taken in this section")
       end
 
       #
@@ -88,10 +90,10 @@ module Panda
         page_existing_block_ids = block_contents.map { |bc| bc.block.id }
         required_block_ids = template_block_ids - page_existing_block_ids
 
-        if required_block_ids.count > 0
-          required_block_ids.each do |block_id|
-            Panda::CMS::BlockContent.find_or_create_by!(page: self, panda_cms_block_id: block_id, content: "")
-          end
+        return unless required_block_ids.count.positive?
+
+        required_block_ids.each do |block_id|
+          Panda::CMS::BlockContent.find_or_create_by!(page: self, panda_cms_block_id: block_id, content: "")
         end
       end
 

data/app/models/panda/cms/post.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "awesome_nested_set"
 
 module Panda
@@ -47,11 +49,13 @@ module Panda
 
       def year
         return nil unless slug.match?(%r{\A/\d{4}/})
+
         slug.split("/")[1]
       end
 
       def month
         return nil unless slug.match?(%r{\A/\d{4}/\d{2}/})
+
         slug.split("/")[2]
       end
 
@@ -96,13 +100,13 @@ module Panda
         self.slug = CGI.unescape(slug.strip.gsub(%r{^/+|/+$}, ""))
 
         # Handle the case where we already have a properly formatted slug
-        if slug.match?(%r{\A\d{4}/\d{2}/[^/]+\z})
-          return self.slug = "/#{slug}"
-        end
+        return self.slug = "/#{slug}" if slug.match?(%r{\A\d{4}/\d{2}/[^/]+\z})
 
         # Handle the case where we have a date-prefixed slug (from JS)
-        if (match = slug.match(%r{\A(\d{4})-(\d{2})-(.+)\z}))
-          year, month, base_slug = match[1], match[2], match[3]
+        if (match = slug.match(/\A(\d{4})-(\d{2})-(.+)\z/))
+          year = match[1]
+          month = match[2]
+          base_slug = match[3]
           return self.slug = "/#{year}/#{month}/#{base_slug}"
         end
 

data/app/models/panda/cms/redirect.rb

@@ -1,16 +1,19 @@
+# frozen_string_literal: true
+
 module Panda
   module CMS
     class Redirect < ApplicationRecord
       belongs_to :origin_page, class_name: "Panda::CMS::Page", foreign_key: :origin_panda_cms_page_id, optional: true
-      belongs_to :destination_page, class_name: "Panda::CMS::Page", foreign_key: :destination_panda_cms_page_id, optional: true
+      belongs_to :destination_page, class_name: "Panda::CMS::Page", foreign_key: :destination_panda_cms_page_id,
+        optional: true
 
       validates :status_code, presence: true
       validates :visits, presence: true
       validates :origin_path, presence: true
       validates :destination_path, presence: true
 
-      validates :origin_path, format: {with: /\A\/.*\z/, message: "must start with a forward slash"}
-      validates :destination_path, format: {with: /\A\/.*\z/, message: "must start with a forward slash"}
+      validates :origin_path, format: {with: %r{\A/.*\z}, message: "must start with a forward slash"}
+      validates :destination_path, format: {with: %r{\A/.*\z}, message: "must start with a forward slash"}
     end
   end
 end

data/app/models/panda/cms/template.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Panda
   module CMS
     # Represents a template in the Panda CMS application.
@@ -5,8 +7,10 @@ module Panda
       self.table_name = "panda_cms_templates"
 
       # Associations
-      has_many :pages, class_name: "Panda::CMS::Page", dependent: :restrict_with_error, inverse_of: :template, foreign_key: :panda_cms_template_id, counter_cache: :pages_count
-      has_many :blocks, class_name: "Panda::CMS::Block", dependent: :restrict_with_error, inverse_of: :template, foreign_key: :panda_cms_template_id
+      has_many :pages, class_name: "Panda::CMS::Page", dependent: :restrict_with_error, inverse_of: :template,
+        foreign_key: :panda_cms_template_id, counter_cache: :pages_count
+      has_many :blocks, class_name: "Panda::CMS::Block", dependent: :restrict_with_error, inverse_of: :template,
+        foreign_key: :panda_cms_template_id
       has_many :block_contents, through: :blocks
 
       # Validations
@@ -15,12 +19,12 @@ module Panda
       validates :file_path,
         presence: true,
         uniqueness: true,
-        format: {with: /\Alayouts\/.*\z/, message: "must be a valid layout file path"}
+        format: {with: %r{\Alayouts/.*\z}, message: "must be a valid layout file path"}
 
       validate :validate_template_file_exists
 
       # Scopes
-      scope :available, -> {
+      scope :available, lambda {
         where("max_uses IS NULL OR (max_uses > 0 AND pages_count < max_uses)")
       }
 
@@ -43,7 +47,7 @@ module Panda
             # Matches:
             # Panda::CMS::RichTextComponent.new(key: :value)
             # Panda::CMS::RichTextComponent.new key: :value, key: value
-            line.match(/Panda::CMS::([a-zA-Z]+)Component\.new[ \(]+([^\)]+)[\)]*/) do |match|
+            line.match(/Panda::CMS::([a-zA-Z]+)Component\.new[ (]+([^)]+)\)*/) do |match|
               # Extract the hash values
               template_path = file.gsub("app/views/", "").gsub(".html.erb", "")
               template_name = template_path.gsub("layouts/", "").titleize
@@ -68,7 +72,8 @@ module Panda
                 # Create the block if it doesn't exist
                 # TODO: +/- the output if it's created or removed
                 begin
-                  block = Panda::CMS::Block.find_or_create_by!(template: template, kind: block_kind, key: block_name) do |block|
+                  block = Panda::CMS::Block.find_or_create_by!(template: template, kind: block_kind,
+                    key: block_name) do |block|
                     block.name = block_name.titleize
                   end
                 rescue ActiveRecord::RecordInvalid => e
@@ -108,7 +113,7 @@ module Panda
           # Extract the file path from the Rails root
           file_path = file.to_s.sub("#{Rails.root}/app/views/", "").sub(".html.erb", "")
 
-          next if file_path == "layouts/application" || file_path == "layouts/mailer"
+          next if ["layouts/application", "layouts/mailer"].include?(file_path)
 
           # Find or create the template based on the file path
           find_or_create_by(file_path: file_path) do |t|

data/app/models/panda/cms/user.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Panda
   module CMS
     class User < ApplicationRecord

data/app/models/panda/cms/visit.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Panda
   module CMS
     class Visit < ApplicationRecord

data/app/models/panda/social/instagram_post.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Panda
   module Social
     class InstagramPost < ApplicationRecord

data/app/services/panda/cms/html_to_editor_js_converter.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Panda
   module CMS
     class HtmlToEditorJsConverter
@@ -54,7 +56,7 @@ module Panda
                     }
                   when "p"
                     text = process_inline_elements(child)
-                    paragraphs = text.split(/<br\s*\/?>\s*<br\s*\/?>/).map(&:strip)
+                    paragraphs = text.split(%r{<br\s*/?>\s*<br\s*/?>}).map(&:strip)
                     paragraphs.each do |paragraph|
                       blocks << create_paragraph_block(paragraph) if paragraph.present?
                     end
@@ -86,7 +88,7 @@ module Panda
               else
                 # Handle p with nested content
                 text = process_inline_elements(node)
-                paragraphs = text.split(/<br\s*\/?>\s*<br\s*\/?>/).map(&:strip)
+                paragraphs = text.split(%r{<br\s*/?>\s*<br\s*/?>}).map(&:strip)
                 paragraphs.each do |paragraph|
                   blocks << create_paragraph_block(paragraph) if paragraph.present?
                 end

data/app/services/panda/social/instagram_feed_service.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "http"
 require "down"
 
@@ -5,7 +7,7 @@ module Panda
   module Social
     class InstagramFeedService
       GRAPH_API_VERSION = "v19.0"
-      GRAPH_API_BASE_URL = "https://graph.instagram.com/#{GRAPH_API_VERSION}"
+      GRAPH_API_BASE_URL = "https://graph.instagram.com/#{GRAPH_API_VERSION}".freeze
 
       def initialize(access_token)
         @access_token = access_token

data/app/views/layouts/different_page.html.erb

@@ -0,0 +1,6 @@
+<%= render "shared/header" %>
+  <h1><%= @page.title %></h1>
+  <h2>Different Page Layout</h2>
+
+  <%= yield %>
+<%= render "shared/footer" %>

data/app/views/layouts/homepage.html.erb

@@ -0,0 +1,37 @@
+<!DOCTYPE html>
+<html>
+<head>
+  <title>Test Homepage</title>
+  <% if params[:embed_id].present? %>
+    <!-- Include Panda CMS assets for editor functionality when in edit mode -->
+    <%= panda_cms_complete_assets %>
+  <% end %>
+</head>
+<body>
+  <h1><%= @page.title %></h1>
+  <h2>Homepage Layout</h2>
+
+  <div class="prose">
+    <p>On this page we expect:</p>
+    <ul>
+      <li>Header</li>
+      <li>Footer</li>
+      <li>Some content</li>
+      <li>Tailwind-styled content</li>
+      <li>JS injected content through vanilla JS:
+        <div class="mt-0 font-bold" id="vanilla-injected-content">
+          Hello, Stimulus!
+        </div>
+      </li>
+      <li>JS injected content through <code>hello-controller</code>:
+        <div class="mt-0 font-bold" data-controller="hello" id="stimulus-injected-content">
+          Hello, Stimulus!
+        </div>
+      </li>
+    </ul>
+  </div>
+
+  <%= render Panda::CMS::RichTextComponent.new(key: :hero_content) %>
+  <%= yield %>
+</body>
+</html>

data/app/views/layouts/page.html.erb

@@ -0,0 +1,18 @@
+<!DOCTYPE html>
+<html>
+<head>
+  <title>Test Page</title>
+  <% if params[:embed_id].present? %>
+    <!-- Include Panda CMS assets for editor functionality when in edit mode -->
+    <%= panda_cms_complete_assets %>
+  <% end %>
+</head>
+<body>
+  <h1><%= @page.title %></h1>
+  <h2>Basic Page Layout</h2>
+  <%= render Panda::CMS::TextComponent.new(key: :plain_text) %>
+  <%= render Panda::CMS::CodeComponent.new(key: :html_code) %>
+  <%= render Panda::CMS::RichTextComponent.new(key: :main_content) %>
+  <%= yield %>
+</body>
+</html>

data/app/views/layouts/panda/cms/application.html.erb

@@ -9,6 +9,7 @@
   <section id="panda-main" class="flex flex-row h-full">
     <div class="flex-1 h-full" id="panda-cms-primary-content">
       <%= render "panda/cms/admin/shared/breadcrumbs" %>
+      <%= render "panda/cms/admin/shared/flash" %>
       <%= yield %>
     </div>
     <% if content_for :sidebar %>

data/app/views/panda/cms/admin/pages/new.html.erb

@@ -2,14 +2,20 @@
   <% component.with_heading(text: "Add Page", level: 1) do |heading| %>
   <% end %>
   <%= panda_cms_form_with model: page, url: admin_pages_path, method: :post do |f| %>
+    <% if page.errors.any? %>
+      <div class="mb-4 p-4 bg-red-50 border border-red-200 rounded-md">
+        <div class="text-sm text-red-600">
+          <% page.errors.full_messages.each do |message| %>
+            <p><%= message %></p>
+          <% end %>
+        </div>
+      </div>
+    <% end %>
     <% options = nested_set_options(Panda::CMS::Page, page) { |i| "#{"-" * i.level} #{i.title} (#{i.path})" } %>
-    <div data-controller="slug">
-      <input type="hidden" value="<%= Panda::CMS::Current.root %>" data-slug-target="existing_root">
-      <%= f.select :parent_id, options, {}, { "data-slug-target": "input_select", "data-action": "change->slug#setPrePath" } %>
-      <%= f.text_field :title, { data: { "slug-target": "input_text", action: "input->slug#generatePath focusout->slug#generatePath" } } %>
-      <%= f.text_field :path, { meta: t(".path.meta"), data: { prefix: Panda::CMS::Current.root, "slug-target": "output_text", action: "input->slug#handlePathInput" } } %>
-      <%= f.collection_select :panda_cms_template_id, available_templates, :id, :name %>
-      <%= f.button "Create Page" %>
-    </div>
+    <%= f.select :parent_id, options %>
+    <%= f.text_field :title %>
+    <%= f.text_field :path, { meta: t(".path.meta") } %>
+    <%= f.collection_select :panda_cms_template_id, available_templates, :id, :name %>
+    <%= f.button "Create Page" %>
   <% end %>
 <% end %>

data/app/views/panda/cms/admin/settings/index.html.erb

@@ -18,6 +18,6 @@
 
   <div class="text-center mt-6 space-y-2">
     <p class="text-sm font-semibold">🐼 Panda CMS version: <%= Panda::CMS::VERSION %></p>
-    <p class="text-sm">&copy; <%= Date.current.year %> Panda Software Limited. All rights reserved.</p>
+    <p class="text-sm">&copy; <%= Date.current.year %> Otaina Limited. All rights reserved.</p>
   </div>
 <% end %>

data/app/views/panda/cms/shared/_header.html.erb

@@ -7,9 +7,17 @@
     <%= csrf_meta_tags %>
     <%= csp_meta_tag %>
     <script src="https://kit.fontawesome.com/7835d81e75.js" defer="true" crossorigin="anonymous"></script>
-    <script async src="https://ga.jspm.io/npm:es-module-shims@1.10.0/dist/es-module-shims.js" defer="true" crossorigin="anonymous"></script>
     <%= stylesheet_link_tag "panda.cms", "data-turbo-track": "reload", media: "all" %>
-    <%= render "panda/cms/shared/importmap" %>
+
+    <% if Panda::CMS::AssetLoader.use_github_assets? || Rails.env.test? %>
+      <!-- Using compiled Panda CMS assets -->
+      <%= panda_cms_complete_assets %>
+    <% else %>
+      <!-- Using development importmap assets -->
+      <script async src="https://ga.jspm.io/npm:es-module-shims@1.10.0/dist/es-module-shims.js" defer="true" crossorigin="anonymous"></script>
+      <%= render "panda/cms/shared/importmap" %>
+    <% end %>
+
     <%= render "panda/cms/shared/favicons" %>
     <%= yield :head %>
   </head>

data/app/views/panda/cms/shared/_importmap.html.erb

@@ -8,7 +8,7 @@
         # Vendored
         "@hotwired/turbo": asset_path("panda/cms/@hotwired--turbo.js"),
         "@hotwired/stimulus": asset_path("panda/cms/@hotwired--stimulus.js"),
-        "@hotwired/stimulus-loading": asset_path("stimulus-loading.js"),
+        "@hotwired/stimulus-loading": asset_path("panda_cms/stimulus-loading.js"),
         "@editorjs/editorjs": asset_path("panda/cms/@editorjs--editorjs.js"),
         "tailwindcss-stimulus-components": asset_path("panda/cms/tailwindcss-stimulus-components.js"),
         # Our page editor

data/app/views/shared/_footer.html.erb

@@ -0,0 +1,3 @@
+<h1>Test Footer</h1>
+</body>
+</html>

data/app/views/shared/_header.html.erb

@@ -0,0 +1,11 @@
+<!DOCTYPE html>
+<html>
+<head>
+  <title>Panda CMS Page</title>
+  <% if params[:embed_id].present? %>
+    <!-- Include Panda CMS assets for editor functionality when in edit mode -->
+    <%= panda_cms_complete_assets %>
+  <% end %>
+</head>
+<body>
+<h1>Test Header</h1>

data/config/importmap.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 pin "application_panda_cms", to: "panda_cms/application_panda_cms.js", preload: true
 
 pin "@hotwired/turbo", to: "@hotwired--turbo.js", preload: true # @8.0.12

data/config/initializers/inflections.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 ActiveSupport::Inflector.inflections(:en) do |inflect|
   inflect.acronym "CMS"
 end

data/config/initializers/panda/cms/form_errors.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 ActionView::Base.field_error_proc = proc do |html_tag, instance|
   html = ""
   form_fields = %w[input select textarea trix-editor label].join(", ")
@@ -6,30 +8,27 @@ ActionView::Base.field_error_proc = proc do |html_tag, instance|
   autofocused = false
 
   Nokogiri::HTML::DocumentFragment.parse(html_tag).css(form_fields).each do |element|
-    if form_fields.include?(element.node_name)
-      if !autofocused
-        # element.attribute("autofocus", "true")
-        autofocused = true
-      end
+    next unless form_fields.include?(element.node_name)
 
-      message = "#{instance.object.class.human_attribute_name(instance.send(:sanitized_method_name))} "
-      message += if instance.error_message.respond_to?(:each)
-        "#{instance.error_message.uniq.to_sentence}."
-      else
-        "#{instance.error_message}."
-      end
+    autofocused ||= true
+
+    message = "#{instance.object.class.human_attribute_name(instance.send(:sanitized_method_name))} "
+    message += if instance.error_message.respond_to?(:each)
+      "#{instance.error_message.uniq.to_sentence}."
+    else
+      "#{instance.error_message}."
+    end
 
-      if element.node_name.eql?("label")
-        html = element.to_s
+    if element.node_name.eql?("label")
+      html = element.to_s
+    else
+      element.add_class(error_class)
+      html = if element.get_attribute("data-prefix")
+        "#{element}</div><div class=\"#{message_class}\">#{message}"
+      elsif element.get_attribute("type") != "checkbox"
+        "#{element}<div class=\"#{message_class}\">#{message}</div>"
       else
-        element.add_class(error_class)
-        html = if element.get_attribute("data-prefix")
-          "#{element}</div><div class=\"#{message_class}\">#{message}"
-        elsif element.get_attribute("type") != "checkbox"
-          "#{element}<div class=\"#{message_class}\">#{message}</div>"
-        else
-          element.to_s
-        end
+        element.to_s
       end
     end
   end

data/config/initializers/panda/cms/healthcheck_log_silencer.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require "silencer/rails/logger"
 
 # Don't log requests to the healthcheck endpoint

data/config/initializers/panda/cms.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 Panda::CMS.configure do |config|
   # The main title of your website
   config.title = "Demo Site"

data/config/initializers/zeitwork.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 Rails.autoloaders.main.inflector.inflect(
   "panda-cms" => "Panda::CMS"
 )

data/config/puma/test.rb

@@ -1,8 +1,10 @@
+# frozen_string_literal: true
+
 # Puma configuration for test environment
 workers 0
 threads 1, 1
 
-port ENV.fetch("PORT") { 3001 }
+port ENV.fetch("PORT", 3001)
 environment "test"
 
 # Allow for proper shutdown

data/config/routes.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require_relative "../app/constraints/panda/cms/admin_constraint"
 
 Panda::CMS::Engine.routes.draw do
@@ -19,10 +21,6 @@ Panda::CMS::Engine.routes.draw do
         get "bulk_editor", to: "bulk_editor#new"
         post "bulk_editor", to: "bulk_editor#create"
       end
-
-      if Rails.env.development?
-        mount Lookbook::Engine, at: "/lookbook"
-      end
     end
 
     get Panda::CMS.route_namespace, to: "admin/dashboard#show", as: :admin_dashboard
@@ -33,8 +31,10 @@ Panda::CMS::Engine.routes.draw do
   # Authentication routes
   get Panda::CMS.route_namespace, to: "admin/sessions#new", as: :admin_login
   # Get and post options here are for OmniAuth coming back in, not going out
-  match "#{Panda::CMS.route_namespace}/auth/:provider/callback", to: "admin/sessions#create", as: :admin_login_callback, via: %i[get post]
-  match "#{Panda::CMS.route_namespace}/auth/failure", to: "admin/sessions#failure", as: :admin_login_failure, via: %i[get post]
+  match "#{Panda::CMS.route_namespace}/auth/:provider/callback", to: "admin/sessions#create",
+    as: :admin_login_callback, via: %i[get post]
+  match "#{Panda::CMS.route_namespace}/auth/failure", to: "admin/sessions#failure", as: :admin_login_failure,
+    via: %i[get post]
   # OmniAuth additionally adds a GET route for "#{Panda::CMS.route_namespace}/auth/:provider" but doesn't name it
   delete Panda::CMS.route_namespace, to: "admin/sessions#destroy", as: :admin_logout
 
@@ -51,7 +51,7 @@ Panda::CMS::Engine.routes.draw do
       constraints: {
         year: /\d{4}/,
         month: /\d{2}/,
-        slug: /[^\/]+/,
+        slug: %r{[^/]+},
         format: /html|json|xml/
       }
 
@@ -60,7 +60,7 @@ Panda::CMS::Engine.routes.draw do
       to: "posts#show",
       as: :post,
       constraints: {
-        slug: /[^\/]+/,
+        slug: %r{[^/]+},
         format: /html|json|xml/
       }
 

data/db/migrate/20240205223709_create_panda_cms_pages.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class CreatePandaCMSPages < ActiveRecord::Migration[7.1]
   def change
     create_table :panda_cms_pages, id: :uuid do |t|