decidim-core 0.27.4 → 0.28.0.rc4

data/config/routes.rb

@@ -5,6 +5,8 @@ Decidim::Core::Engine.routes.draw do
 
   get "/offline", to: "offline#show"
 
+  get "/favicon.ico", to: "favicon#show"
+
   devise_for :users,
              class_name: "Decidim::User",
              module: :devise,
@@ -109,12 +111,10 @@ Decidim::Core::Engine.routes.draw do
     resources :groups, except: [:destroy, :index, :show] do
       resources :join_requests, only: [:create, :update, :destroy], controller: "user_group_join_requests"
       resources :invites, only: [:index, :create, :update, :destroy], controller: "group_invites"
-      resources :users, only: [:index, :destroy], controller: "group_members", as: "manage_users" do
+      resources :users, only: [:destroy], controller: "group_members", as: "manage_users" do
         member do
           post :promote
         end
-      end
-      resources :admins, only: [:index], controller: "group_admins", as: "manage_admins" do
         member do
           post :demote
         end
@@ -134,8 +134,9 @@ Decidim::Core::Engine.routes.draw do
     get "badges", to: "profiles#badges", as: "profile_badges"
     get "groups", to: "profiles#groups", as: "profile_groups"
     get "members", to: "profiles#members", as: "profile_members"
+    get "group_members", to: "profiles#group_members", as: "profile_group_members"
+    get "group_admins", to: "profiles#group_admins", as: "profile_group_admins"
     get "activity", to: "user_activities#index", as: "profile_activity"
-    get "timeline", to: "user_timeline#index", as: "profile_timeline"
     resources :conversations, except: [:destroy], controller: "user_conversations", as: "profile_conversations"
   end
 
@@ -153,7 +154,7 @@ Decidim::Core::Engine.routes.draw do
   get "/scopes/picker", to: "scopes#picker", as: :scopes_picker
 
   get "/static_map", to: "static_map#show", as: :static_map
-  put "/pages/terms-and-conditions/accept", to: "tos#accept_tos", as: :accept_tos
+  put "/pages/terms-of-service/accept", to: "tos#accept_tos", as: :accept_tos
 
   match "/404", to: "errors#not_found", via: :all
   match "/500", to: "errors#internal_server_error", via: :all

data/db/migrate/20180508111640_add_tos_version_to_organization.rb

@@ -8,8 +8,8 @@ class AddTosVersionToOrganization < ActiveRecord::Migration[5.1]
   def up
     add_column :decidim_organizations, :tos_version, :datetime
     Organization.find_each do |organization|
-      tos_version = Decidim::StaticPage.find_by(slug: "terms-and-conditions", organization: organization).updated_at
-      organization.update(tos_version: tos_version)
+      tos_version = Decidim::StaticPage.find_by(slug: ["terms-and-conditions", "terms-of-service"], organization:).updated_at
+      organization.update(tos_version:)
     end
   end
 

data/db/migrate/20180730071851_add_core_content_blocks.rb

@@ -17,7 +17,7 @@ class AddCoreContentBlocks < ActiveRecord::Migration[5.2]
         weight = (index + 1) * 10
         ContentBlock.create(
           decidim_organization_id: organization_id,
-          weight: weight,
+          weight:,
           scope: :homepage,
           manifest_name: manifest.name,
           published_at: Time.current

data/db/migrate/20180810092428_move_organization_fields_to_hero_content_block.rb

@@ -13,7 +13,7 @@ class MoveOrganizationFieldsToHeroContentBlock < ActiveRecord::Migration[5.2]
   def change
     Decidim::ContentBlock.reset_column_information
     Organization.find_each do |organization|
-      content_block = Decidim::ContentBlock.find_by(organization: organization, scope: :homepage, manifest_name: :hero)
+      content_block = Decidim::ContentBlock.find_by(organization:, scope: :homepage, manifest_name: :hero)
       settings = {}
       welcome_text = organization.welcome_text || {}
       settings = welcome_text.inject(settings) { |acc, (k, v)| acc.update("welcome_text_#{k}" => v) }

data/db/migrate/20181001124950_move_users_groups_to_users_table.rb

@@ -65,7 +65,7 @@ class MoveUsersGroupsToUsersTable < ActiveRecord::Migration[5.2]
       }
       new_attributes = clean_attributes.merge(
         nickname: UserBaseEntity.nicknamize(clean_attributes["name"]),
-        extended_data: extended_data
+        extended_data:
       )
       new_user_group = NewUserGroup.create!(new_attributes)
       new_ids[old_user_group.id] = new_user_group.id

