decidim-system 0.27.4 → 0.28.0.rc4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f7ba579f2499dcccfeca4684a045a58cab4118e9633b4011bb3dcdbff1a3bb49
-  data.tar.gz: 879336e7f30271b5f08475787f55fd10072b9aa7883fc0b01be59fc7c72ba4cf
+  metadata.gz: 70370fff5369d1099a226723a77145a27c19ee6ff1e214ff4bb6d8fc445934bd
+  data.tar.gz: 9d38c13b4de6dc0348aeeb2d38038e73241aebf162f967aa2d706e57f20f856c
 SHA512:
-  metadata.gz: 455121ed096cba3e48fdab65a0b9272f837cf66688a1c742f0656ad3acb6c924d34365be5cfd2b376f5e6feaf016e153d088a195a43a3b9cf0adc82be38b5545
-  data.tar.gz: 90274666e74627f0530454f895e67ee273ff983993da5e67cec4c0e2ce7b958c3f0bf4c591b445006de39a4e7c1da29aff6ec40a4a8a57b30297d954096bd554
+  metadata.gz: c81b3698abfbfda68d44bd5c5c33609358ce58d63d2b7bbbad7fa7eca463ac72b3a9bf36d58316f1bc9a11083e6a48c25fe7450518425b6106fc4b1eb03eac6e
+  data.tar.gz: 55060e2ef6cd99807b4c9e61ea110e7cbdc43724ef1963a54c1011fbfcabd7f41140c9db0d523d78a25ead5d3ce7b053663d36bc70f90ed0b268ca7ac2010aec

data/README.md

@@ -27,9 +27,9 @@ When using Decidim as multi-tenant, you should keep these in mind:
 
 * All organizations share the same database.
 * Each organization must have a different hostname.
-* Users aren't shared between each organization (the same email can be registered in different organizations and it will be considered as different users).
+* Users are not shared between each organization (the same email can be registered in different organizations and it will be considered as different users).
 * All configuration related to Decidim (`Decidim.config`) is shared between the organizations.
-* Stylesheets aren't customizable per-tenant so UI styles (colors and other variables) are shared.
+* Stylesheets are not customizable per-tenant so UI styles (colors and other variables) are shared.
 
 ## Glossary
 
@@ -40,26 +40,29 @@ When using Decidim as multi-tenant, you should keep these in mind:
 
 ## Managing System admins
 
-For logging in to this dashboard, you'll need to create a system admin account from your terminal:
+For logging in to this dashboard, you will need to create a system admin account from your terminal:
 
 ```bash
 bin/rails decidim_system:create_admin
 ```
 
-You'll be asked for an email and a password. For security, the password will not get displayed back at you and you'll need to confirm it.
+You will be asked for an email and a password. For security, the password will not get displayed back at you and you will need to confirm it.
 
 Once you have created your first admin you can access the system dashboard at `/system`. For instance, if you have Decidim running at `https://example.org`, this URL would be `https://example.org/system`.
-You'll be able to login with your newly created user.
+You will be able to login with your newly created user.
 
 From the system dashboard you can add new admins.
 
-⚠️ If you need to reset your administrator password you'll need to do it by entering the Rails console and changing it manually. ⚠️
+⚠️ If you need to reset your administrator password you will need to do it by entering the Rails console and changing it manually. ⚠️
 
 . Open the rails console:
+
 ```bash
 bin/rails console
 ```
+
 . Run the following instructions, changing them accordingly:
