pageflow 17.0.5 → 17.2.0

data/app/views/components/pageflow/admin/entry_translations_tab.rb

@@ -0,0 +1,102 @@
+module Pageflow
+  module Admin
+    # @api private
+    class EntryTranslationsTab < ViewComponent
+      def build(entry)
+        default_translation_hint(entry)
+
+        embedded_index_table(
+          entry.translations.preload(:draft, :published_revision, :translation_group),
+          blank_slate_text: I18n.t('pageflow.admin.entry_translations.no_translations')
+        ) do
+          table_for_collection i18n: Entry do
+            column class: 'publication_state' do |translated_entry|
+              entry_publication_state_indicator(translated_entry)
+            end
+            column :title do |translated_entry|
+              entry_link_or_title(entry, translated_entry)
+            end
+            column :locale do |translated_entry|
+              entry_translation_display_locale(translated_entry)
+            end
+            column do |translated_entry|
+              mark_as_default_link(entry, translated_entry)
+            end
+            column do |translated_entry|
+              remove_link(entry, translated_entry)
+            end
+          end
+        end
+
+        add_link(entry)
+        generate_link(entry)
+      end
+
+      private
+
+      def default_translation_hint(entry)
+        return if entry.translations.none?
+
+        div class: 'side_hint' do
+          para text_node I18n.t('pageflow.admin.entry_translations.default_translation_hint')
+        end
+      end
+
+      def entry_link_or_title(entry, translated_entry)
+        if authorized?(:read, translated_entry) && translated_entry != entry
+          link_to(translated_entry.title,
+                  admin_entry_path(translated_entry, tab: 'translations'))
+        else
+          translated_entry.title
+        end
+      end
+
+      def mark_as_default_link(entry, translated_entry)
+        return unless authorized?(:manage_translations, entry)
+        return if translated_entry.default_translation?
+
+        link_to(
+          I18n.t('pageflow.admin.entry_translations.mark_as_default'),
+          default_admin_entry_translation_path(entry, translated_entry),
+          method: :put,
+          data: {
+            confirm: I18n.t('pageflow.admin.entry_translations.mark_as_default_confirmation')
+          }
+        )
+      end
+
+      def remove_link(entry, translated_entry)
+        return unless authorized?(:manage_translations, entry)
+
+        link_to(
+          I18n.t('pageflow.admin.entry_translations.remove'),
+          admin_entry_translation_path(entry, translated_entry),
+          method: :delete,
+          data: {
+            confirm: I18n.t('pageflow.admin.entry_translations.remove_confirmation')
+          }
+        )
+      end
+
+      def add_link(entry)
+        return unless authorized?(:manage_translations, entry)
+
+        text_node(link_to(t('pageflow.admin.entry_translations.add'),
+                          new_admin_entry_translation_path(entry),
+                          class: 'button'))
+      end
+
+      def generate_link(entry)
+        return unless authorized?(:manage_translations, entry)
+
+        entry_translator_url = Pageflow.config_for(entry).entry_translator_url
+        return if entry_translator_url.blank?
+
+        text_node(link_to(t('pageflow.admin.entry_translations.generate'),
+                          entry_translator_url.call(entry),
+                          class: 'button',
+                          style: 'margin-left: 8px'))
+      end
+    end
+  end
+end

data/app/views/components/pageflow/admin/entry_user_badge_list.rb

@@ -1,5 +1,6 @@
 module Pageflow
   module Admin
+    # @api private
     class EntryUserBadgeList < ViewComponent
       builder_method :entry_user_badge_list
 

data/app/views/components/pageflow/admin/extensible_attributes_table.rb

@@ -1,6 +1,7 @@
 module Pageflow
   module Admin
     class ExtensibleAttributesTable < ActiveAdmin::Views::AttributesTable
+      # @api private
       module BuilderMethods
         def extensible_attributes_table_for(model, additional_rows, &block)
           attributes_table_for(model) do
@@ -11,6 +12,7 @@ module Pageflow
         end
       end
 
+      # @api private
       class RowInserter
         def initialize(context, additional_rows)
           @context = context
@@ -18,9 +20,9 @@ module Pageflow
           @rendered_rows = []
         end
 
-        def row(name, options = {}, &block)
+        def row(name, options = {}, &)
           render_additional_rows(rows_at(:before, name))
-          context.row(name, options, &block)
+          context.row(name, options, &)
           render_additional_rows(rows_at(:after, name))
         end
 
