Custom/0.26

decidim-accountability/app/commands/decidim/accountability/admin/create_timeline_entry.rb

@@ -25,13 +25,14 @@ def call
 
         private
 
-        attr_reader :timeline_entry
+        attr_reader :timeline_entry, :form
 
         def create_timeline_entry
           @timeline_entry = TimelineEntry.create!(
-            decidim_accountability_result_id: @form.decidim_accountability_result_id,
-            entry_date: @form.entry_date,
-            description: @form.description
+            decidim_accountability_result_id: form.decidim_accountability_result_id,
+            entry_date: form.entry_date,
+            title: form.title,
+            description: form.description
           )
         end
       end

decidim-accountability/app/commands/decidim/accountability/admin/update_timeline_entry.rb

@@ -34,8 +34,9 @@ def call
 
         def update_timeline_entry
           timeline_entry.update!(
-            entry_date: @form.entry_date,
-            description: @form.description
+            entry_date: form.entry_date,
+            title: form.title,
+            description: form.description
           )
         end
       end

decidim-accountability/app/forms/decidim/accountability/admin/timeline_entry_form.rb

@@ -10,10 +10,11 @@ class TimelineEntryForm < Decidim::Form
 
         attribute :decidim_accountability_result_id, Integer
         attribute :entry_date, Decidim::Attributes::LocalizedDate
+        translatable_attribute :title, String
         translatable_attribute :description, String
 
         validates :entry_date, presence: true
-        validates :description, translatable_presence: true
+        validates :title, translatable_presence: true
       end
     end
   end

decidim-accountability/app/models/decidim/accountability/timeline_entry.rb

@@ -7,6 +7,7 @@ module Accountability
     class TimelineEntry < Accountability::ApplicationRecord
       include Decidim::TranslatableResource
 
+      translatable_fields :title
       translatable_fields :description
       belongs_to :result, foreign_key: "decidim_accountability_result_id", class_name: "Decidim::Accountability::Result", inverse_of: :timeline_entries
     end

decidim-accountability/app/views/decidim/accountability/admin/timeline_entries/_form.html.erb

@@ -9,7 +9,11 @@
     </div>
 
     <div class="row column">
-      <%= form.translated :text_field, :description %>
+      <%= form.translated :text_field, :title %>
+    </div>
+
+    <div class="row column">
+      <%= form.translated :editor, :description %>
     </div>
   </div>
 </div>

decidim-accountability/app/views/decidim/accountability/admin/timeline_entries/index.html.erb

@@ -13,15 +13,15 @@
         <thead>
           <tr>
             <th><%= t("models.timeline_entry.fields.entry_date", scope: "decidim.accountability") %></th>
-            <th><%= t("models.timeline_entry.fields.description", scope: "decidim.accountability") %></th>
+            <th><%= t("models.timeline_entry.fields.title", scope: "decidim.accountability") %></th>
             <th class="actions"><%= t("actions.title", scope: "decidim.accountability") %></th>
           </tr>
         </thead>
         <tbody>
           <% timeline_entries.each do |timeline_entry| %>
             <tr data-id="<%= timeline_entry.id %>">
               <td><%= timeline_entry.entry_date %><br></td>
-              <td><%= translated_attribute(timeline_entry.description) %></td>
+              <td><%= translated_attribute(timeline_entry.title) %></td>
               <td class="table-list__actions">
                 <% if allowed_to? :update, :timeline_entry, timeline_entry: timeline_entry %>
                   <%= icon_link_to "pencil", edit_result_timeline_entry_path(timeline_entry.result, timeline_entry), t("actions.edit", scope: "decidim.accountability"), class: "action-icon--edit" %>

decidim-accountability/app/views/decidim/accountability/results/_timeline.html.erb

@@ -13,8 +13,14 @@
                 <%= timeline_entry.entry_date %>
               </span>
               <h4 class="timeline__title">
-                <%= translated_attribute timeline_entry.description %>
+                <%= translated_attribute timeline_entry.title %>
               </h4>
+
+              <% if translated_attribute(timeline_entry.description).present? %>
+                <div class="timeline__description__description">
+                  <%== translated_attribute timeline_entry.description %>
+                </div>
+              <% end %>
             </div>
           </div>
         </li>

decidim-accountability/config/locales/en.yml

@@ -22,6 +22,7 @@ en:
       timeline_entry:
         description: Description
         entry_date: Date
+        title: Title
     models:
       decidim/accountability/proposal_linked_event: Proposal included in a result
       decidim/accountability/result_progress_updated_event: Result progress updated