+
 ```ruby
 system_admin = Decidim::System::Admin.order(:id).first                        # for the first system admin
 system_admin = Decidim::System::Admin.find_by_email "system@example.org"      # if you already know the email
@@ -73,7 +76,7 @@ system_admin.save
 Once you have your system admin setup you can also start managing the organizations in your deploy. To do it, login at the system dashboard and create a new organization
 following the form instructions. After creating it, a new admin user will be created and invited to start managing it.
 
-Remember that System admins and regular Admins are completely different users (they don't even share the same database table), so you can't use your
+Remember that System admins and regular Admins are completely different users (they do not even share the same database table), so you cannot use your
 system user to login in as an organization admin.
 
 ## Contributing

data/app/commands/decidim/system/create_admin.rb

@@ -15,7 +15,7 @@ module Decidim
       # Executes the command. Broadcasts these events:
       #
       # - :ok when everything is valid.
-      # - :invalid if the form wasn't valid and we couldn't proceed.
+      # - :invalid if the form was not valid and we could not proceed.
       #
       # Returns nothing.
       def call

data/app/commands/decidim/system/create_default_content_blocks.rb

@@ -19,24 +19,11 @@ module Decidim
       #
       # Returns nothing.
       def call
-        content_blocks.each_with_index do |manifest, index|
-          weight = (index + 1) * 10
-          Decidim::ContentBlock.create(
-            decidim_organization_id: organization.id,
-            weight: weight,
-            scope_name: :homepage,
-            manifest_name: manifest.name,
-            published_at: Time.current
-          )
-        end
+        Decidim::ContentBlocksCreator.new(organization).create_default!
       end
 
       private
 
-      def content_blocks
-        Decidim.content_blocks.for(:homepage).select(&:default)
-      end
-
       attr_reader :organization
     end
   end

data/app/commands/decidim/system/create_default_pages.rb

@@ -14,16 +14,20 @@ module Decidim
       end
 
       # Executes the command.
+      # i18n-tasks-use t('decidim.system.default_pages.terms-of-service')
       #
       # Returns nothing.
       def call
         Decidim::StaticPage::DEFAULT_PAGES.map do |slug|
-          Decidim::StaticPage.find_or_create_by!(organization: organization, slug: slug) do |page|
-            page.title = localized_attribute(slug, :title)
-            page.content = localized_attribute(slug, :content)
+          static_page = Decidim::StaticPage.find_or_create_by!(organization:, slug:) do |page|
+            translated_slug = I18n.t(slug, scope: "decidim.system.default_pages")
+            page.title = localized_attribute(translated_slug, :title)
+            page.content = localized_attribute(translated_slug, :content)
             page.show_in_footer = true
-            page.allow_public_access = true if slug == "terms-and-conditions"
+            page.allow_public_access = true if slug == "terms-of-service"
           end
+
+          create_summary_content_blocks_for(static_page) if slug == "terms-of-service"
         end
       end
 
@@ -40,6 +44,21 @@ module Decidim
           result.update(locale => text)
         end
       end
+
+      def create_summary_content_blocks_for(page)
+        content_block_summary = Decidim::ContentBlock.create(
+          organization:,
+          scope_name: :static_page,
+          manifest_name: :summary,
+          weight: 1,
+          scoped_resource_id: page.id,
+          published_at: Time.current
+        )
+
+        translated_slug = I18n.t(page.slug, scope: "decidim.system.default_pages")
+        content_block_summary.settings = { summary: localized_attribute(translated_slug, :summary) }
+        content_block_summary.save!
+      end
     end
   end
 end

data/app/commands/decidim/system/populate_help.rb

@@ -27,7 +27,7 @@ module Decidim
             slug: "help",
             title: multi_translation("decidim.help.main_topic.default_page.title", organization: @organization.name),
             content: multi_translation("decidim.help.main_topic.default_page.content", organization: @organization.name),
-            topic: topic,
+            topic:,
             organization: @organization,
             weight: 0
           )
@@ -40,7 +40,7 @@ module Decidim
               title: multi_translation("#{scope}.title"),
               content: multi_translation("#{scope}.page"),
               slug: manifest.name,
-              topic: topic,
+              topic:,
               organization: @organization
             )
 

data/app/commands/decidim/system/register_organization.rb

@@ -16,7 +16,7 @@ module Decidim
       # Executes the command. Broadcasts these events:
       #
       # - :ok when everything is valid.
-      # - :invalid if the form wasn't valid and we couldn't proceed.
+      # - :invalid if the form was not valid and we could not proceed.
       #
       # Returns nothing.
       def call
@@ -64,10 +64,23 @@ module Decidim
           omniauth_settings: form.encrypted_omniauth_settings,
           smtp_settings: form.encrypted_smtp_settings,
           send_welcome_notification: true,
-          file_upload_settings: form.file_upload_settings.final
+          file_upload_settings: form.file_upload_settings.final,
+          colors: default_colors,
+          content_security_policy: form.content_security_policy
         )
       end
 
+      def default_colors
+        {
+          alert: "#ec5840",
+          primary: "#53bf40",
+          success: "#57d685",
+          warning: "#ffae00",
+          tertiary: "#bf4053",
+          secondary: "#4053bf"
+        }
+      end
+
       def invite_user_form(organization)
         Decidim::InviteUserForm.from_params(
           name: form.organization_admin_name,

data/app/commands/decidim/system/update_admin.rb

@@ -16,7 +16,7 @@ module Decidim
       # Executes the command. Broadcasts these events:
       #
       # - :ok when everything is valid.
-      # - :invalid if the form wasn't valid and we couldn't proceed.
+      # - :invalid if the form was not valid and we could not proceed.
       #
       # Returns nothing.
       def call

data/app/commands/decidim/system/update_organization.rb

@@ -17,7 +17,7 @@ module Decidim
       # Executes the command. Broadcasts these events:
       #
       # - :ok when everything is valid.
-      # - :invalid if the form wasn't valid and we couldn't proceed.
+      # - :invalid if the form was not valid and we could not proceed.
       #
       # Returns nothing.
       def call
@@ -53,6 +53,7 @@ module Decidim
         organization.omniauth_settings = form.encrypted_omniauth_settings
         organization.smtp_settings = form.encrypted_smtp_settings
         organization.file_upload_settings = form.file_upload_settings.final
+        organization.content_security_policy = form.content_security_policy
 
         organization.save!
       end

data/app/controllers/decidim/system/application_controller.rb

@@ -6,11 +6,12 @@ module Decidim
     class ApplicationController < ActionController::Base
       include FormFactory
       include PayloadInfo
-      include HttpCachingDisabler
+      include Headers::HttpCachingDisabler
       include DisableRedirectionToExternalHost
 
       protect_from_forgery with: :exception, prepend: true
 
+      helper Decidim::MetaTagsHelper
       helper Decidim::TranslationsHelper
       helper Decidim::DecidimFormHelper
       helper Decidim::ReplaceButtonsHelper

data/app/controllers/decidim/system/dashboard_controller.rb

@@ -7,6 +7,7 @@ module Decidim
 
       def show
         @organizations = Organization.all
+        @admins = Admin.all
       end
 
       def check_organizations_presence

data/app/controllers/decidim/system/organizations_controller.rb

@@ -9,7 +9,7 @@ module Decidim
       helper Decidim::OmniauthHelper
 
       def new
-        @form = form(RegisterOrganizationForm).instance
+        @form = form(RegisterOrganizationForm).from_params(default_params)
         @form.file_upload_settings = form(FileUploadSettingsForm).from_model({})
       end
 
@@ -18,7 +18,7 @@ module Decidim
 
         RegisterOrganization.call(@form) do
           on(:ok) do
-            flash[:notice] = t("organizations.create.success", scope: "decidim.system")
+            flash[:notice] = t("organizations.create.success_html", scope: "decidim.system", host: @form.host, email: @form.organization_admin_email)
             redirect_to organizations_path
           end
 
@@ -38,8 +38,8 @@ module Decidim
       end
 
       def edit
-        organization = Organization.find(params[:id])
-        @form = form(UpdateOrganizationForm).from_model(organization)
+        @organization = Organization.find(params[:id])
+        @form = form(UpdateOrganizationForm).from_model(@organization)
       end
 
       def update
@@ -58,8 +58,34 @@ module Decidim
         end
       end
 
+      def resend_invitation
+        organization = Organization.find(params[:id])
+        InviteUserAgain.call(organization.users.first, "invite_admin") do
+          on(:ok) do
+            flash[:notice] = t("organizations.resend_invitation.success", scope: "decidim.system")
+          end
+
+          on(:invalid) do
+            flash[:alert] = I18n.t("organizations.resend_invitation.error", scope: "decidim.system")
+          end
+        end
+
+        redirect_to organizations_path
+      end
+
       private
 
+      def default_params
+        {
+          host: request.host,
+          organization_admin_name: current_admin.email.split("@")[0],
+          organization_admin_email: current_admin.email,
+          available_locales: Decidim.available_locales.map(&:to_s),
+          default_locale: Decidim.default_locale,
+          users_registration_mode: "enabled"
+        }
+      end
+
       # The current organization for the request.
       #
       # Returns an Organization.

data/app/forms/decidim/system/admin_form.rb

@@ -12,7 +12,7 @@ module Decidim
       attribute :password_confirmation, String
 
       validates :email, presence: true
-      validates :password, presence: true, unless: :admin_exists?
+      validates :password, confirmation: true, presence: true, unless: :admin_exists?
       validates :password_confirmation, presence: true, unless: :admin_exists?
 
       validates :password, password: { email: :email }
@@ -22,7 +22,7 @@ module Decidim
       private
 
       def email_uniqueness
-        return unless Admin.where(email: email).where.not(id: id).any?
+        return unless Admin.where(email:).where.not(id:).any?
 
         errors.add(:email, I18n.t("models.admin.validations.email_uniqueness",
                                   scope: "decidim.system"))

data/app/forms/decidim/system/update_organization_form.rb

@@ -29,6 +29,18 @@ module Decidim
         [:authentication, String],
         [:enable_starttls_auto, Boolean]
       ]
+
+      jsonb_attribute :content_security_policy, [
+        [:"default-src", String],
+        [:"img-src", String],
+        [:"media-src", String],
+        [:"script-src", String],
+        [:"style-src", String],
+        [:"frame-src", String],
+        [:"font-src", String],
+        [:"connect-src", String]
+      ]
+
       attribute :password, String
       attribute :file_upload_settings, FileUploadSettingsForm
 
@@ -93,8 +105,8 @@ module Decidim
       private
 
       def validate_organization_uniqueness
-        errors.add(:name, :taken) if Decidim::Organization.where(name: name).where.not(id: id).exists?
-        errors.add(:host, :taken) if Decidim::Organization.where(host: host).where.not(id: id).exists?
+        errors.add(:name, :taken) if Decidim::Organization.where(name:).where.not(id:).exists?
+        errors.add(:host, :taken) if Decidim::Organization.where(host:).where.not(id:).exists?
       end
     end
   end

data/app/jobs/decidim/system/application_job.rb

@@ -4,7 +4,7 @@ module Decidim
   module System
     # Custom ApplicationJob scoped to the system panel.
     #
-    class ApplicationJob < ActiveJob::Base
+    class ApplicationJob < Decidim::ApplicationJob
     end
   end
 end

data/app/packs/entrypoints/decidim_system.js

@@ -1,3 +1,5 @@
-import "src/decidim/system/application"
-import "entrypoints/decidim_system.scss"
-import "src/decidim/direct_uploads/upload_field"
+// Images
+require.context("../images", true)
+
+// CSS
+import "stylesheets/decidim/system/application.scss"

data/app/packs/stylesheets/decidim/system/application.scss

@@ -1,22 +1,109 @@
-$primary-color: null;
-$secondary-color: null;
-$success-color: null;
-$warning-color: null;
-$alert-color: null;
-$-zf-size: null;
-$-zf-bp-value: null;
-
-@import "stylesheets/decidim/system/foundation_and_overrides";
-@import "stylesheets/decidim/system/layout";
-@import "stylesheets/decidim/system/login";
-@import "stylesheets/decidim/system/sidebar";
-@import "stylesheets/decidim/system/tables";
-@import "stylesheets/decidim/system/actions";
-@import "stylesheets/decidim/system/forms";
-@import "stylesheets/decidim/editor";
-@import "stylesheets/decidim/utils/fontface";
-@import "stylesheets/decidim/modules/upload_modal";
-
-body{
-  font-family: 'Source Sans Pro', sans-serif;
+@import "tom-select/dist/scss/tom-select";
+
+:root {
+  --primary: #e02d2d;
+  --primary-rgb: 224 45 45;
+  --secondary: #155abf;
+  --secondary-rgb: 21 90 191;
+  --tertiary: #ebc34b;
+  --tertiary-rgb: 235 195 75;
+  --success: #28a745;
+  --success-rgb: 40 167 69;
+  --warning: #ffb703;
+  --warning-rgb: 255 183 3;
+  --alert: #e7131a;
+  --alert-rgb: 231 19 26;
+}
+
+aside {
+  @apply bg-gray-4 w-full md:w-1/6 min-w-[250px] flex-none text-white relative translate-x-0 transition;
+
+  &[aria-hidden="true"] {
+    @apply absolute -translate-x-full;
+  }
+}
+
+.main-nav {
+  li {
+    @apply border-l-2 border-transparent font-semibold hover:bg-black transition;
+
+    &.active {
+      @apply border-primary bg-black;
+    }
+  }
+
+  a {
+    @apply block p-4;
+  }
+}
+
+table {
+  @apply w-full border-collapse;
+
+  th {
+    @apply font-semibold bg-background;
+  }
+
+  td,
+  th {
+    @apply p-2 text-center first:text-left;
+  }
+
+  tr:nth-child(even) td {
+    @apply bg-gray-3;
+  }
+
+  a {
+    @apply text-primary underline;
+  }
+}
+
+[id^="advanced-settings"] {
+  &[aria-expanded="false"] span:first-child,
+  &[aria-expanded="true"] span:last-child {
+    @apply block;
+  }
+
+  &[aria-expanded="false"] span:last-child,
+  &[aria-expanded="true"] span:first-child {
+    @apply hidden;
+  }
+
+  &[aria-hidden="true"] {
+    @apply hidden;
+  }
+}
+
+dl {
+  dt {
+    @apply font-semibold;
+  }
+
+  dd {
+    @apply text-gray-2;
+  }
+
+  dt + dd,
+  dd + dt {
+    @apply mt-4;
+  }
+}
+
+.flash__message {
+  p {
+    @apply my-4;
+  }
+
+  ol {
+    @apply list-decimal;
+  }
+}
+
+/* overwrite tom-select defaults */
+.ts-control {
+  margin-bottom: 0.5rem;
+}
+
+.ts-dropdown {
+  margin: 0;
 }

data/app/views/decidim/system/admins/_form.html.erb

@@ -1,11 +1,5 @@
-<div class="field">
+<div class="form__wrapper">
   <%= form.text_field :email, autofocus: true %>
-</div>
-
-<div class="field">
   <%= form.password_field :password %>
-</div>
-
-<div class="field">
   <%= form.password_field :password_confirmation %>
 </div>

data/app/views/decidim/system/admins/edit.html.erb

@@ -1,11 +1,13 @@
+<% add_decidim_page_title(t(".title")) %>
+
 <% provide :title do %>
-  <h2><%= t ".title" %></h2>
+  <h1 class="h1"><%= t ".title" %></h1>
 <% end %>
 
 <%= decidim_form_for(@form) do |f| %>
   <%= render partial: "form", object: f %>
 
-  <div class="actions">
-    <%= f.submit t(".update") %>
+  <div class="form__wrapper-block">
+    <%= f.submit t(".update"), class: "button button__sm md:button__lg button__primary" %>
   </div>
 <% end %>

data/app/views/decidim/system/admins/index.html.erb

@@ -1,35 +1,9 @@
+<% add_decidim_page_title(t(".title")) %>
+
 <% provide :title do %>
-  <h2><%= t ".title" %></h2>
+  <h1 class="h1"><%= t ".title" %></h1>
 <% end %>
 
-<div class="actions title">
-  <%= link_to t("actions.new", scope: "decidim.system", name: t("models.admin.name", scope: "decidim.system")), [:new, :admin], class: "new" %>
-</div>
+<%= link_to t("actions.new_admin", scope: "decidim.system"), [:new, :admin], class: "button button__sm md:button__lg button__primary" %>
 
-<table class="stack">
-  <thead>
-    <tr>
-      <th><%= t("models.admin.fields.email", scope: "decidim.system") %></th>
-      <th><%= t("models.admin.fields.created_at", scope: "decidim.system") %></th>
-      <th class="actions"><%= t("actions.title", scope: "decidim.system") %></th>
-    </tr>
-  </thead>
-  <tbody>
-    <% @admins.each do |admin| %>
-      <tr>
-        <td>
-          <%= link_to admin.email, admin %><br>
-        </td>
-        <td>
-          <%= l admin.created_at, format: :short %>
-        </td>
-        <td class="actions">
-          <%= link_to t("actions.edit", scope: "decidim.system"), [:edit, admin] %>
-          <% unless current_admin?(admin) %>
-            <%= link_to t("actions.destroy", scope: "decidim.system"), admin, method: :delete, class: "small alert button", data: { confirm: t("actions.confirm_destroy", scope: "decidim.system") } %>
-          <% end %>
-        </td>
-      </tr>
-    <% end %>
-  </tbody>
-</table>
+<%= render partial: "decidim/system/shared/admins_list", locals: { admins: @admins } %>

data/app/views/decidim/system/admins/new.html.erb

@@ -1,11 +1,13 @@
+<% add_decidim_page_title(t(".title")) %>
+
 <% provide :title do %>
-  <h2><%= t ".title" %></h2>
+  <h1 class="h1"><%= t ".title" %></h1>
 <% end %>
 
 <%= decidim_form_for(@form) do |f| %>
   <%= render partial: "form", object: f %>
 
-  <div class="actions">
-    <%= f.submit t(".create") %>
+  <div class="form__wrapper-block">
+    <%= f.submit t(".create"), class: "button button__sm md:button__lg button__primary" %>
   </div>
 <% end %>

data/app/views/decidim/system/dashboard/show.html.erb

@@ -1,5 +1,17 @@
+<% add_decidim_page_title(t("decidim.system.titles.dashboard")) %>
+
 <% provide :title do %>
-  <h2><%= t("decidim.system.titles.dashboard") %></h2>
-  <h3><%= t ".current_organizations" %></h3>
-  <%= render partial: "decidim/system/shared/organizations_list", locals: { organizations: @organizations } %>
+  <h1 class="h1"><%= t("decidim.system.titles.dashboard") %></h1>
 <% end %>
+
+<h2 class="h3"><%= t ".current_organizations" %></h2>
+
+<%= link_to t("actions.new_organization", scope: "decidim.system"), [:new, :organization], class: "button button__sm md:button__lg button__primary" %>
+
+<%= render partial: "decidim/system/shared/organizations_list", locals: { organizations: @organizations } %>
+
+<h2 class="h3"><%= t ".admins" %></h2>
+
+<%= link_to t("actions.new_admin", scope: "decidim.system"), [:new, :admin], class: "button button__sm md:button__lg button__primary" %>
+
+<%= render partial: "decidim/system/shared/admins_list", locals: { admins: @admins } %>

data/app/views/decidim/system/devise/mailers/password_change.html.erb

@@ -1,3 +1,3 @@
 <p>Hello <%= @resource.email %>!</p>
 
-<p>We're contacting you to notify you that your password has been changed.</p>
+<p>We are contacting you to notify you that your password has been changed.</p>

data/app/views/decidim/system/devise/mailers/reset_password_instructions.html.erb

@@ -4,5 +4,5 @@
 
 <p><%= link_to "Change my password", edit_password_url(@resource, reset_password_token: @token) %></p>
 
-<p>If you didn't request this, please ignore this email.</p>
-<p>Your password won't change until you access the link above and create a new one.</p>
+<p>If you did not request this, please ignore this email.</p>
+<p>Your password will not change until you access the link above and create a new one.</p>