@@ -55,14 +57,15 @@ module Pageflow
         end
       end
 
+      # @api private
       class RowDelegator
         def initialize(context, row_handler)
           @context = context
           @row_handler = row_handler
         end
 
-        def row(name, options = {}, &block)
-          @row_handler.row(name, options, &block)
+        def row(name, options = {}, &)
+          @row_handler.row(name, options, &)
         end
 
         private
@@ -71,17 +74,15 @@ module Pageflow
           @context.respond_to?(name)
         end
 
-        # rubocop:disable Style/MethodMissing
         # Normally we would delegate to super if context does not
         # respond_to? method. But Arbre appears to report not to
         # repond to helpers like authorized? even if it does.
         #
         # This is also the reason we can not use SimpleDelegator here
         # and also delegate_missing in Rails 5 would not work.
-        def method_missing(method, *args, **kwargs, &block)
-          @context.public_send(method, *args, **kwargs, &block)
+        def method_missing(method, ...)
+          @context.public_send(method, ...)
         end
-        # rubocop:enable Style/MethodMissing
       end
     end
   end

data/app/views/components/pageflow/admin/features_tab.rb

@@ -1,5 +1,6 @@
 module Pageflow
   module Admin
+    # @api private
     class FeaturesTab < ViewComponent
       def build(resource)
         feature_target = resource.is_a?(Site) ? resource.account : resource

data/app/views/components/pageflow/admin/grouped_folder_list.rb

@@ -1,5 +1,6 @@
 module Pageflow
   module Admin
+    # @api private
     class GroupedFolderList < ViewComponent
       builder_method :grouped_folder_list
 

data/app/views/components/pageflow/admin/icon_link_to.rb

@@ -1,5 +1,6 @@
 module Pageflow
   module Admin
+    # @api private
     class IconLinkTo < ViewComponent
       builder_method :icon_link_to
 

data/app/views/components/pageflow/admin/members_tab.rb

@@ -1,5 +1,6 @@
 module Pageflow
   module Admin
