decidim-core 0.30.0.rc2 → 0.30.0.rc3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 51f519d636709acef9ba64ba8adb8be0c2d554f9f6f38a9e741f607fed90a6da
-  data.tar.gz: bb93e7720dd8f5e005cda7dc6fab14d21443d6fcf9c0ba1428b1887a1a785d5f
+  metadata.gz: 1741dcb0de873587b3e2ff7eb8faae32fa98107e26b42a1c30e1cff7c876f7b3
+  data.tar.gz: 7d5c4a405e277850a00645b6625097cf69a0b4b18dfdfa876be69ca045e71062
 SHA512:
-  metadata.gz: 00663f06f72fc52a152b00aac2d0c680a7b9c8d349f737ea5d8f34970ea7f791d71f224579469cfe9ca4aeee3ad7d2aaa1d33bccd898bbd7cc7fa5b8bf158675
-  data.tar.gz: 1e504e001f0f88b185a8eae321dffe70696bb17055b0826416bdb16ed44ca51d269d8a461cf56318c2278c0eba2976d41f34fa947be2cb2c88d8d49f9bba6acb
+  metadata.gz: fd16b3dbf24ccaf8cced5f0b23eb3dd496e5269bb19fb14010c4db9cc21e01f19c9cd19cff38540d1c9fd474d5e28d3e2a191d9a4e7d2bebde039eff54cdcf1b
+  data.tar.gz: 598051abedb973a6c95e2504d03a30de39f050a57d414aa625c474a2483cc0db42b78e0c923456aab83b960f2ed0dc42983304bc1798d24e9e80a4f08907ad1d

data/app/cells/decidim/amendable/amendments/show.erb

@@ -6,7 +6,7 @@
 
   <div id="amendment-list">
     <% emendations.each do |emendation| %>
-      <%= card_for emendation, hide_voting: true, context: { current_user: } %>
+      <%= card_for emendation, context: { current_user: } %>
     <% end %>
   </div>
 </section>

data/app/cells/decidim/profile_actions/show.erb

@@ -7,6 +7,6 @@
   <%= cell("decidim/follow_button", profile_holder) %>
 
   <% if user_flaggable? && model.try(:id) != current_user.try(:id) %>
-    <%= cell("decidim/report_button", profile_holder) %>
+    <%= cell("decidim/report_user_button", profile_holder) %>
   <% end %>
 </div>

data/app/cells/decidim/report_button/already_reported_modal.erb

@@ -1,10 +1,10 @@
 <%= decidim_modal id: modal_id, class: "flag-modal" do %>
   <div data-dialog-container>
     <%= icon "flag-line" %>
-    <h2 id="dialog-title-flagModal" tabindex="-1" data-dialog-title><%= t("decidim.shared.flag_modal.title") %></h2>
+    <h2 tabindex="-1" data-dialog-title><%= t("decidim.shared.flag_modal.title") %></h2>
     <div>
       <div class="form__wrapper flag-modal__form">
-        <p id="dialog-desc-flagModal" class="flag-modal__form-description"><%= t("decidim.shared.flag_modal.already_reported") %></p>
+        <p class="flag-modal__form-description"><%= t("decidim.shared.flag_modal.already_reported") %></p>
       </div>
     </div>
   </div>

data/app/cells/decidim/report_button/flag_modal.erb

@@ -2,7 +2,7 @@
   <%= form_for report_form, builder:, url: report_path, method: :post, html: { id: nil } do |f| %>
     <div data-dialog-container>
       <%= icon "flag-line" %>
-      <h2 id="dialog-title-<%= modal_id %>" tabindex="-1" data-dialog-title><%= t("decidim.shared.flag_modal.title") %></h2>
+      <h2 tabindex="-1" data-dialog-title><%= t("decidim.shared.flag_modal.title") %></h2>
       <div>
         <div class="form__wrapper flag-modal__form">
           <p id="dialog-desc-<%= modal_id %>" class="flag-modal__form-description"><%= t("decidim.shared.flag_modal.description") %></p>
