avo 1.22.2 → 1.23.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d1b3e02ee566af1fcf8ac642f0b59693fc9ca60840f5a57e46c0e57df32831c9
-  data.tar.gz: 621dc29f10b4693db5e09531fb7cd8cd591666e0be8d098d3927a48e0e2cb0dd
+  metadata.gz: 93ddc5b52a7eb149886833f2eda8dcf69e5cfa29af5d357f57dfbc3b6e03800e
+  data.tar.gz: 44050a09443ff49550f7d9e3c22c3a4b03eea83f9c0d3a08bbc4a97d5084a355
 SHA512:
-  metadata.gz: 325009723e404b72ab8ff8cde8c0491ecf4c520b075b7731ee837c56998f1c45d57b50805ebbce1358dfff3b9f7e6b8a53d3ada14a9fe365523a6a3f077dbf29
-  data.tar.gz: de4a2bcb927f0d1eb4053185e3683c0d44d06c0fce7848f3740edaee52cdabfc3c6a5cde067b6ad9f8506013ea8eb214639073e19793cc9fc4e51b72bbdcaf38
+  metadata.gz: 346e58ffb98b4b60576282f43a766f6e9e388dcf83041dffc8b1f9c1e81292271869e2f356df92471afa77e51eff5945f127a236e2adc1c41a24ccb6847d2a5f
+  data.tar.gz: 5dbeae53b1d1179117f035c1bc2b806c9f88064f5af9c9c1498fbd6fe24107d96a67c02345f437f2cd73789de4935932c97f52730142e970808a68e49c5e2aa0

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    avo (1.22.2)
+    avo (1.23.0)
       active_link_to
       addressable
       breadcrumbs_on_rails

data/README.md