@@ -159,8 +160,8 @@ en:
             progress: Progress
         timeline_entry:
           fields:
-            description: Description
             entry_date: Date
+            title: Title
       result_m:
         executed: Executed
         view: View

decidim-accountability/db/migrate/20220331150008_add_title_to_timeline_entries.rb

@@ -0,0 +1,7 @@
+# frozen_string_literal: true
+
+class AddTitleToTimelineEntries < ActiveRecord::Migration[6.0]
+  def change
+    add_column :decidim_accountability_timeline_entries, :title, :jsonb
+  end
+end

decidim-accountability/db/migrate/20220331150155_move_legacy_description_to_title_of_timeline_entries.rb

@@ -0,0 +1,13 @@
+# frozen_string_literal: true
+
+class MoveLegacyDescriptionToTitleOfTimelineEntries < ActiveRecord::Migration[6.0]
+  class TimelineEntry < ApplicationRecord
+    self.table_name = :decidim_accountability_timeline_entries
+  end
+
+  def up
+    TimelineEntry.find_each do |timeline_entry|
+      timeline_entry.update!(title: timeline_entry.description, description: nil)
+    end
+  end
+end

decidim-accountability/lib/decidim/accountability/component.rb

@@ -159,7 +159,8 @@
           rand(0..5).times do |i|
             child_result.timeline_entries.create!(
               entry_date: child_result.start_date + i.days,
-              description: Decidim::Faker::Localized.sentence(word_count: 2)
+              title: Decidim::Faker::Localized.sentence(word_count: 2),
+              description: Decidim::Faker::Localized.paragraph(sentence_count: 1)
             )
           end
 

decidim-accountability/lib/decidim/accountability/test/factories.rb

@@ -45,6 +45,7 @@
   factory :timeline_entry, class: "Decidim::Accountability::TimelineEntry" do
     result { create(:result) }
     entry_date { "12/7/2017" }
+    title { generate_localized_title }
     description { generate_localized_title }
   end
 end

decidim-accountability/lib/decidim/api/timeline_entry_type.rb

@@ -7,6 +7,7 @@ class TimelineEntryType < Decidim::Api::Types::BaseObject
 
       field :id, GraphQL::Types::ID, "The internal ID for this timeline entry", null: false
       field :entry_date, Decidim::Core::DateType, "The entry date for this timeline entry", null: true
+      field :title, Decidim::Core::TranslatedFieldType, "The title for this timeline entry", null: true
       field :description, Decidim::Core::TranslatedFieldType, "The description for this timeline entry", null: true
       field :created_at, Decidim::Core::DateTimeType, "When this timeline entry was created", null: true
       field :updated_at, Decidim::Core::DateTimeType, "When this timeline entry was updated", null: true

decidim-accountability/spec/commands/admin/create_timeline_entry_spec.rb

@@ -12,13 +12,15 @@ module Decidim::Accountability
     let(:result) { create :result, component: current_component }
 
     let(:date) { "2017-8-23" }
+    let(:title) { "Title" }
     let(:description) { "description" }
 
     let(:form) do
       double(
         invalid?: invalid,
         decidim_accountability_result_id: result.id,
         entry_date: date,
+        title: { en: title },
         description: { en: description }
       )
     end
@@ -44,6 +46,11 @@ module Decidim::Accountability
         expect(timeline_entry.entry_date).to eq(Date.new(2017, 8, 23))
       end
 
+      it "sets the title" do
+        subject.call
+        expect(translated(timeline_entry.title)).to eq title
+      end
+
       it "sets the description" do
         subject.call
         expect(translated(timeline_entry.description)).to eq description

decidim-accountability/spec/commands/admin/update_timeline_entry_spec.rb

@@ -14,12 +14,14 @@ module Decidim::Accountability
     let(:timeline_entry) { create :timeline_entry, result: result }
 
     let(:date) { "2017-9-23" }
+    let(:title) { "New title" }
     let(:description) { "new description" }
 
     let(:form) do
       double(
         invalid?: invalid,
         entry_date: date,
+        title: { en: title },
         description: { en: description }
       )
     end

decidim-accountability/spec/forms/admin/timeline_entry_form_spec.rb

@@ -18,6 +18,9 @@ module Decidim::Accountability
     let(:result) { create :result, component: current_component }
 
     let(:entry_date) { "12/3/2017" }