data/db/migrate/20181022090732_add_columns_to_pages.rb

@@ -13,7 +13,7 @@ class AddColumnsToPages < ActiveRecord::Migration[5.2]
     end
 
     Decidim::StaticPage.where(
-      slug: ["faq", "terms-and-conditions", "accessibility"]
+      slug: ["faq", "terms-and-conditions", "terms-of-service", "accessibility"]
     ).update_all(show_in_footer: true)
     # rubocop:enable Rails/SkipsModelValidations
   end

data/db/migrate/20190412131728_fix_user_names.rb

@@ -15,7 +15,7 @@ class FixUserNames < ActiveRecord::Migration[5.2]
     characters_to_remove = "<>?%&^*\#@()[]=+:;\"{}\\|/"
 
     weird_characters.each do |character|
-      UserBaseEntity.where(deleted_at: nil).where("name like '%#{character}%' escape '\' OR nickname like '%#{character}%' escape '\'").find_each do |entity|
+      UserBaseEntity.where(deleted_at: nil).where("name like '%#{character}%' escape '\\' OR nickname like '%#{character}%' escape '\\'").find_each do |entity|
         Rails.logger.debug { "detected character: #{character}" }
         Rails.logger.debug { "UserBaseEntity ID: #{entity.id}" }
         Rails.logger.debug { "#{entity.name} => #{entity.name.delete(characters_to_remove).strip}" }

data/db/migrate/20201128130723_add_allow_public_access_to_static_pages.rb

@@ -7,7 +7,7 @@ class AddAllowPublicAccessToStaticPages < ActiveRecord::Migration[5.2]
     reversible do |direction|
       direction.up do
         # rubocop:disable Rails/SkipsModelValidations