@@ -18,32 +18,18 @@
           <%= f.text_area :details, rows: 4, label_options: { class: "flag-modal__form-textarea-label", for: nil }, id: nil %>
 
           <% if frontend_administrable? %>
-            <% if user_reportable? %>
-              <% if current_user&.admin? %>
-                <%= f.check_box :block,
-                                label: t("decidim.shared.flag_user_modal.block"),
-                                include_hidden: false,
-                                data: {
-                                  label_action: t("decidim.shared.flag_user_modal.block"),
-                                  label_report: t("decidim.shared.flag_user_modal.report"),
-                                  block: "true"
-                                } %>
-                <%= f.check_box :hide, label: t("decidim.shared.flag_user_modal.hide"), label_options: { class: :invisible, id: "block_and_hide" } %>
-              <% end %>
-            <% else %>
-              <%= f.check_box :hide,
-                              label: t("decidim.shared.flag_modal.hide_content"),
-                              include_hidden: false,
-                              id: hide_checkbox_id,
-                              data: {
-                                label_action: t("decidim.shared.flag_modal.hide"),
-                                label_report: t("decidim.shared.flag_modal.report"),
-                                hide: "true"
-                              },
-                              label_options: {
-                                for: hide_checkbox_id
-                              } %>
-            <% end %>
+            <%= f.check_box :hide,
+                            label: t("decidim.shared.flag_modal.hide_content"),
+                            include_hidden: false,
+                            id: hide_checkbox_id,
+                            data: {
+                              label_action: t("decidim.shared.flag_modal.hide"),
+                              label_report: t("decidim.shared.flag_modal.report"),
+                              hide: "true"
+                            },
+                            label_options: {
+                              for: hide_checkbox_id
+                            } %>
           <% end %>
 
         </div>

data/app/cells/decidim/report_button_cell.rb

@@ -11,8 +11,6 @@ module Decidim
     end
 
     def frontend_administrable?
-      return true if user_reportable? && current_user&.admin?
-
       user_entity? &&
         model.can_be_administered_by?(current_user) &&
         (model.respond_to?(:official?) && !model.official?)
@@ -48,16 +46,12 @@ module Decidim
       options[:modal_id] || "flagModal"
     end
 
-    def user_reportable?
-      model.is_a?(Decidim::UserReportable)
-    end
-
     def report_form
-      @report_form ||= user_reportable? ? Decidim::ReportForm.from_params(reason: "spam") : Decidim::ReportForm.new(reason: "spam")
+      @report_form ||= Decidim::ReportForm.new(reason: "spam")
     end
 
     def report_path
-      @report_path ||= user_reportable? ? decidim.report_user_path(sgid: model.to_sgid.to_s) : decidim.report_path(sgid: model.to_sgid.to_s)
+      @report_path ||= decidim.report_path(sgid: model.to_sgid.to_s)
     end
 
     def builder

data/app/cells/decidim/report_user_button/already_reported_modal.erb

@@ -0,0 +1,11 @@
+<%= decidim_modal id: modal_id, class: "flag-user-modal" do %>
+  <div data-dialog-container>
+    <%= icon "flag-line" %>
+    <h2 tabindex="-1" data-dialog-title><%= t("decidim.shared.flag_user_modal.title") %></h2>
+    <div>
+      <div class="form__wrapper flag-modal__form">
+        <p class="flag-modal__form-description"><%= t("decidim.shared.flag_user_modal.already_reported") %></p>
+      </div>
+    </div>
+  </div>
+<% end %>

data/app/cells/decidim/report_user_button/flag_modal.erb