+    let(:title) do
+      Decidim::Faker::Localized.sentence(word_count: 3)
+    end
     let(:description) do
       Decidim::Faker::Localized.sentence(word_count: 3)
     end
@@ -26,6 +29,7 @@ module Decidim::Accountability
       {
         decidim_accountability_result_id: result.id,
         entry_date: entry_date,
+        title_en: title[:en],
         description_en: description[:en]
       }
     end
@@ -38,10 +42,16 @@ module Decidim::Accountability
       it { is_expected.not_to be_valid }
     end
 
+    describe "when title is missing" do
+      let(:title) { { en: nil } }
+
+      it { is_expected.not_to be_valid }
+    end
+
     describe "when description is missing" do
       let(:description) { { en: nil } }
 
-      it { is_expected.not_to be_valid }
+      it { is_expected.to be_valid }
     end
   end
 end

decidim-accountability/spec/types/integration_schema_spec.rb

@@ -50,6 +50,7 @@
       "timelineEntries" => [
         {
           "createdAt" => result.timeline_entries.first.created_at.iso8601.to_s.gsub("Z", "+00:00"),
+          "title" => { "translation" => result.timeline_entries.first.title[locale] },
           "description" => { "translation" => result.timeline_entries.first.description[locale] },
           "entryDate" => result.timeline_entries.first.entry_date.to_s,
           "id" => result.timeline_entries.first.id.to_s,
@@ -159,6 +160,9 @@
               timelineEntries {
                 id
                 createdAt
+                title {
+                  translation(locale:"#{locale}")
+                }
                 description {
                   translation(locale:"#{locale}")
                 }
@@ -265,6 +269,9 @@
           timelineEntries {
             id
             createdAt
+            title {
+              translation(locale:"#{locale}")
+            }
             description {
               translation(locale:"#{locale}")
             }

decidim-accountability/spec/types/timeline_entry_type_spec.rb

@@ -26,6 +26,14 @@ module Accountability
         end
       end
 
+      describe "title" do
+        let(:query) { '{ title { translation(locale:"en")}}' }
+
+        it "returns the title field" do
+          expect(response["title"]["translation"]).to eq(model.title["en"])
+        end
+      end
+
       describe "description" do
         let(:query) { '{ description { translation(locale:"en")}}' }
 

decidim-admin/app/controllers/concerns/decidim/admin/participatory_space_admin_context.rb

@@ -54,7 +54,12 @@ def permissions_context
       end
 
       def layout
-        current_participatory_space_manifest.context(current_participatory_space_context).layout
+        space_manifest = if current_participatory_space_manifest.is_a? Decidim::ResourceManifest
+                           Decidim.participatory_space_manifests.find { |manifest| manifest.model_class_name == current_participatory_space.class.name }
+                         else
+                           current_participatory_space_manifest
+                         end
+        space_manifest.context(current_participatory_space_context).layout
       end
     end
   end

decidim-core/app/controllers/concerns/decidim/participatory_space_context.rb

@@ -59,7 +59,12 @@ def authorize_participatory_space
     end
 
     def layout
-      current_participatory_space_manifest.context(current_participatory_space_context).layout
+      space_manifest = if current_participatory_space_manifest.is_a? Decidim::ResourceManifest
+                         Decidim.participatory_space_manifests.find { |manifest| manifest.model_class_name == current_participatory_space.class.name }
+                       else
+                         current_participatory_space_manifest
+                       end
+      space_manifest.context(current_participatory_space_context).layout
     end
 
     # Method for current user can visit the space (assembly or proces)

decidim-core/app/helpers/decidim/participatory_space_helpers.rb

@@ -18,7 +18,7 @@ module ParticipatorySpaceHelpers
     def participatory_space_helpers
       return @participatory_space_helpers if defined?(@participatory_space_helpers)
 
-      helper = current_participatory_space_manifest.context(current_participatory_space_context).helper
+      helper = participatory_space_manifest.context(current_participatory_space_context).helper
 
       klass = Class.new(SimpleDelegator) do
         include helper.constantize if helper
@@ -41,5 +41,15 @@ def participatory_space_wrapper(&block)
         concat(capture(&block))
       end
     end
+
+    def participatory_space_manifest
+      @participatory_space_manifest ||= begin
+        if current_participatory_space_manifest.is_a? Decidim::ResourceManifest
+          Decidim.participatory_space_manifests.find { |manifest| manifest.model_class_name == current_participatory_space.class.name }
+        else
+          current_participatory_space_manifest
+        end
+      end
+    end
   end
 end