-        Decidim::StaticPage.where(slug: "terms-and-conditions").update_all(
+        Decidim::StaticPage.where(slug: ["terms-and-conditions", "terms-of-service"]).update_all(
           allow_public_access: true
         )
         # rubocop:enable Rails/SkipsModelValidations

data/db/migrate/20220629194812_create_decidim_authorization_transfers.rb

@@ -0,0 +1,20 @@
+# frozen_string_literal: true
+
+class CreateDecidimAuthorizationTransfers < ActiveRecord::Migration[6.1]
+  def change
+    create_table :decidim_authorization_transfers do |t|
+      t.references :user, null: false, foreign_key: { to_table: :decidim_users }, index: true
+      t.references :source_user, null: false, foreign_key: { to_table: :decidim_users }, index: true
+      t.references :authorization, null: false, foreign_key: { to_table: :decidim_authorizations }, index: true
+
+      t.datetime :created_at, null: false
+    end
+
+    create_table :decidim_authorization_transfer_records do |t|
+      t.references :transfer, null: false, foreign_key: { to_table: :decidim_authorization_transfers }, index: true
+      t.references :resource, polymorphic: true, null: false
+
+      t.datetime :created_at, null: false
+    end
+  end
+end

data/db/migrate/20230322101707_rename_terms_of_use.rb

@@ -0,0 +1,23 @@
+# frozen_string_literal: true
+
+class RenameTermsOfUse < ActiveRecord::Migration[6.1]
+  def change
+    rename_column :decidim_organizations, :admin_terms_of_use_body, :admin_terms_of_service_body
+
+    # rubocop:disable Rails/SkipsModelValidations
+    reversible do |dir|
+      dir.up do
+        Decidim::StaticPage.where(slug: "terms-and-conditions").update_all(
+          slug: "terms-of-service"
+        )
+      end
+
+      dir.down do
+        Decidim::StaticPage.where(slug: "terms-of-service").update_all(
+          slug: "terms-and-conditions"
+        )
+      end
+    end
+    # rubocop:enable Rails/SkipsModelValidations
+  end
+end

data/db/migrate/20230409123300_add_content_policy_to_decidim_organizations.rb

@@ -0,0 +1,7 @@
+# frozen_string_literal: true
+
+class AddContentPolicyToDecidimOrganizations < ActiveRecord::Migration[6.1]
+  def change
+    add_column :decidim_organizations, :content_security_policy, :jsonb, default: {}
+  end
+end

data/db/migrate/20230427105700_change_states_on_amendments.rb

@@ -0,0 +1,34 @@
+# frozen_string_literal: true
+
+class ChangeStatesOnAmendments < ActiveRecord::Migration[6.1]
+  class Amendment < ApplicationRecord
+    self.table_name = :decidim_amendments
+    STATES = { draft: 0, evaluating: 10, accepted: 20, rejected: 30, withdrawn: -1 }.freeze
+  end
+
+  def up
+    rename_column :decidim_amendments, :state, :old_state
+    add_column :decidim_amendments, :state, :integer, default: 0, null: false
+
+    Amendment.reset_column_information
+
+    Amendment::STATES.each_pair do |status, index|
+      Amendment.where(old_state: status).update_all(state: index) # rubocop:disable Rails/SkipsModelValidations
+    end
+
+    remove_column :decidim_amendments, :old_state
+  end
+
+  def down
+    rename_column :decidim_amendments, :state, :old_state
+    add_column :decidim_amendments, :state, :string, default: "draft", null: false
+
+    Amendment.reset_column_information
+
+    Amendment::STATES.each_pair do |status, index|
+      Amendment.where(old_state: index).update_all(state: status) # rubocop:disable Rails/SkipsModelValidations
+    end
+
+    remove_column :decidim_amendments, :old_state
+  end
+end

data/db/migrate/20230427105701_change_states_on_reminder_records.rb

@@ -0,0 +1,33 @@
+# frozen_string_literal: true
+
+class ChangeStatesOnReminderRecords < ActiveRecord::Migration[6.1]
+  class ReminderRecord < ApplicationRecord
+    self.table_name = :decidim_reminder_records
+    STATES = { active: 0, pending: 10, completed: 20, deleted: -1 }.freeze
+  end
+
+  def up
+    rename_column :decidim_reminder_records, :state, :old_state
+    add_column :decidim_reminder_records, :state, :integer, default: 0, null: false
+
+    ReminderRecord.reset_column_information
+
+    ReminderRecord::STATES.each_pair do |status, index|
+      ReminderRecord.where(old_state: status).update_all(state: index) # rubocop:disable Rails/SkipsModelValidations
+    end
+
+    remove_column :decidim_reminder_records, :old_state
+  end
+
+  def down
+    rename_column :decidim_reminder_records, :state, :old_state
+    add_column :decidim_reminder_records, :state, :string, default: "draft", null: false
+    ReminderRecord.reset_column_information
+
+    ReminderRecord::STATES.each_pair do |status, index|
+      ReminderRecord.where(old_state: index).update_all(state: status) # rubocop:disable Rails/SkipsModelValidations
+    end
+
+    remove_column :decidim_reminder_records, :old_state
+  end
+end

data/db/migrate/20231027142329_change_default_value_for_decidim_endorsements.rb

@@ -0,0 +1,11 @@
+# frozen_string_literal: true
+
+class ChangeDefaultValueForDecidimEndorsements < ActiveRecord::Migration[6.1]
+  def up
+    change_column_default :decidim_endorsements, :decidim_user_group_id, 0
+  end
+
+  def down
+    change_column_default :decidim_endorsements, :decidim_user_group_id, nil
+  end
+end

data/db/migrate/20231123111229_remove_official_img_header.rb

@@ -0,0 +1,11 @@
+# frozen_string_literal: true
+
+class RemoveOfficialImgHeader < ActiveRecord::Migration[6.1]
+  def up
+    remove_column :decidim_organizations, :official_img_header
+  end
+
+  def down
+    add_column :decidim_organizations, :official_img_header, :string
+  end
+end

data/db/seeds.rb

@@ -1,225 +1,5 @@
 # frozen_string_literal: true
 
-if !Rails.env.production? || ENV.fetch("SEED", nil)
-  print "Creating seeds for decidim-core...\n" unless Rails.env.test?
+require "decidim/core/seeds"
 
-  require "decidim/faker/localized"
-  require "decidim/faker/internet"
-
-  seeds_root = File.join(__dir__, "seeds")
-
-  # Since we usually migrate and seed in the same process, make sure
-  # that we don't have invalid or cached information after a migration.
-  decidim_tables = ActiveRecord::Base.connection.tables.select do |table|
-    table.starts_with?("decidim_")
-  end
-  decidim_tables.map do |table|
-    table.tr("_", "/").classify.safe_constantize
-  end.compact.each(&:reset_column_information)
-
-  smtp_label = ENV.fetch("SMTP_FROM_LABEL", Faker::Twitter.unique.screen_name)
-  smtp_email = ENV.fetch("SMTP_FROM_EMAIL", Faker::Internet.email)
-
-  organization = Decidim::Organization.first || Decidim::Organization.create!(
-    name: Faker::Company.name,
-    twitter_handler: Faker::Hipster.word,
-    facebook_handler: Faker::Hipster.word,
-    instagram_handler: Faker::Hipster.word,
-    youtube_handler: Faker::Hipster.word,
-    github_handler: Faker::Hipster.word,
-    smtp_settings: {
-      from: "#{smtp_label} <#{smtp_email}>",
-      from_email: smtp_email,
-      from_label: smtp_label,
-      user_name: ENV.fetch("SMTP_USERNAME", Faker::Twitter.unique.screen_name),
-      encrypted_password: Decidim::AttributeEncryptor.encrypt(ENV.fetch("SMTP_PASSWORD", Faker::Internet.password(min_length: 8))),
-      address: ENV.fetch("SMTP_ADDRESS", nil) || ENV.fetch("DECIDIM_HOST", "localhost"),
-      port: ENV.fetch("SMTP_PORT", nil) || ENV.fetch("DECIDIM_SMTP_PORT", "25")
-    },
-    host: ENV.fetch("DECIDIM_HOST", "localhost"),
-    external_domain_whitelist: ["decidim.org", "github.com"],
-    description: Decidim::Faker::Localized.wrapped("<p>", "</p>") do
-      Decidim::Faker::Localized.sentence(word_count: 15)
-    end,
-    default_locale: Decidim.default_locale,
-    available_locales: Decidim.available_locales,
-    reference_prefix: Faker::Name.suffix,
-    available_authorizations: Decidim.authorization_workflows.map(&:name),
-    users_registration_mode: :enabled,
-    tos_version: Time.current,
-    badges_enabled: true,
-    user_groups_enabled: true,
-    send_welcome_notification: true,
-    file_upload_settings: Decidim::OrganizationSettings.default(:upload)
-  )
-
-  if organization.top_scopes.none?
-    province = Decidim::ScopeType.create!(
-      name: Decidim::Faker::Localized.literal("province"),
-      plural: Decidim::Faker::Localized.literal("provinces"),
-      organization: organization
-    )
-
-    municipality = Decidim::ScopeType.create!(
-      name: Decidim::Faker::Localized.literal("municipality"),
-      plural: Decidim::Faker::Localized.literal("municipalities"),
-      organization: organization
-    )
-
-    3.times do |time|
-      parent = Decidim::Scope.create!(
-        name: Decidim::Faker::Localized.literal(Faker::Address.unique.state),
-        code: "#{Faker::Address.country_code}_#{time}",
-        scope_type: province,
-        organization: organization
-      )
-
-      5.times do
-        Decidim::Scope.create!(
-          name: Decidim::Faker::Localized.literal(Faker::Address.unique.city),
-          code: "#{parent.code}-#{Faker::Address.unique.state_abbr}",
-          scope_type: municipality,
-          organization: organization,
-          parent: parent
-        )
-      end
-    end
-  end
-
-  territorial = Decidim::AreaType.create!(
-    name: Decidim::Faker::Localized.literal("territorial"),
-    plural: Decidim::Faker::Localized.literal("territorials"),
-    organization: organization
-  )
-
-  sectorial = Decidim::AreaType.create!(
-    name: Decidim::Faker::Localized.literal("sectorials"),
-    plural: Decidim::Faker::Localized.literal("sectorials"),
-    organization: organization
-  )
-
-  3.times do
-    Decidim::Area.create!(
-      name: Decidim::Faker::Localized.word,
-      area_type: territorial,
-      organization: organization
-    )
-  end
-
-  5.times do
-    Decidim::Area.create!(
-      name: Decidim::Faker::Localized.word,
-      area_type: sectorial,
-      organization: organization
-    )
-  end
-
-  admin = Decidim::User.find_or_initialize_by(email: "admin@example.org")
-  admin_hash = {
-    name: Faker::Name.name,
-    nickname: Faker::Twitter.unique.screen_name,
-    organization: organization,
-    confirmed_at: Time.current,
-    locale: I18n.default_locale,
-    admin: true,
-    tos_agreement: true,
-    personal_url: Faker::Internet.url,
-    about: Faker::Lorem.paragraph(sentence_count: 2),
-    accepted_tos_version: organization.tos_version + 1.hour,
-    password_updated_at: Time.current,
-    admin_terms_accepted_at: Time.current
-  }
-  admin_hash.merge!(password: "decidim123456789", password_confirmation: "decidim123456789") if admin.encrypted_password.blank?
-  admin.update!(admin_hash)
-
-  ["user@example.org", "user2@example.org"].each do |email|
-    Decidim::User.find_or_initialize_by(email: email).update!(
-      name: Faker::Name.name,
-      nickname: Faker::Twitter.unique.screen_name,
-      password: "decidim123456789",
-      password_confirmation: "decidim123456789",
-      confirmed_at: Time.current,
-      locale: I18n.default_locale,
-      organization: organization,
-      tos_agreement: true,
-      personal_url: Faker::Internet.url,
-      about: Faker::Lorem.paragraph(sentence_count: 2),
-      accepted_tos_version: organization.tos_version + 1.hour
-    )
-  end
-
-  regular_user = Decidim::User.find_or_initialize_by(email: "user@example.org")
-
-  locked_user = Decidim::User.find_or_initialize_by(email: "locked_user@example.org")
-
-  locked_user.update!(
-    name: Faker::Name.name,
-    nickname: Faker::Twitter.unique.screen_name,
-    password: "decidim123456789",
-    password_confirmation: "decidim123456789",
-    confirmed_at: Time.current,
-    locale: I18n.default_locale,
-    organization: organization,
-    tos_agreement: true,
-    personal_url: Faker::Internet.url,
-    about: Faker::Lorem.paragraph(sentence_count: 2),
-    accepted_tos_version: organization.tos_version + 1.hour
-  )
-
-  locked_user.lock_access!
-
-  Decidim::Messaging::Conversation.start!(
-    originator: admin,
-    interlocutors: [regular_user],
-    body: "Hey! I'm glad you like Decidim"
-  )
-
-  Decidim::User.find_each do |user|
-    [nil, Time.current].each do |verified_at|
-      user_group = Decidim::UserGroup.create!(
-        name: Faker::Company.unique.name,
-        nickname: Faker::Twitter.unique.screen_name,
-        email: Faker::Internet.email,
-        confirmed_at: Time.current,
-        extended_data: {
-          document_number: Faker::Number.number(digits: 10).to_s,
-          phone: Faker::PhoneNumber.phone_number,
-          verified_at: verified_at
-        },
-        decidim_organization_id: user.organization.id
-      )
-
-      Decidim::UserGroupMembership.create!(
-        user: user,
-        role: "creator",
-        user_group: user_group
-      )
-    end
-  end
-
-  oauth_application = Decidim::OAuthApplication.create!(
-    organization: organization,
-    name: "Test OAuth application",
-    organization_name: "Example organization",
-    organization_url: "http://www.example.org",
-    redirect_uri: "https://www.example.org/oauth/decidim",
-    scopes: "public"
-  )
-
-  oauth_application.organization_logo.attach(io: File.open(File.join(seeds_root, "homepage_image.jpg")), filename: "organization_logo.jpg", content_type: "image/jpeg")
-
-  Decidim::System::CreateDefaultContentBlocks.call(organization)
-
-  hero_content_block = Decidim::ContentBlock.find_by(organization: organization, manifest_name: :hero, scope_name: :homepage)
-  hero_content_block.images_container.background_image = ActiveStorage::Blob.create_and_upload!(
-    io: File.open(File.join(seeds_root, "homepage_image.jpg")),
-    filename: "homepage_image.jpg",
-    content_type: "image/jpeg",
-    metadata: nil
-  )
-  settings = {}
-  welcome_text = Decidim::Faker::Localized.sentence(word_count: 5)
-  settings = welcome_text.inject(settings) { |acc, (k, v)| acc.update("welcome_text_#{k}" => v) }
-  hero_content_block.settings = settings
-  hero_content_block.save!
-end
+Decidim::Core::Seeds.new.call if !Rails.env.production? || ENV.fetch("SEED", nil)

data/lib/decidim/acts_as_tree.rb

@@ -37,7 +37,7 @@ module Decidim
     end
 
     def descendants
-      @descendants ||= self_and_descendants.where.not(id: id)
+      @descendants ||= self_and_descendants.where.not(id:)
     end
 
     def self_and_descendants

data/lib/decidim/amendable.rb

@@ -78,7 +78,7 @@ module Decidim
         raise "You must provide a set of fields to amend" unless fields
         raise "You must provide a form class of the amendable" unless form
 
-        @amendable_options = { fields: fields, form: form }
+        @amendable_options = { fields:, form: }
       end
     end
 
@@ -157,7 +157,7 @@ module Decidim
         end
       else # Assume is_a?(Decidim::Coauthorable)
         coauthorships.clear
-        add_coauthor(author, user_group: user_group)
+        add_coauthor(author, user_group:)
       end
     end
 

data/lib/decidim/api/functions/component_finder_base.rb

@@ -22,7 +22,7 @@ module Decidim
       end
 
       def call(component, args, _ctx)
-        @query = { component: component }
+        @query = { component: }
         args.keys.each do |key|
           @query[key] = args[key]
         end

data/lib/decidim/api/functions/component_list.rb

@@ -27,7 +27,7 @@ module Decidim
         @query = Decidim::Component
         # remove default ordering if custom order required
         @query = @query.unscoped if args[:order]
-        @query = @query.where(participatory_space: participatory_space).published
+        @query = @query.where(participatory_space:).published
         add_filter_keys(args[:filter])
         add_order_keys(args[:order].to_h)
         add_default_order

data/lib/decidim/api/functions/component_list_base.rb

@@ -35,7 +35,7 @@ module Decidim
       end
 
       def call(component, args, _ctx)
-        @query = query_scope.where(component: component)
+        @query = query_scope.where(component:)
                             .includes(:component)
 
         add_filter_keys(args[:filter])

data/lib/decidim/api/functions/needs_api_default_order.rb

@@ -13,7 +13,7 @@ module Decidim
       # the results unless the order of the records is explicitly defined.
       #
       # Note that this needs to be called as the last method before returning
-      # the query so that it won't affect the desired ordering specified in the
+      # the query so that it will not affect the desired ordering specified in the
       # GraphQ query. In that case, ordering by ID will be the secondary order
       # of the records.
       def add_default_order

data/lib/decidim/api/functions/participatory_space_list_base.rb

@@ -22,10 +22,19 @@ module Decidim
       end
 
       def call(_obj, args, ctx)
-        @query = model_class.public_spaces.where(
+        base_query = model_class.where(
           organization: ctx[:current_organization]
         )
 
+        @query =
+          if ctx[:current_user]&.admin?
+            base_query
+          elsif base_query.respond_to?(:visible_for)
+            base_query.visible_for(ctx[:current_user])
+          else
+            base_query.public_spaces
+          end
+
         add_filter_keys(args[:filter])
         add_order_keys(args[:order].to_h)
         add_default_order

data/lib/decidim/api/input_filters/category_input_filter.rb

@@ -11,7 +11,7 @@ module Decidim
                description: "Returns the sub-categories for the given parent category or top-level categories if set to `null`",
                required: false,
                prepare: lambda { |parent_id, _ctx|
-                          { parent_id: parent_id }
+                          { parent_id: }
                         }
     end
   end

data/lib/decidim/api/interfaces/categories_container_interface.rb

@@ -12,7 +12,7 @@ module Decidim
       end
 
       def categories(filter: {})
-        CategoryList.new.call(object, { filter: filter }, context)
+        CategoryList.new.call(object, { filter: }, context)
       end
     end
   end

data/lib/decidim/api/interfaces/participatory_space_interface.rb

@@ -13,17 +13,23 @@ module Decidim
 
       field :type, String, description: "The participatory space class name. i.e. Decidim::ParticipatoryProcess", null: false
 
+      field :manifest, Decidim::Core::ParticipatorySpaceManifestType, description: "The manifest information for the participatory space.", null: false
+
       def type
         object.class.name
       end
 
+      def manifest
+        ParticipatorySpaceManifestPresenter.new(object.manifest, object.organization)
+      end
+
       field :components, [ComponentInterface], null: true, description: "Lists the components this space contains." do
         argument :filter, ComponentInputFilter, "Provides several methods to filter the results", required: false
         argument :order, ComponentInputSort, "Provides several methods to order the results", required: false
       end
 
       def components(filter: {}, order: {})
-        ComponentList.new.call(object, { filter: filter, order: order }, context)
+        ComponentList.new.call(object, { filter:, order: }, context)
       end
 
       field :stats, [Decidim::Core::StatisticType, { null: true }], null: true

data/lib/decidim/api/types/attachment_type.rb

@@ -5,9 +5,11 @@ module Decidim
     class AttachmentType < Decidim::Api::Types::BaseObject
       description "A file attachment"
 
+      field :title, Decidim::Core::TranslatedFieldType, "The title of this attachment.", null: false
+      field :description, Decidim::Core::TranslatedFieldType, "The description of this attachment.", null: false
       field :url, GraphQL::Types::String, "The url of this attachment", null: false
       field :type, GraphQL::Types::String, "The type of this attachment", method: :file_type, null: false
-      field :thumbnail, GraphQL::Types::String, "A thumbnail of this attachment, if it's an image.", method: :thumbnail_url, null: true
+      field :thumbnail, GraphQL::Types::String, "A thumbnail of this attachment, if it is an image.", method: :thumbnail_url, null: true
     end
   end
 end