@@ -0,0 +1,46 @@
+<%= decidim_modal id: modal_id, class: "flag-user-modal" do %>
+  <%= form_for report_form, builder:, url: report_path, method: :post, html: { id: nil } do |f| %>
+    <div data-dialog-container>
+      <%= icon "flag-line" %>
+      <h2 tabindex="-1" data-dialog-title><%= t("decidim.shared.flag_user_modal.title") %></h2>
+      <div>
+        <div class="form__wrapper flag-modal__form">
+          <p class="flag-modal__form-description"><%= t("decidim.shared.flag_user_modal.description") %></p>
+          <p class="flag-modal__form-reason"><%= t("decidim.shared.flag_modal.reason") %>:</p>
+          <%= f.collection_radio_buttons :reason, [
+            [:spam, t("decidim.shared.flag_user_modal.spam")],
+            [:offensive, t("decidim.shared.flag_user_modal.offensive")],
+            [:does_not_belong, t("decidim.shared.flag_user_modal.does_not_belong", organization_name: current_organization_name)]
+          ], :first, :last do |builder|
+            builder.label(for: nil, class: "form__wrapper-checkbox-label") { builder.radio_button(id: nil) + builder.text }
+          end %>
+
+          <%= f.text_area :details, rows: 4, label_options: { class: "flag-modal__form-textarea-label", for: nil }, id: nil %>
+
+          <% if frontend_administrable? %>
+            <%= f.check_box :block,
+                            label: t("decidim.shared.flag_user_modal.block"),
+                            include_hidden: false,
+                            data: {
+                              label_action: t("decidim.shared.flag_user_modal.block"),
+                              label_report: t("decidim.shared.flag_user_modal.report"),
+                              block: "true"
+                            } %>
+            <%= f.check_box :hide, label: t("decidim.shared.flag_user_modal.hide"), label_options: { class: :invisible, id: "block_and_hide" } %>
+          <% end %>
+        </div>
+      </div>
+    </div>
+
+    <div data-dialog-actions>
+      <button type="button" class="button button__lg button__transparent-secondary" data-dialog-close="<%= modal_id %>">
+        <%= t("decidim.shared.confirm_modal.cancel") %>
+      </button>
+
+      <button type="submit" class="button button__lg button__secondary">
+        <span><%= t("decidim.shared.flag_modal.report") %></span>
+        <%= icon "arrow-right-line", class: "fill-current" %>
+      </button>
+    </div>
+  <% end %>
+<% end %>

data/app/cells/decidim/report_user_button/show.erb

@@ -0,0 +1,2 @@
+<%= render :button %>
+<%= flag_modal unless only_button? %>

data/app/cells/decidim/report_user_button_cell.rb

@@ -0,0 +1,59 @@
+# frozen_string_literal: true
+
+module Decidim
+  class ReportUserButtonCell < ButtonCell
+    include ActionView::Helpers::FormOptionsHelper
+
+    def flag_modal
+      return render :already_reported_modal if model.reported_by?(current_user)
+
+      render
+    end
+
+    private
+
+    def report_form
+      @report_form ||= Decidim::ReportForm.from_params(reason: "spam")
+    end
+
+    def report_path
+      @report_path ||= decidim.report_user_path(sgid: model.to_sgid.to_s)
+    end
+
+    def user_reportable?
+      model.is_a?(Decidim::UserReportable)
+    end
+
+    def frontend_administrable?
+      current_user&.admin?
+    end
+
+    def builder
+      Decidim::FormBuilder
+    end
+
+    def only_button?
+      options[:only_button]
+    end
+
+    def modal_id
+      options[:modal_id] || "flagUserModal"
+    end
+
+    def button_classes
+      options[:button_classes] || "button button__sm button__text button__text-secondary"
+    end
+
+    def text
+      t("decidim.shared.flag_modal.report")
+    end
+
+    def icon_name
+      "flag-line"
+    end
+
+    def html_options
+      { data: { "dialog-open": current_user ? modal_id : "loginModal" } }
+    end
+  end
+end

data/app/cells/decidim/resource_types_filter/show.erb

@@ -11,7 +11,7 @@
   <ul id="dropdown-menu-resource">
     <% resource_types.each do |resource_type| %>
       <li role="menuitem">
-        <%= link_to decidim.last_activities_path(filter: { with_resource_type: resource_type[0] } ), class: "filter#{" is-active" if filter_param == resource_type[0]}" do %>
+        <%= link_to filter_url(resource_type[0]), class: "filter#{" is-active" if filter_param == resource_type[0]}" do %>
           <span class="sr-only"><%= resource_type[1] %></span>
           <%= text_with_resource_icon(*resource_type) %>
         <% end %>

