Add footer data to site details API

Author: etewiah

app/controllers/pwb/pages_controller.rb

@@ -13,9 +13,7 @@ def show_page
       default_page_slug = "home"
       page_slug = params[:page_slug] || default_page_slug
       @page = @current_website.pages.find_by_slug page_slug
-      if @page.blank?
-        @page = @current_website.pages.find_by_slug default_page_slug
-      end
+      @page = @current_website.pages.find_by_slug default_page_slug if @page.blank?
       @content_to_show = []
       @page_contents_for_edit = []
       @has_rails_parts = false
@@ -85,7 +83,7 @@ def header_image_url
       # lc_content = Content.where(tag: 'landing-carousel')[0]
       lc_photo = ContentPhoto.find_by_block_key "landing_img"
       # used for header background images
-      @header_image_url = lc_photo.present? ? lc_photo.optimized_image_url : nil
+      @header_image_url = lc_photo.presence&.optimized_image_url
     end
 
     # Extract LCP (Largest Contentful Paint) image from page content

app/models/pwb/website.rb

@@ -366,10 +366,19 @@ def as_json(options = nil)
                 "style_variables", "css_variables",
                 "contact_info", "social_links",
                 "top_nav_links", "footer_links",
-                "agency"
+                "agency", "footer_data"
               ] }.merge(options || {}))
     end
 
+    # API helper: Returns all footer-related data in a single object
+    def footer_data
+      {
+        "page_parts" => footer_page_parts,
+        "whitelabel" => whitelabel_for_api,
+        "admin_url" => admin_url
+      }
+    end
+
     # API helper: Returns contact information from agency
     def contact_info
       return {} unless agency
@@ -378,10 +387,52 @@ def contact_info
         "phone" => agency.phone_number_primary,
         "phone_mobile" => agency.phone_number_mobile,
         "email" => agency.email_primary,
-        "address" => format_agency_address
+        "address" => format_agency_address,
+        "address_details" => full_agency_address
       }
     end
 
+    # API helper: Returns full address details
+    def full_agency_address
+      return nil unless agency&.primary_address
+
+      addr = agency.primary_address
+      {
+        "street_number" => addr.street_number,
+        "street_address" => addr.street_address,
+        "city" => addr.city,
+        "region" => addr.region,
+        "postal_code" => addr.postal_code,
+        "country" => addr.country,
+        "latitude" => addr.latitude,
+        "longitude" => addr.longitude
+      }.compact
+    end
+
+    # API helper: Returns footer page parts (custom HTML content)
+    def footer_page_parts
+      footer_part = ordered_visible_page_contents&.find_by_page_part_key("footer_content_html")
+      return {} unless footer_part&.content
+
+      {
+        "footer_content_html" => footer_part.content.raw
+      }
+    end
+
+    # API helper: Returns whitelabel configuration for API
+    def whitelabel_for_api
+      config = whitelabel_config || {}
+      {
+        "show_powered_by" => config["show_powered_by"] != false,
+        "powered_by_url" => config["powered_by_url"] || "https://www.propertywebbuilder.com"
+      }
+    end
+
+    # API helper: Returns admin login URL
+    def admin_url
+      "/pwb_login"
+    end
+
     # API helper: Returns structured social media links
     def social_links
       {

spec/requests/api_public/v1/site_details_spec.rb

@@ -16,5 +16,98 @@
       json = response.parsed_body
       expect(json).to be_present
     end
+
+    it "includes footer_data object" do
+      get "/api_public/v1/site_details", params: { locale: "en" }
+      json = response.parsed_body
+
+      expect(json).to have_key("footer_data")
+      expect(json["footer_data"]).to have_key("page_parts")
+      expect(json["footer_data"]).to have_key("whitelabel")
+      expect(json["footer_data"]).to have_key("admin_url")
+    end
+
+    it "returns whitelabel configuration with defaults in footer_data" do
+      get "/api_public/v1/site_details", params: { locale: "en" }
+      json = response.parsed_body
+
+      expect(json["footer_data"]["whitelabel"]).to include(
+        "show_powered_by" => true,
+        "powered_by_url" => "https://www.propertywebbuilder.com"
+      )
+    end
+
+    it "returns admin_url in footer_data" do
+      get "/api_public/v1/site_details", params: { locale: "en" }
+      json = response.parsed_body
+
+      expect(json["footer_data"]["admin_url"]).to eq("/pwb_login")
+    end
+
+    it "returns empty page_parts in footer_data when none exist" do
+      get "/api_public/v1/site_details", params: { locale: "en" }
+      json = response.parsed_body
+
+      expect(json["footer_data"]["page_parts"]).to eq({})
+    end
+  end
+end
+
+RSpec.describe Pwb::Website, "footer data methods", type: :model do
+  let(:website) { FactoryBot.create(:pwb_website) }
+
+  describe "#footer_data" do
+    it "returns nested object with page_parts, whitelabel, and admin_url" do
+      expect(website.footer_data).to have_key("page_parts")
+      expect(website.footer_data).to have_key("whitelabel")
+      expect(website.footer_data).to have_key("admin_url")
+    end
+  end
+
+  describe "#whitelabel_for_api" do
+    it "returns default values when whitelabel_config is nil" do
+      expect(website.whitelabel_for_api).to eq({
+        "show_powered_by" => true,
+        "powered_by_url" => "https://www.propertywebbuilder.com"
+      })
+    end
+
+    it "respects custom whitelabel_config" do
+      website.update!(whitelabel_config: {
+        "show_powered_by" => false,
+        "powered_by_url" => "https://custom.com"
+      })
+
+      expect(website.whitelabel_for_api).to eq({
+        "show_powered_by" => false,
+        "powered_by_url" => "https://custom.com"
+      })
+    end
+  end
+
+  describe "#admin_url" do
+    it "returns /pwb_login" do
+      expect(website.admin_url).to eq("/pwb_login")
+    end
+  end
+
+  describe "#footer_page_parts" do
+    it "returns empty hash when no footer content exists" do
+      expect(website.footer_page_parts).to eq({})
+    end
+  end
+
+  describe "#full_agency_address" do
+    it "returns nil when no agency" do
+      expect(website.full_agency_address).to be_nil
+    end
+  end
+
+  describe "#contact_info" do
+    it "returns empty hash when no agency" do
+      website.agency&.destroy
+      website.reload
+      expect(website.contact_info).to eq({})
+    end
   end
 end