Add rendered page content support

Author: etewiah

app/controllers/api_public/v1/pages_controller.rb

@@ -2,6 +2,7 @@ module ApiPublic
   module V1
     class PagesController < BaseController
       include ApiPublic::Cacheable
+      include UrlLocalizationHelper
 
       def show
         unless website_provisioned?
@@ -69,12 +70,39 @@ def available_page_slugs
       def page_json(page)
         json = page.as_json
 
-        # Include page parts if requested
+        # Include page parts if requested (legacy - metadata only)
         json["page_parts"] = build_page_parts(page) if params[:include_parts] == "true"
 
+        # Include rendered page contents with pre-rendered HTML
+        # This is the preferred approach for frontend clients that need to display content
+        json["page_contents"] = build_rendered_page_contents(page) if params[:include_rendered] == "true"
+
         json
       end
 
+      # Build rendered page contents with pre-rendered HTML
+      # This matches how Pwb::PagesController#show_page works
+      # The HTML is already rendered via Liquid and stored in Content.raw
+      def build_rendered_page_contents(page)
+        return [] unless page.respond_to?(:ordered_visible_page_contents)
+
+        page.ordered_visible_page_contents.map do |page_content|
+          raw_html = page_content.is_rails_part ? nil : page_content.content&.raw
+          # Localize URLs in HTML content based on current locale
+          localized_html = raw_html.present? ? localize_html_urls(raw_html) : nil
+
+          {
+            page_part_key: page_content.page_part_key,
+            sort_order: page_content.sort_order,
+            visible: page_content.visible_on_page,
+            is_rails_part: page_content.is_rails_part || false,
+            rendered_html: localized_html,
+            # Include label for debugging/admin purposes
+            label: page_content.label
+          }
+        end
+      end
+
       def build_page_parts(page)
         return [] unless page.respond_to?(:page_parts)
 

spec/requests/api_public/v1/pages_spec.rb

@@ -63,6 +63,87 @@
     end
   end
 
+  describe "include_rendered parameter" do
+    let!(:page_with_content) do
+      ActsAsTenant.with_tenant(website) do
+        page = FactoryBot.create(:pwb_page, slug: "home", website: website)
+        # Create page content with rendered HTML
+        content = Pwb::Content.create!(
+          website: website,
+          page_part_key: "heroes/hero_centered",
+          raw: "<section class='hero'><h1>Welcome</h1></section>"
+        )
+        Pwb::PageContent.create!(
+          page: page,
+          website: website,
+          content: content,
+          page_part_key: "heroes/hero_centered",
+          sort_order: 1,
+          visible_on_page: true,
+          is_rails_part: false
+        )
+        # Create a Rails part (no pre-rendered HTML)
+        Pwb::PageContent.create!(
+          page: page,
+          website: website,
+          page_part_key: "properties/featured",
+          sort_order: 2,
+          visible_on_page: true,
+          is_rails_part: true
+        )
+        page
+      end
+    end
+
+    it "returns rendered HTML for page contents when include_rendered=true" do
+      get "/api_public/v1/pages/by_slug/home", params: { include_rendered: "true" }
+      expect(response).to have_http_status(200)
+      json = response.parsed_body
+
+      expect(json["page_contents"]).to be_an(Array)
+      expect(json["page_contents"].length).to eq(2)
+
+      # First content: Liquid-rendered HTML
+      hero_content = json["page_contents"].find { |c| c["page_part_key"] == "heroes/hero_centered" }
+      expect(hero_content["is_rails_part"]).to be false
+      expect(hero_content["rendered_html"]).to include("<section class='hero'>")
+      expect(hero_content["rendered_html"]).to include("<h1>Welcome</h1>")
+
+      # Second content: Rails part (no HTML)
+      rails_content = json["page_contents"].find { |c| c["page_part_key"] == "properties/featured" }
+      expect(rails_content["is_rails_part"]).to be true
+      expect(rails_content["rendered_html"]).to be_nil
+    end
+
+    it "does not include page_contents when include_rendered is not specified" do
+      get "/api_public/v1/pages/by_slug/home"
+      expect(response).to have_http_status(200)
+      json = response.parsed_body
+
+      expect(json["page_contents"]).to be_nil
+    end
+
+    context "with non-default locale" do
+      it "localizes URLs in rendered HTML" do
+        # Create content with a link that should be localized
+        ActsAsTenant.with_tenant(website) do
+          content = page_with_content.page_contents.first.content
+          content.update!(raw: '<a href="/search/buy">Search</a>')
+        end
+
+        I18n.with_locale(:es) do
+          get "/api_public/v1/pages/by_slug/home", params: { include_rendered: "true", locale: "es" }
+          expect(response).to have_http_status(200)
+          json = response.parsed_body
+
+          hero_content = json["page_contents"].find { |c| c["page_part_key"] == "heroes/hero_centered" }
+          # URL should be localized to /es/search/buy
+          expect(hero_content["rendered_html"]).to include('href="/es/search/buy"')
+        end
+      end
+    end
+  end
+
   describe "website not provisioned errors" do
     context "when website has no pages" do
       before do