@@ -73,3 +73,9 @@ Please read the [UPGRADE_GUIDE.MD](https://docs.avohq.io/1.0/upgrade.html)
 
 
 ![Alt](https://repobeats.axiom.co/api/embed/1481a6a259064f02a7936470d12a50802a9c98a4.svg "Repobeats analytics image")
+
+# Shoutouts
+
+[Get a box of waffles and some of the best app monitoring from Appsignal](https://appsignal.com/r/93dbe69bfb) 🧇
+
+[Get $100 in credits from Digital Ocean](https://www.digitalocean.com/?refcode=efc1fe881d74&utm_campaign=Referral_Invite&utm_medium=Referral_Program&utm_source=badge) 💸

data/app/components/avo/alert_component.rb

@@ -0,0 +1,11 @@
+# frozen_string_literal: true
+
+class Avo::AlertComponent < ViewComponent::Base
+  attr_reader :type
+  attr_reader :message
+
+  def initialize(type, message)
+    @type = type
+    @message = message
+  end
+end

data/app/components/avo/alerts_component.html.erb

@@ -0,0 +1,3 @@
+<% helpers.flash.each do |type, message| %>
+  <%= render Avo::AlertComponent.new type.to_sym, message %>
+<% end %>

data/app/components/avo/alerts_component.rb

@@ -0,0 +1,5 @@
+# frozen_string_literal: true
+
+class Avo::AlertsComponent < ViewComponent::Base
+
+end

data/app/components/avo/navigation_link_component.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 class Avo::NavigationLinkComponent < ViewComponent::Base
-  def initialize(label: nil, path: nil, active: :inclusive, size: :md, target: "_self")
+  def initialize(label: nil, path: nil, active: :inclusive, size: :md, target: nil)
     @label = label
     @path = path
     @active = active

data/app/{views/avo/partials/_turbo_frame_wrap.html.erb → components/avo/turbo_frame_wrapper_component.html.erb}

@@ -3,7 +3,7 @@
     # When rendering the frames the flashed content gets lost.
     # By including the alerts partial, the stimulus will pick them up and display them to the user.
   %>
-  <%= render partial: 'avo/partials/alerts' if flash.present? && name.present? %>
+  <%= render Avo::AlertsComponent.new if helpers.flash.present? && name.present? %>
 
-  <%= yield %>
+  <%= content %>
 <% if name.present? %></turbo-frame><% end %>

data/app/components/avo/turbo_frame_wrapper_component.rb

@@ -0,0 +1,9 @@
+# frozen_string_literal: true
+
+class Avo::TurboFrameWrapperComponent < ViewComponent::Base
+  attr_reader :name
+
+  def initialize(name = nil)
+    @name = name
+  end
+end

data/app/controllers/avo/application_controller.rb

@@ -13,6 +13,7 @@ module Avo
     protect_from_forgery with: :exception
     before_action :init_app
     before_action :check_avo_license
+    before_action :set_locale
     before_action :set_authorization
     before_action :_authenticate!
     before_action :set_container_classes
@@ -245,5 +246,11 @@ module Avo
     def model_param_key
       @resource.form_scope
     end
+
+    def set_locale
+      I18n.locale = params[:locale] || I18n.default_locale
+
+      I18n.default_locale = I18n.locale
+    end
   end
 end

data/app/controllers/avo/base_controller.rb

@@ -84,7 +84,6 @@ module Avo
     def new
       @model = @resource.model_class.new
       @resource = @resource.hydrate(model: @model, view: :new, user: _current_user)
-      # abort @model.course.inspect
 
       @page_title = @resource.default_panel_name
       add_breadcrumb resource_name.humanize, resources_path(resource: @resource)

data/app/helpers/avo/application_helper.rb

@@ -21,7 +21,7 @@ module Avo
     end
 
     def turbo_frame_wrap(name, &block)
-      render layout: "avo/partials/turbo_frame_wrap", locals: {name: name} do
+      render Avo::TurboFrameWrapperComponent.new name do
         capture(&block)
       end
     end

data/app/javascript/js/application.js

@@ -1,4 +1,4 @@
-import { Application } from 'stimulus'
+import { Application } from '@hotwired/stimulus'
 
 const application = Application.start()
 

data/app/javascript/js/controllers/action_controller.js

@@ -1,4 +1,4 @@
-import { Controller } from 'stimulus'
+import { Controller } from '@hotwired/stimulus'
 import { castBoolean } from '../helpers/cast_boolean'
 
 export default class extends Controller {

data/app/javascript/js/controllers/actions_picker_controller.js

@@ -1,5 +1,5 @@
 import { AttributeObserver } from '@stimulus/mutation-observers'
-import { Controller } from 'stimulus'
+import { Controller } from '@hotwired/stimulus'
 
 export default class extends Controller {
   static targets = ['resourceAction', 'standaloneAction']

data/app/javascript/js/controllers/alerts_controller.js

@@ -1,4 +1,4 @@
-import { Controller } from 'stimulus'
+import { Controller } from '@hotwired/stimulus'
 
 export default class extends Controller {
   static targets = ['container']

data/app/javascript/js/controllers/attachments_controller.js

@@ -1,4 +1,4 @@
-import { Controller } from 'stimulus'
+import { Controller } from '@hotwired/stimulus'
 
 export default class extends Controller {
   static targets = ['form']

data/app/javascript/js/controllers/fields/belongs_to_field_controller.js

@@ -1,4 +1,4 @@
-import { Controller } from 'stimulus'
+import { Controller } from '@hotwired/stimulus'
 
 export default class extends Controller {
   static targets = ['select', 'type', 'loadAssociationLink'];

data/app/javascript/js/controllers/fields/code_field_controller.js

@@ -14,7 +14,7 @@ import 'codemirror/mode/vue/vue'
 import 'codemirror/mode/xml/xml'
 import 'codemirror/mode/yaml/yaml'
 
-import { Controller } from 'stimulus'
+import { Controller } from '@hotwired/stimulus'
 import CodeMirror from 'codemirror'
 
 import { castBoolean } from '../../helpers/cast_boolean'

data/app/javascript/js/controllers/fields/date_field_controller.js

@@ -1,4 +1,4 @@
-import { Controller } from 'stimulus'
+import { Controller } from '@hotwired/stimulus'
 import { DateTime } from 'luxon'
 import flatpickr from 'flatpickr'
 

data/app/javascript/js/controllers/fields/key_value_controller.js

@@ -1,5 +1,5 @@
 /* eslint-disable max-len */
-import { Controller } from 'stimulus'
+import { Controller } from '@hotwired/stimulus'
 import { castBoolean } from '../../helpers/cast_boolean'
 
 export default class extends Controller {
@@ -39,7 +39,7 @@ export default class extends Controller {
 
   deleteRow(event) {
     if (this.options.disable_deleting_rows || !this.options.editable) return
-    const { index } = event.target.dataset
+    const { index } = event.params
     this.fieldValue.splice(index, 1)
     this.updateTextareaInput()
     this.updateKeyValueComponent()
@@ -93,7 +93,7 @@ export default class extends Controller {
     if (this.options.editable) {
       result += `<a
   href="javascript:void(0);"
-  data-index="${index}"
+  data-key-value-index-param="${index}"
   data-action="click->key-value#deleteRow"
   title="${this.options.delete_text}"
   data-tippy="tooltip"

data/app/javascript/js/controllers/fields/simple_mde_controller.js

@@ -1,4 +1,4 @@
-import { Controller } from 'stimulus'
+import { Controller } from '@hotwired/stimulus'
 import SimpleMDE from 'simplemde'
 
 export default class extends Controller {

data/app/javascript/js/controllers/fields/trix_field_controller.js

@@ -1,5 +1,5 @@
 import 'trix'
-import { Controller } from 'stimulus'
+import { Controller } from '@hotwired/stimulus'
 import { castBoolean } from '../../helpers/cast_boolean'
 
 export default class extends Controller {

data/app/javascript/js/controllers/filter_controller.js

@@ -1,4 +1,4 @@
-import { Controller } from 'stimulus'
+import { Controller } from '@hotwired/stimulus'
 import URI from 'urijs'
 
 export default class extends Controller {
@@ -26,16 +26,12 @@ export default class extends Controller {
     // then we convert the percent encodings into raw bytes which
     // can be fed into btoa.
     return btoa(encodeURIComponent(str).replace(/%([0-9A-F]{2})/g,
-      function toSolidBytes(match, p1) {
-        return String.fromCharCode('0x' + p1);
-      }));
+      (match, p1) => String.fromCharCode(`0x${p1}`)))
   }
 
   b64DecodeUnicode(str) {
     // Going backwards: from bytestream, to percent-encoding, to original string.
-    return decodeURIComponent(atob(str).split('').map(function(c) {
-      return '%' + ('00' + c.charCodeAt(0).toString(16)).slice(-2);
-    }).join(''));
+    return decodeURIComponent(atob(str).split('').map((c) => `%${(`00${c.charCodeAt(0).toString(16)}`).slice(-2)}`).join(''))
   }
 
   changeFilter() {

data/app/javascript/js/controllers/hidden_input_controller.js

@@ -1,4 +1,4 @@
-import { Controller } from 'stimulus'
+import { Controller } from '@hotwired/stimulus'
 
 export default class extends Controller {
   static targets = ['content']

data/app/javascript/js/controllers/item_select_all_controller.js

@@ -1,20 +1,20 @@
-import { Controller } from 'stimulus'
+import { Controller } from '@hotwired/stimulus'
 
 export default class extends Controller {
-  static targets = [ "itemCheckbox", "checkbox" ]
+  static targets = ['itemCheckbox', 'checkbox']
 
   connect() {
     this.resourceName = this.element.dataset.resourceName
   }
 
   toggle(event) {
-    var value = !!event.target.checked
+    const value = !!event.target.checked
     document.querySelectorAll(`[data-controller="item-selector"][data-resource-name="${this.resourceName}"] input[type=checkbox]`)
       .forEach((checkbox) => checkbox.checked != value && checkbox.click())
   }
 
   update() {
-    var allSelected = true
+    let allSelected = true
     this.itemCheckboxTargets.forEach((checkbox) => allSelected = allSelected && checkbox.checked)
     this.checkboxTarget.checked = allSelected
   }

data/app/javascript/js/controllers/item_selector_controller.js

@@ -1,4 +1,4 @@
-import { Controller } from 'stimulus'
+import { Controller } from '@hotwired/stimulus'
 
 export default class extends Controller {
   static targets = ['panel'];

data/app/javascript/js/controllers/loading_button_controller.js

@@ -1,4 +1,4 @@
-import { Controller } from 'stimulus'
+import { Controller } from '@hotwired/stimulus'
 
 export default class extends Controller {
   spinnerMarkup = `<div class="button-spinner">

data/app/javascript/js/controllers/modal_controller.js

@@ -1,4 +1,4 @@
-import { Controller } from 'stimulus'
+import { Controller } from '@hotwired/stimulus'
 
 export default class extends Controller {
   static targets = ['modal']
@@ -8,4 +8,13 @@ export default class extends Controller {
 
     document.dispatchEvent(new Event('actions-modal:close'))
   }
+
+  delayedClose() {
+    const vm = this
+
+    setTimeout(() => {
+      vm.modalTarget.remove()
+      document.dispatchEvent(new Event('actions-modal:close'))
+    }, 500)
+  }
 }

data/app/javascript/js/controllers/per_page_controller.js

@@ -1,4 +1,4 @@
-import { Controller } from 'stimulus'
+import { Controller } from '@hotwired/stimulus'
 
 export default class extends Controller {
   static targets = ['selector']

data/app/javascript/js/controllers/search_controller.js

@@ -1,6 +1,6 @@
 /* eslint-disable no-underscore-dangle */
 import * as Mousetrap from 'mousetrap'
-import { Controller } from 'stimulus'
+import { Controller } from '@hotwired/stimulus'
 import { Turbo } from '@hotwired/turbo-rails'
 import { autocomplete } from '@algolia/autocomplete-js'
 import URI from 'urijs'
@@ -177,11 +177,16 @@ export default class extends Controller {
       openOnFocus: true,
       detachedMediaQuery: '',
       getSources: ({ query }) => {
+        document.body.classList.add('search-loading')
         const endpoint = that.searchUrl(query)
 
         return that
           .debouncedFetch(endpoint)
-          .then((response) => response.json())
+          .then((response) => {
+            document.body.classList.remove('search-loading')
+
+            return response.json()
+          })
           .then((data) => Object.keys(data).map((resourceName) => that.addSource(resourceName, data[resourceName])))
       },
     })

data/app/javascript/js/controllers/tippy_controller.js

@@ -1,4 +1,4 @@
-import { Controller } from 'stimulus'
+import { Controller } from '@hotwired/stimulus'
 import tippy from 'tippy.js'
 
 export default class extends Controller {

data/app/javascript/js/controllers/toggle_panel_controller.js

@@ -1,4 +1,4 @@
-import { Controller } from 'stimulus'
+import { Controller } from '@hotwired/stimulus'
 import { useClickOutside } from 'stimulus-use'
 
 export default class extends Controller {

data/app/views/avo/actions/show.html.erb

@@ -7,7 +7,11 @@
     data-resource-id="<%= params[:id] %>"
     class="hidden text-slate-800"
   >
-    <%= form_with model: @model, scope: 'fields', url: "#{@resource.records_path}/actions/#{@action.param_id}", data: {'turbo-frame': '_top', 'action-target': 'form'} do |form| %>
+    <%= form_with model: @model,
+      scope: 'fields',
+      url: "#{@resource.records_path}/actions/#{@action.param_id}",
+      data: @action.class.form_data_attributes do |form|
+    %>
       <%= render Avo::ModalComponent.new do |c| %>
         <% c.heading do %>
           <%= @action.action_name %>
@@ -26,8 +30,8 @@
         <% end %>
 
         <% c.controls do %>
-          <%= a_button @action.cancel_button_label, 'data-action': 'click->modal#close', size: :sm %>
-          <%= a_button @action.confirm_button_label, type: :submit, color: :green, size: :sm %>
+          <%= a_button @action.cancel_button_label, data: { action: 'click->modal#close' }, size: :sm %>
+          <%= a_button @action.confirm_button_label, type: :submit, color: :green, size: :sm, data: @action.class.submit_button_data_attributes %>
         <% end %>
       <% end %>
     <% end %>

data/app/views/layouts/avo/application.html.erb

@@ -51,7 +51,7 @@
   <%= turbo_frame_tag 'destroy_attachment_form' %>
 
   <%= turbo_frame_tag 'alerts' do %>
-    <%= render partial: 'avo/partials/alerts' %>
+    <%= render Avo::AlertsComponent.new %>
   <% end %>
 </div>
 

data/lib/avo/base_action.rb

@@ -15,6 +15,7 @@ module Avo
     class_attribute :fields
     class_attribute :standalone, default: false
     class_attribute :visible
+    class_attribute :may_download_file, default: false
 
     attr_accessor :response
     attr_accessor :model
@@ -22,6 +23,26 @@ module Avo
     attr_accessor :user
     attr_accessor :fields_loader
 
+    class << self
+      def form_data_attributes
+        # We can't respond with a file download from Turbo se we disable it on the form
+        if may_download_file
+          { 'turbo': false }
+        else
+          { 'turbo-frame': '_top', 'action-target': 'form' }
+        end
+      end
+
+      # We can't respond with a file download from Turbo se we disable close the modal manually after a while (it's a hack, we know)
+      def submit_button_data_attributes
+        if may_download_file
+          { action: 'click->modal#delayedClose' }
+        else
+          {}
+        end
+      end
+    end
+
     def action_name
       return name if name.present?
 

data/lib/avo/licensing/h_q.rb

@@ -23,6 +23,12 @@ module Avo
 
         begin
           perform_and_cache_request
+        rescue Errno::EHOSTUNREACH => exception
+          cache_and_return_error "HTTP host not reachable error.", exception.message
+        rescue Errno::ECONNRESET => exception
+          cache_and_return_error "HTTP connection reset error.", exception.message
+        rescue Errno::ECONNREFUSED => exception
+          cache_and_return_error "HTTP connection refused error.", exception.message
         rescue HTTParty::Error => exception
           cache_and_return_error "HTTP client error.", exception.message
         rescue Net::OpenTimeout => exception
@@ -71,7 +77,55 @@ module Avo
           environment: Rails.env,
           ip: current_request.ip,
           host: current_request.host,
-          port: current_request.port
+          port: current_request.port,
+          app_name: app_name
+        }
+      end
+
+      def app_name
+        Rails.application.class.to_s.split("::").first
+      rescue
+        nil
+      end
+
+      def avo_metadata
+        resources = App.resources
+        field_definitions = resources.map(&:get_field_definitions)
+        fields_count = field_definitions.map(&:count).sum
+        fields_per_resource = sprintf("%0.01f", fields_count / (resources.count + 0.0))
+
+        field_types = {}
+        custom_fields_count = 0
+        field_definitions.each do |fields|
+          fields.each do |field|
+            field_types[field.type] ||= 0
+            field_types[field.type] += 1
+
+            custom_fields_count += 1 if field.custom?
+          end
+        end
+
+        {
+          resources_count: resources.count,
+          fields_count: fields_count,
+          fields_per_resource: fields_per_resource,
+          custom_fields_count: custom_fields_count,
+          field_types: field_types,
+          **other_metadata(:actions),
+          **other_metadata(:filters),
+        }
+      end
+
+      def other_metadata(type = :actions)
+        resources = App.resources
+
+        types = resources.map(&:"get_#{type}")
+        type_count = types.flatten.uniq.count
+        type_per_resource = sprintf("%0.01f", types.map(&:count).sum / (resources.count + 0.0))
+
+        {
+          "#{type}_count": type_count,
+          "#{type}_per_resource": type_per_resource,
         }
       end
 

data/lib/avo/version.rb

@@ -1,3 +1,3 @@
 module Avo
-  VERSION = "1.22.2"
+  VERSION = "1.23.0"
 end