data/app/cells/decidim/resource_types_filter_cell.rb

@@ -27,8 +27,12 @@ module Decidim
       options[:id] || "filters"
     end
 
-    def form_path
-      options[:form_path]
+    def filter_url(resource_type)
+      if options[:source] == :last_activities
+        last_activities_path(filter: { with_resource_type: resource_type })
+      else
+        profile_activity_path(nickname: params[:nickname], filter: { resource_type: })
+      end
     end
 
     def filter_param_key
@@ -39,10 +43,6 @@ module Decidim
       @filter_param ||= params.dig(:filter, filter_param_key) || all_types_key
     end
 
-    def filter
-      options[:filter]
-    end
-
     def all_resource_types_option
       [all_types_key, I18n.t("all", scope: "decidim.last_activities")]
     end

data/app/cells/decidim/user_activity/show.erb

@@ -1,5 +1,5 @@
 <div class="profile__activity">
-  <%= cell "decidim/resource_types_filter", resource_types, form_path: url_for, filter: %>
+  <%= cell "decidim/resource_types_filter", resource_types, source: :profile_activity %>
   <div class="profile__activity__container" id="activities-container">
     <% if activities.length == 0 %>
       <%= cell "decidim/announcement", t("decidim.user_activity.index.no_activities_warning") %>

data/app/commands/decidim/create_omniauth_registration.rb

@@ -47,6 +47,8 @@ module Decidim
 
     attr_reader :form, :verified_email
 
