bump version 0.11.3 (#344)

* bump version 0.11.3

* underscore

* add last updates

* ffi

* erblint

* fix modal

* button to a

* changelog

* method

* displayblock

* add an organization memoizer for helpers

* ensure request exits

* memoize in request

* Fix cell spec

---------

Co-authored-by: Ivan Vergés <Ivan Vergés>

Author: microstudi

CHANGELOG.md

@@ -1,13 +1,16 @@
 CHANGELOG
 =========
 
-Current
+v0.11.3
 -------
 Compatibility:
   - Decidim v0.28.x
 
 Features:
   - Added user time zones in account settings
+  - Added custom styles for the admin panel
+  - Added Verification tweaks
+  - Added Admin manual verifications
 
 v0.11.2
 -------

Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    decidim-decidim_awesome (0.11.2)
+    decidim-decidim_awesome (0.11.3)
       decidim-admin (>= 0.28.0, < 0.29)
       decidim-core (>= 0.28.0, < 0.29)
       deface (>= 1.5)

app/controllers/decidim/decidim_awesome/admin/admin_authorizations_controller.rb

@@ -6,7 +6,7 @@ module Admin
       class AdminAuthorizationsController < DecidimAwesome::Admin::ApplicationController
         include NeedsAwesomeConfig
 
-        layout false
+        layout "layouts/decidim/decidim_awesome/admin/admin_authorizations"
         helper_method :user, :authorization, :workflow, :handler, :conflict
         # overwrite original rescue_from to ensure we print messages from ajax methods
         rescue_from Decidim::ActionForbidden, with: :json_error
@@ -23,16 +23,16 @@ def update
           if conflict
             message = render_to_string("conflict")
           else
-            message = render_to_string(partial: "callout", locals: { i18n_key: "user_authorized", klass: "success" })
+            message = render_to_string("callout", locals: { i18n_key: "user_authorized", klass: "success" })
             Decidim::Verifications::AuthorizeUser.call(handler, current_organization) do
               on(:transferred) do |transfer|
-                message += render_to_string(partial: "callout", locals: { i18n_key: "authorization_transferred", klass: "success" }) if transfer.records.any?
+                message += render_to_string("callout", locals: { i18n_key: "authorization_transferred", klass: "success" }) if transfer.records.any?
               end
               on(:invalid) do
                 if force_verification.present?
                   create_forced_authorization
                 else
-                  message = render_to_string(partial: "callout", locals: { i18n_key: "user_not_authorized", klass: "alert" })
+                  message = render_to_string("callout", locals: { i18n_key: "user_not_authorized", klass: "alert" })
                   message += render_to_string("edit", locals: { with_override: true })
                 end
               end
@@ -52,9 +52,9 @@ def update
 
         def destroy
           message = if destroy_authorization
-                      render_to_string(partial: "callout", locals: { i18n_key: "authorization_destroyed", klass: "success" })
+                      render_to_string("callout", locals: { i18n_key: "authorization_destroyed", klass: "success" })
                     else
-                      render_to_string(partial: "callout", locals: { i18n_key: "authorization_not_destroyed", klass: "alert" })
+                      render_to_string("callout", locals: { i18n_key: "authorization_not_destroyed", klass: "alert" })
                     end
 
           render json: {
@@ -80,7 +80,7 @@ def destroy_authorization
         end
 
         def json_error(exception)
-          render json: render_to_string(partial: "callout", locals: { message: exception.message, klass: "alert" }), status: :unprocessable_entity
+          render plain: render_to_string("callout", locals: { message: exception.message, klass: "alert" }), status: :unprocessable_entity
         end
 
         def user

app/packs/src/decidim/decidim_awesome/admin/verifications.js

@@ -3,19 +3,16 @@ document.addEventListener("DOMContentLoaded", () => {
   if (!dialog) {
     return;
   }
-  const title = dialog.querySelector("[data-dialog-title]");
-  const content = dialog.querySelector("[data-dialog-content]");
-
+  const container = dialog.querySelector("[data-dialog-container]");
+  
   dialog.addEventListener("open.dialog", async (el) => {
     const modal = window.Decidim.currentDialogs[el.target.id];
     const button = modal.openingTrigger;
     const url = button.dataset.verificationUrl;
-    const user = button.dataset.verificationUser;
-    title.innerText = title.innerText.replace("{{user}}", user);
-    content.innerHTML = '<br><br><span class="loading-spinner"></span>';
-    // console.log("open.dialog", el, "content", content, "button", button, "url", url);
+    container.innerHTML = '<br><br><span class="loading-spinner"></span>';
+    // console.log("open.dialog", el, "container", container, "button", button, "url", url);
     fetch(url).then((res) => res.text()).then((html) => {
-      content.innerHTML = html;
+      container.innerHTML = html;
     });
   });
 
@@ -24,17 +21,16 @@ document.addEventListener("DOMContentLoaded", () => {
     const response = JSON.parse(responseText.detail[0].response)
     const button = document.querySelector(`[data-verification-handler="${response.handler}"][data-verification-user-id="${response.userId}"]`);
     // console.log("ajax:complete", responseText, "response", response, "button", button);
-    content.innerHTML = response.message;
+    container.innerHTML = response.message;
 
     if (response.granted) {
       button.classList.add("granted");
     } else {
       button.classList.remove("granted");
-      const forceVerificationCheck = content.querySelector("#force_verification_check");
-      const forceVerification = content.querySelector("#force_verification");
+      const forceVerificationCheck = container.querySelector("#force_verification_check");
+      const forceVerification = container.querySelector("#force_verification");
 
       if (forceVerificationCheck) {
-        console.log(forceVerificationCheck);
         forceVerificationCheck.addEventListener("change", function() {
           forceVerification.disabled = !forceVerification.disabled;
           if (forceVerificationCheck.checked) {

app/packs/stylesheets/decidim/decidim_awesome/admin/verifications.scss

@@ -6,7 +6,9 @@
   &-td:not(:first-child) {
     @apply text-left;
 
-    button {
+    a {
+      @apply text-sm font-normal block;
+
       &:hover {
         @apply text-secondary underline;
       }

app/views/decidim/decidim_awesome/admin/admin_authorizations/_callout.html.erb renamed to app/views/decidim/decidim_awesome/admin/admin_authorizations/callout.html.erb

@@ -15,7 +15,7 @@
         data[:dialog_open] = "awesome-verification-modal"
       end
     %>
-    <%= content_tag(auth[:managed] ? "button" : "div", class: classes, data:) do %>
+    <%= content_tag(auth[:managed] ? "a" : "div", href: "#", class: classes, data:) do %>
       <%= icon "check-line", class: "checked" %>
       <%= icon "time-line", class: "undeterminated" %>
       <%= icon "close-line", class: "unchecked" %>

app/views/decidim/decidim_awesome/admin/officializations/_participants_td.html.erb

@@ -1,11 +1,5 @@
 <%= decidim_modal id: "awesome-verification-modal", class:"reveal awesome_participants-modal" do %>
   <div data-dialog-container>
-    <%= icon "lock-line" %>
-    <h3 tabindex="-1" data-dialog-title>
-      <%= t(".authorizations_for", user: "{{user}}") %>
-    </h3>
-    <div data-dialog-content>
-    </div>
   </div>
   <div data-dialog-actions>
     <button data-dialog-close="awesome-verification-modal" class="button button__lg button__transparent-secondary">

app/views/decidim/decidim_awesome/admin/officializations/_verification_modal.html.erb

@@ -0,0 +1,7 @@
+<%= icon "lock-line" %>
+<h3 tabindex="-1" data-dialog-title>
+  <%= t("decidim.decidim_awesome.admin.officializations.verification_modal.authorizations_for", user: user.name) %>
+</h3>
+<div data-dialog-content>
+  <%= yield %>
+</div>

app/views/layouts/decidim/decidim_awesome/admin/admin_authorizations.html.erb

@@ -65,7 +65,7 @@ class AdminEngine < ::Rails::Engine
         end
         # submenus
         Decidim::DecidimAwesome::Menu.register_custom_fields_submenu!
-        Decidim::DecidimAwesome::Menu.register_custom__styles_submenu!
+        Decidim::DecidimAwesome::Menu.register_custom_styles_submenu!
         Decidim::DecidimAwesome::Menu.register_menu_hacks_submenu!
         Decidim::DecidimAwesome::Menu.register_maintenance_admin_menu!
         Decidim::DecidimAwesome::Menu.register_awesome_admin_menu!