+    # @api private
     class MembersTab < ViewComponent
       def build(entry)
         embedded_index_table(entry.memberships.includes(:user).references(:users),
@@ -50,9 +51,9 @@ module Pageflow
           para text_node I18n.t('pageflow.admin.resource_tabs.account_editor_hint')
         end
 
-        if authorized?(:add_member_to, entry)
-          add_membership_button(current_user, entry, 'entry')
-        end
+        return unless authorized?(:add_member_to, entry)
+
+        add_membership_button(current_user, entry, 'entry')
       end
     end
   end

data/app/views/components/pageflow/admin/membership_role_with_tooltip.rb

@@ -1,13 +1,14 @@
 module Pageflow
   module Admin
+    # @api private
     class MembershipRoleWithTooltip < ViewComponent
       builder_method :membership_role_with_tooltip
 
       def build(role, options = {})
         span t(role, scope: 'activerecord.values.pageflow/membership.role'),
-        class: "membership_role #{role} tooltip_clue" do
+             class: "membership_role #{role} tooltip_clue" do
           div t(role, scope: tooltip_scope(options)),
-          class: 'tooltip_bubble'
+              class: 'tooltip_bubble'
         end
       end
 

data/app/views/components/pageflow/admin/revisions_tab.rb

@@ -1,15 +1,17 @@
 module Pageflow
   module Admin
+    # @api private
     class RevisionsTab < ViewComponent
       def build(entry)
-        embedded_index_table entry.revisions, :blank_slate_text => I18n.t('pageflow.admin.entries.no_revisions') do
+        embedded_index_table entry.revisions,
+                             blank_slate_text: I18n.t('pageflow.admin.entries.no_revisions') do
           scope(:publications)
           scope(:publications_and_user_snapshots)
           scope(:frozen)
 
-          table_for_collection :class => 'revisions', :i18n => Pageflow::Revision do
+          table_for_collection class: 'revisions', i18n: Pageflow::Revision do
             row_attributes do |revision|
-              {:class => revision_css_class(revision)}
+              {class: revision_css_class(revision)}
             end
 
             column :frozen_at
@@ -56,7 +58,8 @@ module Pageflow
               end
             end
             column do |revision|
-              text_node(link_to(t('pageflow.admin.entries.show'), pageflow.revision_path(revision), :class => 'show'))
+              text_node(link_to(t('pageflow.admin.entries.show'), pageflow.revision_path(revision),
+                                class: 'show'))
               if authorized?(:restore, entry)
                 text_node(link_to(t('pageflow.admin.entries.restore'),
                                   restore_admin_revision_path(revision, params.permit(:tab)),
@@ -72,13 +75,14 @@ module Pageflow
           end
         end
 
-        para(I18n.t('pageflow.admin.entries.published_revision_legend'), :class => 'legend published')
+        para(I18n.t('pageflow.admin.entries.published_revision_legend'),
+             class: 'legend published')
 
-        if authorized?(:snapshot, entry)
-          text_node(button_to(t('pageflow.admin.entries.snapshot'),
-                              snapshot_admin_entry_path(entry, params.permit(:tab)),
-                              method: :post))
-        end
+        return unless authorized?(:snapshot, entry)
+
+        text_node(button_to(t('pageflow.admin.entries.snapshot'),
+                            snapshot_admin_entry_path(entry, params.permit(:tab)),
+                            method: :post))
       end
     end
   end

data/app/views/components/pageflow/admin/sites_tab.rb

@@ -1,5 +1,6 @@
 module Pageflow
   module Admin
+    # @api private
     class SitesTab < ViewComponent
       def build(account)
         embedded_index_table(account.sites,

data/app/views/components/pageflow/admin/tabs_view.rb

@@ -1,5 +1,6 @@
 module Pageflow
   module Admin
+    # @api private
     class TabsView < ViewComponent
       builder_method :tabs_view
 

data/app/views/components/pageflow/admin/timestamp.rb

@@ -1,14 +1,15 @@
 module Pageflow
   module Admin
+    # @api private
     class Timestamp < ViewComponent
       builder_method :timestamp
 
       def build(time)
-        if time
-          span class: 'tooltip_clue' do
-            text_node(l(time, format: time.today? ? :time_today : :date))
-            span(l(time, format: :long), class: 'tooltip_bubble')
-          end
+        return unless time
+
+        span class: 'tooltip_clue' do
+          text_node(l(time, format: time.today? ? :time_today : :date))
+          span(l(time, format: :long), class: 'tooltip_bubble')
         end
       end
 

data/app/views/components/pageflow/admin/user_account_badge_list.rb

@@ -1,14 +1,13 @@
 module Pageflow
   module Admin
+    # @api private
     class UserAccountBadgeList < ViewComponent
       builder_method :user_account_badge_list
 
       def build(user)
         ul class: 'badge_list' do
           user.account_memberships.each do |membership|
-            if authorized?(:see_badge_belonging_to, membership.entity)
-              build_badge(membership)
-            end
+            build_badge(membership) if authorized?(:see_badge_belonging_to, membership.entity)
           end
 
           build_admin_badge if user.admin?

data/app/views/components/pageflow/admin/user_accounts_tab.rb

@@ -1,5 +1,6 @@
 module Pageflow
   module Admin
+    # @api private
     class UserAccountsTab < ViewComponent
       def build(user)
         embedded_index_table(user.memberships.on_accounts

data/app/views/components/pageflow/admin/user_entries_tab.rb

@@ -1,5 +1,6 @@
 module Pageflow
   module Admin
+    # @api private
     class UserEntriesTab < ViewComponent
       def build(user)
         embedded_index_table(user.memberships.on_entries

data/app/views/components/pageflow/admin/users_tab.rb

@@ -1,5 +1,6 @@
 module Pageflow
   module Admin
+    # @api private
     class UsersTab < ViewComponent
       def build(account)
         embedded_index_table(account.memberships.on_accounts.includes(:user)

data/app/views/pageflow/audio_files/_audio_file.json.jbuilder

@@ -1,5 +1,4 @@
 json.call(audio_file, :duration_in_ms)
-json.created_at audio_file.created_at.try(:utc).try(:iso8601, 0)
 
 variants = [
   :m4a,

data/app/views/pageflow/editor/config/_seeds.json.jbuilder

@@ -7,7 +7,7 @@ json.call(Pageflow.config,
           :available_file_licenses,
           :available_text_track_kinds,
           :available_share_providers)
-json.file_types Pageflow.config_for(entry).file_types do |file_type|
+json.file_types entry_config.file_types do |file_type|
   json.collection_name file_type.collection_name
   json.top_level_type file_type.top_level_type
   json.type_name file_type.type_name
@@ -18,9 +18,9 @@ json.file_types Pageflow.config_for(entry).file_types do |file_type|
   end
 end
 
-json.file_importers Pageflow.config_for(entry).file_importers do |file_importer|
+json.file_importers entry_config.file_importers do |file_importer|
   json.importer_name file_importer.name
-  json.logo_source file_importer.logo_source
+  json.logo_source asset_url(file_importer.logo_source)
   json.authentication_provider file_importer.authentication_provider
   json.authentication_required file_importer.authentication_required current_user
 end
@@ -29,3 +29,5 @@ json.default_publisher_meta_tag Pageflow.config.default_publisher_meta_tag
 json.default_keywords_meta_tag Pageflow.config.default_keywords_meta_tag
 json.themes(Pageflow.config.themes, :name, :preview_image_path)
 json.edit_lock_polling_interval_in_seconds Pageflow.config.edit_lock_polling_interval
+
+json.entry_structured_data_types entry_config.entry_structured_data_types.names

data/app/views/pageflow/editor/entries/_entry.json.jbuilder

@@ -11,6 +11,7 @@ json.last_published_with_noindex(entry.last_published_with_noindex?)
 json.metadata do
   json.(entry,
         :title, :locale, :summary, :credits, :author, :publisher, :keywords, :share_providers,
-        :share_url, :share_image_id, :share_image_x, :share_image_y, :configuration)
+        :share_url, :share_image_id, :share_image_x, :share_image_y, :configuration,
+        :structured_data_type_name)
   json.theme_name entry.theme.name
 end

data/app/views/pageflow/editor/entries/index.json.jbuilder

@@ -1 +1 @@
-json.array!(@entries, :partial => 'other_entry', :as => :other_entry)
+json.array!(@entries, partial: 'other_entry', as: :other_entry)

data/app/views/pageflow/editor/entries/seed.json.erb

@@ -10,6 +10,7 @@
   "widgets": <%= widgets_json_seeds(@entry) %>,
 
   "files": <%= editor_files_json_seed(@entry) %>,
+  "file_folders": <%= render_json_seed(@entry.file_folders) %>,
 
   "account": <%= render_json_seed(@entry.account) %>,
   "site": <%= render_json_seed(@entry.site) %>,

data/app/views/pageflow/editor/entries/show.json.jbuilder

@@ -3,7 +3,7 @@ Pageflow.config.file_types.each do |file_type|
     json.cache!(file) do
       json.partial!(object: file,
                     partial: 'pageflow/editor/files/file',
-                    locals: {file_type: file_type},
+                    locals: {file_type:},
                     as: :file)
     end
   end

data/app/views/pageflow/editor/entry_publications/check.json.jbuilder

@@ -13,5 +13,5 @@ json.exhausted_html(render_html_partial('pageflow/editor/quotas/published_entrie
                                         quota: @entry_publication.quota,
                                         account: @entry_publication.quota.account))
 json.published_message_html(render_html_partial('published_message',
-                                                 entry: @entry_publication.entry,
-                                                 account: @entry_publication.quota.account))
+                                                entry: @entry_publication.entry,
+                                                account: @entry_publication.quota.account))

data/app/views/pageflow/editor/file_folders/_file_folder.json.jbuilder

@@ -0,0 +1,5 @@
+json.call(file_folder,
+          :id,
+          :perma_id,
+          :parent_folder_perma_id,
+          :name)

data/app/views/pageflow/editor/file_folders/create.json.jbuilder

@@ -0,0 +1 @@
+json.partial!(partial: 'pageflow/editor/file_folders/file_folder', object: @file_folder)

data/app/views/pageflow/editor/file_folders/index.json.jbuilder

@@ -0,0 +1,3 @@
+json.array!(@file_folders,
+            partial: 'pageflow/editor/file_folders/file_folder',
+            as: :file_folder)

data/app/views/pageflow/editor/files/_file.json.jbuilder

@@ -1,30 +1,23 @@
 json.partial!(partial: 'pageflow/files/file',
               object: file,
-              locals: {file_type: file_type})
+              locals: {file_type:})
 
 json.call(file,
           :state,
           :rights,
-          :usage_id)
+          :usage_id,
+          :folder_perma_id)
 
-if file.can_upload?
-  json.direct_upload_config(file.direct_upload_config)
-end
+json.direct_upload_config(file.direct_upload_config) if file.can_upload?
 
 json.retryable(file.retryable?)
 json.file_name(file.file_name)
 
-if file.url.present?
-  json.url(file.url)
-end
+json.url(file.url) if file.url.present?
 
-if file.original_url.present?
-  json.original_url(file.original_url)
-end
+json.original_url(file.original_url) if file.original_url.present?
 
-if file_type.editor_partial.present?
-  json.partial!(object: file, partial: file_type.editor_partial)
-end
+json.partial!(object: file, partial: file_type.editor_partial) if file_type.editor_partial.present?
 
 if file.respond_to?(:thumbnail_url)
   json.thumbnail_url(asset_path(file.thumbnail_url(:thumbnail)))

data/app/views/pageflow/editor/files/create.json.jbuilder

@@ -1 +1 @@
-json.partial!(partial: 'pageflow/editor/files/file', object: @file, locals: {file_type: file_type})
+json.partial!(partial: 'pageflow/editor/files/file', object: @file, locals: {file_type:})

data/app/views/pageflow/editor/files/index.json.jbuilder

@@ -1,4 +1,4 @@
 json.partial!(collection: @files,
               partial: 'pageflow/editor/files/file',
-              locals: {file_type: file_type},
+              locals: {file_type:},
               as: :file)

data/app/views/pageflow/editor/image_files/_image_file.json.jbuilder

@@ -1,3 +1,6 @@
 json.(image_file, :width, :height, :panorama_url)
 
 json.dimensions(file_dimensions(image_file))
+
+# Uncropped, unlike the thumbnail styles used elsewhere in the editor.
+json.preview_url(image_file.attachment.url(:medium)) if image_file.ready?

data/app/views/pageflow/editor/sites/_site.json.jbuilder

@@ -1 +1,2 @@
+json.(site, :cutoff_mode_name)
 json.pretty_url pretty_site_url(site)

data/app/views/pageflow/editor/video_files/_video_file.json.jbuilder

@@ -10,8 +10,9 @@ if video_file.state == 'encoded'
     json.src source[:high_src]
     json.type source[:type]
   end
-end
 
-if video_file.poster.present?
-  json.poster_url(video_file.poster.url(:large))
+  # Smallest encoded version, sufficient for the preview in the editor.
+  json.preview_url(video_file.mp4_low.url)
 end
+
+json.poster_url(video_file.poster.url(:large)) if video_file.poster.present?

data/app/views/pageflow/entries/share_menu/_bluesky_link.html.erb

@@ -0,0 +1,8 @@
+<%= social_share_link(:bluesky, entry) do %>
+  <svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 512 512">
+    <path d="M111.8 62.2C170.2 105.9 233 194.7 256 242.4c23-47.6 85.8-136.4 144.2-180.2c42.1-31.6 110.3-56 110.3 21.8c0 15.5-8.9 130.5-14.1 149.2C478.2 298 412 314.6 353.1 304.5c102.9 17.5 129.1 75.5 72.5 133.5c-107.4 110.2-154.3-27.6-166.3-62.9l0 0c-1.7-4.9-2.6-7.8-3.3-7.8s-1.6 3-3.3 7.8l0 0c-12 35.3-59 173.1-166.3 62.9c-56.5-58-30.4-116 72.5-133.5C100 314.6 33.8 298 15.7 233.1C10.4 214.4 1.5 99.4 1.5 83.9c0-77.8 68.2-53.4 110.3-21.8z"/>
+  </svg>
+
+  <span class="hint"><%= t('pageflow.public.share_on_bluesky') %></span>
+  <p>Bluesky</p>
+<% end %>

data/app/views/pageflow/entries/share_menu/_threads_link.html.erb

@@ -0,0 +1,8 @@
+<%= social_share_link(:threads, entry) do %>
+  <svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 448 512">
+    <path d="M331.5 235.7c2.2 .9 4.2 1.9 6.3 2.8c29.2 14.1 50.6 35.2 61.8 61.4c15.7 36.5 17.2 95.8-30.3 143.2c-36.2 36.2-80.3 52.5-142.6 53h-.3c-70.2-.5-124.1-24.1-160.4-70.2c-32.3-41-48.9-98.1-49.5-169.6V256v-.2C17 184.3 33.6 127.2 65.9 86.2C102.2 40.1 156.2 16.5 226.4 16h.3c70.3 .5 124.9 24 162.3 69.9c18.4 22.7 32 50 40.6 81.7l-40.4 10.8c-7.1-25.8-17.8-47.8-32.2-65.4c-29.2-35.8-73-54.2-130.5-54.6c-57 .5-100.1 18.8-128.2 54.4C72.1 146.1 58.5 194.3 58 256c.5 61.7 14.1 109.9 40.3 143.3c28 35.6 71.2 53.9 128.2 54.4c51.4-.4 85.4-12.6 113.7-40.9c32.3-32.2 31.7-71.8 21.4-95.9c-6.1-14.2-17.1-26-31.9-34.9c-3.7 26.9-11.8 48.3-24.7 64.8c-17.1 21.8-41.4 33.6-72.7 35.3c-23.6 1.3-46.3-4.4-63.9-16c-20.8-13.8-33-34.8-34.3-59.3c-2.5-48.3 35.7-83 95.2-86.4c21.1-1.2 40.9-.3 59.2 2.8c-2.4-14.8-7.3-26.6-14.6-35.2c-10-11.7-25.6-17.7-46.2-17.8H227c-16.6 0-39 4.6-53.3 26.3l-34.4-23.6c19.2-29.1 50.3-45.1 87.8-45.1h.8c62.6 .4 99.9 39.5 103.7 107.7l-.2 .2zm-156 68.8c1.3 25.1 28.4 36.8 54.6 35.3c25.6-1.4 54.6-11.4 59.5-73.2c-13.2-2.9-27.8-4.4-43.4-4.4c-4.8 0-9.6 .1-14.4 .4c-42.9 2.4-57.2 23.2-56.2 41.8l-.1 .1z"/>
+  </svg>
+
+  <span class="hint"><%= t('pageflow.public.share_on_threads') %></span>
+  <p>Threads</p>
+<% end %>

data/app/views/pageflow/files/_file.json.jbuilder

@@ -8,8 +8,10 @@ json.call(file,
           :parent_file_id,
           :parent_file_model_type)
 
+json.display_name(file.display_name.presence || file.file_name)
+
+json.created_at(file.created_at.try(:utc).try(:iso8601, 0))
+
 json.is_ready(file.ready?)
 
-if file_type.partial.present?
-  json.partial!(object: file, partial: file_type.partial)
-end
+json.partial!(object: file, partial: file_type.partial) if file_type.partial.present?

data/app/views/pageflow/image_files/_image_file.json.jbuilder

@@ -1,3 +1,2 @@
 json.call(image_file, :width, :height)
 json.processed_extension image_file.output_present?(:webp) ? 'webp' : 'JPG'
-json.created_at image_file.created_at.try(:utc).try(:iso8601, 0)

data/app/views/pageflow/review/comment_threads/_comment_thread.json.jbuilder

@@ -0,0 +1,20 @@
+json.key_format!(camelize: :lower)
+
+json.call(comment_thread,
+          :id,
+          :perma_id,
+          :subject_type,
+          :subject_id,
+          :subject_range,
+          :section_perma_id,
+          :creator_id,
+          :resolved_at,
+          :resolved_by_id,
+          :created_at,
+          :updated_at)
+
+json.resolver_name comment_thread.resolver&.full_name
+
+json.comments(comment_thread.comments) do |comment|
+  json.partial!('pageflow/review/comments/comment', comment:)
+end

data/app/views/pageflow/review/comment_threads/create.json.jbuilder

@@ -0,0 +1,2 @@
+json.partial!('pageflow/review/comment_threads/comment_thread',
+              comment_thread: @comment_thread)

data/app/views/pageflow/review/comment_threads/index.json.jbuilder

@@ -0,0 +1,13 @@
+json.key_format!(camelize: :lower)
+
+json.current_user do
+  json.id current_user.id
+  json.name current_user.full_name
+  json.unread_comments_since_at current_user.unread_comments_since_at
+end
+
+json.comment_threads(@comment_threads) do |comment_thread|
+  json.partial!('pageflow/review/comment_threads/comment_thread', comment_thread:)
+end
+
+json.comment_thread_reads(@read_at_by_perma_id.transform_keys(&:to_s))

data/app/views/pageflow/review/comments/_comment.json.jbuilder

@@ -0,0 +1,13 @@
+json.key_format!(camelize: :lower)
+
+json.call(comment,
+          :id,
+          :perma_id,
+          :creator_id,
+          :body,
+          :quote,
+          :created_at,
+          :updated_at,
+          :edited_at)
+
+json.creator_name comment.creator.full_name

data/app/views/pageflow/review/comments/create.json.jbuilder

@@ -0,0 +1 @@
+json.partial!('pageflow/review/comments/comment', comment: @comment)