+    REGEXP_SANITIZER = /[<>?%&\^*#@()\[\]=+:;"{}\\|]/
+
     def create_or_find_user
       @user = User.find_or_initialize_by(
         email: verified_email,
@@ -65,16 +67,11 @@ module Decidim
         @user.save!
       else
         @user.email = (verified_email || form.email)
-        @user.name = form.name
+        @user.name = form.name.gsub(REGEXP_SANITIZER, "")
         @user.nickname = form.normalized_nickname
         @user.newsletter_notifications_at = nil
         @user.password = SecureRandom.hex
-        if form.avatar_url.present?
-          url = URI.parse(form.avatar_url)
-          filename = File.basename(url.path)
-          file = url.open
-          @user.avatar.attach(io: file, filename:)
-        end
+        attach_avatar(form.avatar_url) if form.avatar_url.present?
         @user.tos_agreement = form.tos_agreement
         @user.accepted_tos_version = Time.current
         raise NeedTosAcceptance if @user.tos_agreement.blank?
@@ -85,6 +82,15 @@ module Decidim
       end
     end
 
+    def attach_avatar(avatar_url)
+      url = URI.parse(avatar_url)
+      filename = File.basename(url.path)
+      file = url.open
+      @user.avatar.attach(io: file, filename:)
+    rescue OpenURI::HTTPError, Errno::ECONNREFUSED
+      # Do not attach the avatar, as it fails to fetch it.
+    end
+
     def create_identity
       @user.identities.create!(
         provider: form.provider,
@@ -130,7 +136,7 @@ module Decidim
         provider: form.provider,
         uid: form.uid,
         email: form.email,
-        name: form.name,
+        name: form.name.gsub(REGEXP_SANITIZER, ""),
         nickname: form.normalized_nickname,
         avatar_url: form.avatar_url,
         raw_data: form.raw_data,

data/app/controllers/decidim/profiles_controller.rb

@@ -24,7 +24,7 @@ module Decidim
     def show
       return redirect_to profile_members_path if profile_holder.is_a?(Decidim::UserGroup)
 
-      redirect_to profile_activity_path(nickname: params[:nickname])
+      redirect_to profile_activity_path(nickname: params[:nickname].downcase)
     end
 
     def tooltip
@@ -116,7 +116,7 @@ module Decidim
     def profile_holder
       return if params[:nickname].blank?
 
-      @profile_holder ||= Decidim::UserBaseEntity.find_by("LOWER(nickname) = ? AND decidim_organization_id = ?", params[:nickname].downcase, current_organization.id)
+      @profile_holder ||= Decidim::UserBaseEntity.find_by("nickname = ? AND decidim_organization_id = ?", params[:nickname].downcase, current_organization.id)
     end
   end
 end

data/app/controllers/decidim/user_activities_controller.rb

@@ -22,7 +22,7 @@ module Decidim
     def user
       return unless params[:nickname]
 
-      @user ||= current_organization.users.find_by("LOWER(nickname) = ?", params[:nickname].downcase)
+      @user ||= current_organization.users.find_by("nickname = ?", params[:nickname].downcase)
     end
 
     def activities

data/app/forms/decidim/account_form.rb

@@ -21,7 +21,7 @@ module Decidim
 
     validates :name, presence: true, format: { with: Decidim::User::REGEXP_NAME }
     validates :email, presence: true, "valid_email_2/email": { disposable: true }
-    validates :nickname, presence: true, format: { with: Decidim::User::REGEXP_NICKNAME }
+    validates :nickname, presence: true, format: { with: Decidim::UserBaseEntity::REGEXP_NAME }
 
     validates :nickname, length: { maximum: Decidim::User.nickname_max_length, allow_blank: true }
     validates :password, password: { name: :name, email: :email, username: :nickname }, if: -> { password.present? }
@@ -66,7 +66,7 @@ module Decidim
 
     def unique_nickname
       return true if Decidim::UserBaseEntity.where(
-        "decidim_organization_id = ? AND LOWER(nickname) = ? ",
+        "decidim_organization_id = ? AND nickname = ? ",
         context.current_organization.id,
         nickname.downcase
       ).where.not(id: context.current_user.id).empty?

data/app/jobs/decidim/hide_child_resources_job.rb

@@ -4,9 +4,9 @@ module Decidim
   class HideChildResourcesJob < ApplicationJob
     queue_as :user_report
 
-    def perform(resource, _user_id)
-      spam_user = (resource.organization.users.find_by!(email: Decidim::Ai::SpamDetection.reporting_user_email) if Decidim.module_installed?(:ai))
-      spam_user = resource.organization.admins.first if spam_user.nil?
+    def perform(resource, user_id)
+      spam_user = (resource.organization.users.find_by(email: Decidim::Ai::SpamDetection.reporting_user_email) if Decidim.module_installed?(:ai))
+      spam_user = resource.organization.admins.find(user_id) if spam_user.nil?
 
       tool = Decidim::ModerationTools.new(resource, spam_user)
 

data/app/jobs/decidim/migrate/paper_trail_job.rb

@@ -0,0 +1,33 @@
+# frozen_string_literal: true
+
+module Decidim
+  module Migrate
+    # this job is created to help migrating the Paperclip from YAML to JSON
+    class PaperTrailJob < Decidim::ApplicationJob
+      def perform(id)
+        version = PaperTrail::Version.where.not(old_object_changes: nil).find(id)
+        # This is an adaptation of PaperTrail internal load_changeset method,having in mind that we
+        # need to call also the code from PaperTrail::AttributeSerializer::ObjectChangesAttribute
+        object_changes = ActiveSupport::HashWithIndifferentAccess.new(YAML.unsafe_load(version.old_object_changes))
+        unless version.item_type.constantize.unscoped.find_by(id: version.item_id).nil?
+          # This is the deserialization code from `PaperTrail::AttributeSerializer::ObjectChangesAttribute`
+          # where we skip checking the object changeset column type, as we migrate it from YAML to JSON
+          changes_to_serialize = object_changes.clone
+          if changes_to_serialize.present?
+            serializer = PaperTrail::AttributeSerializers::CastAttributeSerializer.new(version.item_type.constantize)
+            changes_to_serialize.each do |key, change|
+              # `change` is an Array with two elements, representing before and after.
+              object_changes[key] = Array(change).map do |value|
+                serializer.send(:deserialize, key, value)
+              end
+            end
+          end
+        end
+
+        version.update_columns(old_object_changes: nil, object_changes:) # rubocop:disable Rails/SkipsModelValidations
+      rescue NameError
+        Rails.logger.info "Skipping History of #{version.item_type} with id #{version.item_id}"
+      end
+    end
+  end
+end

data/app/models/decidim/user.rb

@@ -13,8 +13,6 @@ module Decidim
     include Decidim::UserReportable
     include Decidim::Traceable
 
-    REGEXP_NICKNAME = /\A[\w-]+\z/
-
     class Roles
       def self.all
         Decidim.config.user_roles
@@ -53,8 +51,6 @@ module Decidim
 
     has_one_attached :download_your_data_file
 
-    scope :not_deleted, -> { where(deleted_at: nil) }
-
     scope :managed, -> { where(managed: true) }
     scope :not_managed, -> { where(managed: false) }
 

data/app/models/decidim/user_base_entity.rb

@@ -21,11 +21,13 @@ module Decidim
 
     # Regex for name & nickname format validations
     REGEXP_NAME = /\A(?!.*[<>?%&\^*#@()\[\]=+:;"{}\\|])/
+    REGEXP_NICKNAME = /\A[a-z0-9_-]+\z/
 
     has_one_attached :avatar
     validates_avatar :avatar, uploader: Decidim::AvatarUploader
 
     validates :name, format: { with: REGEXP_NAME }
+    validates :nickname, format: { with: REGEXP_NICKNAME }, unless: -> { deleted? || managed? }
 
     scope :confirmed, -> { where.not(confirmed_at: nil) }
     scope :not_confirmed, -> { where(confirmed_at: nil) }
@@ -34,6 +36,8 @@ module Decidim
     scope :not_blocked, -> { where(blocked: false) }
     scope :available, -> { where(deleted_at: nil, blocked: false, managed: false) }
 
+    scope :not_deleted, -> { where(deleted_at: nil) }
+
     # Public: Returns a collection with all the public entities this user is following.
     #
     # This cannot be done as with a `has_many :following, through: :following_follows`

data/app/packs/src/decidim/sticky_footer.js

@@ -8,12 +8,27 @@
 
 const footer = document.querySelector("footer");
 const stickyButtons = document.querySelector("[data-sticky-buttons]");
+import { screens } from "tailwindcss/defaultTheme"
 
+/**
+ * Checks if a key is in the current viewport
+ *
+ * @param {('sm'|'md'|'lg'|'xl'|'2xl')} key - The key to check the screen size.
+ * @returns {boolean} - Returns true if the screen size corresponds with the key
+ */
+const isScreenSize = (key) => {
+  return window.matchMedia(`(min-width: ${screens[key]})`).matches;
+}
 const adjustCtasButtons = () => {
   if (!stickyButtons) {
     return;
   }
 
+  if (isScreenSize("md")) {
+    footer.style.marginBottom = "0px";
+    return;
+  }
+
   const marginBottom = stickyButtons.offsetHeight;
   footer.style.marginBottom = `${marginBottom}px`;
 };
@@ -26,4 +41,8 @@ if (stickyButtons) {
   document.addEventListener("on:toggle", () => {
     adjustCtasButtons();
   });
+
+  window.addEventListener("resize", () => {
+    adjustCtasButtons();
+  });
 }

data/app/packs/stylesheets/decidim/_cards.scss

@@ -151,6 +151,10 @@
       }
     }
 
+    & > div.success {
+      @apply text-sm text-success;
+    }
+
     [data-author] + [data-author] {
       @apply -ml-4;
     }

data/app/packs/stylesheets/decidim/_content_blocks.scss

@@ -30,7 +30,7 @@
   }
 
   &__span {
-    @apply text-gray-2 font-semibold uppercase;
+    @apply mb-4 text-gray-2 font-semibold uppercase;
   }
 
   & ~ & {

data/app/packs/stylesheets/decidim/_login.scss

@@ -32,6 +32,10 @@ $google_background_hover: #dcdcdc; // 30% opacity
 
       &--developer {
         @apply flex;
+
+        svg {
+          @apply fill-current;
+        }
       }
 
       &--facebook {

data/app/packs/stylesheets/decidim/_modal_authorization.scss

@@ -15,7 +15,7 @@
     }
 
     &-container {
-      @apply mt-8 space-y-8;
+      @apply m-8 space-y-8;
     }
   }
 }

data/app/packs/stylesheets/decidim/_profile.scss

@@ -45,7 +45,7 @@
 
   &__actions {
     &-main {
-      @apply w-fit mx-auto gap-x-4 gap-y-6 md:gap-4;
+      @apply grid w-fit mx-auto gap-x-4 gap-y-6 md:gap-4;
 
       &__dropdown {
         @apply divide-y divide-gray-3 z-20 w-64;

data/app/resolvers/decidim/meta_image_url_resolver.rb

@@ -25,6 +25,7 @@ module Decidim
     # @return [String, nil] - The resolved image blob or nil if no image is found.
     def resolve
       return unless blob
+      return unless blob.service.exist?(blob.key)
 
       resized_variant = blob.variant(resize_to_limit: [1200, 630]).processed
       Rails.application.routes.url_helpers.rails_representation_url(resized_variant, only_path: true)

data/app/services/decidim/static_map_generator.rb

@@ -15,7 +15,7 @@ module Decidim
     def data
       return if @resource.blank? || map_utility.nil?
 
-      Rails.cache.fetch(@resource.cache_key) do
+      Rails.cache.fetch(@resource.cache_key_with_version) do
         map_utility.image_data(
           latitude: @resource.latitude,
           longitude: @resource.longitude,

data/app/views/decidim/devise/registrations/new.html.erb

@@ -4,7 +4,7 @@
   <%= render "decidim/devise/shared/links" %>
 <% end %>
 
-<%= render(layout: "layouts/decidim/shared/layout_center", locals: { columns: 8 }) do %>
+<%= render(layout: "layouts/decidim/shared/layout_center") do %>
   <div class="flex justify-center">
     <h1 class="title-decorator my-12"><%= t("decidim.devise.registrations.new.sign_up") %></h1>
   </div>

data/app/views/decidim/devise/sessions/new.html.erb

@@ -1,6 +1,6 @@
 <% add_decidim_page_title(t("devise.sessions.new.log_in")) %>
 
-<%= render(layout: "layouts/decidim/shared/layout_center", locals: { columns: 8 }) do %>
+<%= render(layout: "layouts/decidim/shared/layout_center") do %>
 
   <div class="flex justify-center">
     <h1 class="title-decorator my-12"><%= t("devise.sessions.new.log_in") %></h1>

data/app/views/decidim/devise/shared/_omniauth_buttons.html.erb

@@ -5,7 +5,7 @@
       <% link_classes = "login__omniauth-button login__omniauth-button--#{normalize_provider_name(provider)}" %>
       <%= link_to decidim.send("user_#{provider}_omniauth_authorize_path"), class: link_classes, method: :post, title: t("devise.shared.links.log_in_with_provider", provider: normalize_provider_name(provider).titleize) do %>
         <%= oauth_icon provider %>
-        <span><%= t("devise.shared.links.log_in_with_provider", provider: normalize_provider_name(provider).titleize) %></span>
+        <span class="sr-only"><%= t("devise.shared.links.log_in_with_provider", provider: normalize_provider_name(provider).titleize) %></span>
       <% end %>
     <% end %>
   </div>

data/app/views/decidim/last_activities/index.html.erb

@@ -5,7 +5,7 @@
   <h1 class="title-decorator my-12"><%= t("last_activity", scope: "decidim.last_activities.index") %></h1>
 
   <div class="profile__activity pb-16">
-    <%= cell "decidim/resource_types_filter", resource_types, form_path: last_activities_path, filter_param_key: :with_resource_type, filter: %>
+    <%= cell "decidim/resource_types_filter", resource_types, source: :last_activities, filter_param_key: :with_resource_type %>
     <div id="activities">
       <%= render partial: "activities" %>
     </div>