nuntius 1.3.6 → 1.3.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 11b30e0625c1b47a70a22803a0bdec45d92faf881b1e83ae72e00582d95b36f4
-  data.tar.gz: ec9251af770d4b3cd4ae58b75968bddc4e4917dc1ba01d8073eb73df56c9fe4e
+  metadata.gz: 9b687606f90025f60156f83577dbc1b9d4d2edecdc70f84e497141ff37476965
+  data.tar.gz: 8f17f799c7d5c991bb8141cb29dd0e026651dc24993ab67e1fe5ed77f47aebfd
 SHA512:
-  metadata.gz: a8dd4831dcbb7c4e8440e4a1ebf281a712959de7a47323e2590a24d71bf039c5a7c7b5899e3427832949a061f1bed2ac72b3f70acdc86b534aafe16f7d364689
-  data.tar.gz: 1c5e6ad9964622533dad4c91df0ff7bcdf05299bbaddcdf81efcfee6eeea02f9b50bc7ad662b174e8433e422d040dda57b8e9f627fd2c8360cc27575653735c8
+  metadata.gz: 27c5d73a3fc860ecb126e2967f75c8d985faf47af0900d59b704be4cf8a00821525344570e1bd611d92fa3ea33689ed1d0420c6edc46d9d69fa2aae58b3b605a
+  data.tar.gz: 1152e80c5e52d47adc53ac5519e328852874495534daf54b4e96491acd911bbdba9e0001f61b30a1491506432b3accd0bf49971487709ee36240d525c2a5026e

data/README.md

@@ -165,6 +165,8 @@ The main key of the hash passed will also be the liquid variable.
 
 Outbound mail is handled through SMTP. We've only exposed the HTML of mail, we can create text-versions based on sanitized HTML. HTML allows for [Foundation for Emails](https://get.foundation/emails/docs/index.html). You will have to include the CSS in the layouts.
 
+In the layout you can add `<a href="{{message_url}}">Link to mail</a>` to provide a link to the online version of the message.
+
 #### AWS SES
 
 In case you use AWS SES, you can use the SNS Feedback Notifications to automatically mark messages as read, or deal with complaints and bounces. Create a AWS SNS topic, with a HTTPS subscription with the following URL (pattern):

data/app/controllers/nuntius/admin/campaigns_controller.rb

@@ -51,7 +51,7 @@ module Nuntius
       def campaign_params
         return unless params[:campaign]
 
-        params.require(:campaign).permit(:name, :transport, :layout_id, :list_id, :from, :subject, :text, :html)
+        params.require(:campaign).permit(:name, :transport, :layout_id, :list_id, :from, :subject, :text, :html, :metadata_yaml)
       end
     end
   end

data/app/controllers/nuntius/admin/layouts_controller.rb

@@ -31,6 +31,11 @@ module Nuntius
       def update
         @layout = Nuntius::Layout.visible.find(params[:id])
         @layout.update(layout_params)
+        if params[:layout][:attachments].present?
+          params[:layout][:attachments].each do |attachment|
+            @layout.attachments.attach(attachment)
+          end
+        end
         respond_with :admin, @layout
       end
 
@@ -44,7 +49,6 @@ module Nuntius
 
       def layout_params
         permitted = %i[name data metadata_yaml]
-        permitted += [attachments: []] if params[:layout][:attachments].compact_blank.present?
         params.require(:layout).permit(permitted)
       end
     end

data/app/models/nuntius/campaign.rb

@@ -3,6 +3,9 @@
 module Nuntius
   class Campaign < ApplicationRecord
     include Nuntius::Concerns::MetadataScoped
+    include Nuntius::Concerns::Yamlify
+
+    yamlify :metadata
 
     belongs_to :list
     accepts_nested_attributes_for :list, reject_if: :all_blank

data/app/providers/nuntius/smtp_mail_provider.rb

@@ -45,7 +45,7 @@ module Nuntius
           charset: "UTF-8"
         )
         if message.html.present?
-          message.html = message.html.gsub("%7B%7Bmessage_url%7D%7D") { message_url(message) }
+          message.html = message.html.gsub("{{message_url}}") { message_url(message) }
           p.html_part = Mail::Part.new(
             body: message.html,
             content_type: "text/html",
@@ -84,7 +84,7 @@ module Nuntius
     private
 
     def message_url(message)
-      Nuntius::Engine.routes.url_helpers.message_url(message.id, host: Nuntius.config.host.call(message))
+      Nuntius::Engine.routes.url_helpers.message_url(message.id, host: Nuntius.config.host(message))
     end
   end
 end

data/app/services/nuntius/deliver_campaign_service.rb

@@ -11,7 +11,7 @@ module Nuntius
 
     def perform
       deliver
-      campaign.sent!
+      # campaign.sent!
     end
 
     def deliver
@@ -22,6 +22,7 @@ module Nuntius
     end
 
     def new_message(subscriber, assigns = {})
+      assigns["campaign"] = context.campaign
       assigns["subscriber"] = subscriber
       if subscriber.nuntiable
         name = Nuntius::BaseMessenger.liquid_variable_name_for(subscriber.nuntiable)
@@ -29,8 +30,11 @@ module Nuntius
       end
       message = Nuntius::Message.new(transport: campaign.transport, campaign: campaign, nuntiable: subscriber.nuntiable, metadata: campaign.metadata)
 
-      locale_proc = Nuntius::BaseMessenger.messenger_for_obj(subscriber.nuntiable).locale
-      locale = instance_exec(subscriber.nuntiable, &locale_proc) if subscriber.nuntiable && locale_proc
+      locale = nil
+      if subscriber.nuntiable
+        locale_proc = Nuntius::BaseMessenger.messenger_for_obj(subscriber.nuntiable).locale
+        locale = instance_exec(subscriber.nuntiable, &locale_proc) if locale_proc
+      end
 
       message.from = render(:from, assigns, locale)
       message.to = case campaign.transport

data/app/tables/nuntius/campaign_messages_table.rb

@@ -0,0 +1,24 @@
+# frozen_string_literal: true
+
+module Nuntius
+  class CampaignMessagesTable < Nuntius::ApplicationTable
+    definition do
+      model Nuntius::Message
+
+      column(:to)
+      column(:status)
+      column(:created_at)
+
+      order created_at: :desc
+
+      link { |message| nuntius.admin_message_path(message) }
+    end
+
+    private
+
+    def scope
+      @scope = Nuntius::Campaign.find_by(params[:campaign_id]).messages
+      @scope
+    end
+  end
+end

data/app/views/nuntius/admin/campaigns/edit.html.slim

@@ -27,21 +27,13 @@
           .col-span-12
             = f.input :subject
           .col-span-12
-            = f.input :html, as: :hidden
-            trix-editor input="campaign_html" style="background-color: #fff; height: 400px; margin-bottom: 20px;"
+            = f.rich_text :html
+          .col-span-12
+            = f.input :metadata_yaml, as: :editor, mode: 'application/yaml', label: t('.metadata')
 
         template data-toggle-target='toggleable' data-toggle-value='voice'
           .col-span-12
             = f.input :text, as: :editor, mode: 'text/plain'
 - else
-  = sts.card :nuntius_admin_campaigns, title: @campaign.name, description: 'We are sending this campaign, you can no longer make any changes.', icon: 'fal fa-megaphone', content_padding: false do |card|
-    table.table
-      thead
-        tr
-          th Status
-          th To
-      tbody
-        - for message in @messages
-          tr
-            td = link_to message.status, admin_message_path(message)
-            td = message.to
+  = sts.card :nuntius_admin_campaigns, title: @campaign.name, description: 'We are sending or have sent this campaign, you can no longer make any changes.', icon: 'fal fa-megaphone', content_padding: false do |card|
+    = sts.table :"nuntius/campaign_messages", params: { campaign_id: @campaign.id }

data/app/views/nuntius/admin/layouts/edit.html.slim

@@ -32,10 +32,11 @@
                 = (attachment.blob.byte_size / 1048576.0).round(2)
                 ' MB
                 = link_to admin_layout_attachment_path(@layout, attachment.id), data: { controller: 'attachment-delete', 'action': 'attachment-delete#delete' } do
-                    i.fal.fa-xmark
+                    i.fas.fa-xmark
                 =< link_to(main_app.rails_blob_path(attachment, disposition: 'attachment'),
                       title: attachment.filename)
-                    i.fal.fa-download
+                    i.fas.fa-download
+                code.text-xs = sts.copyable main_app.rails_blob_url(attachment)
 
     - if @layout.templates.exists?
       - card.with_tab :templates, padding: false

data/config/locales/en.yml

@@ -55,6 +55,7 @@ en:
           card:
             nuntius_admin_campaigns:
               title: Campaigns
+          metadata: Metadata
         index:
           card:
             nuntius_admin_campaigns:

data/config/locales/nl.yml

@@ -55,6 +55,7 @@ nl:
           card:
             nuntius_admin_campaigns:
               title: Campagne
+          metadata: Metadata
         index:
           card:
             nuntius_admin_campaigns:

data/lib/nuntius/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Nuntius
-  VERSION = "1.3.6"
+  VERSION = "1.3.7"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: nuntius
 version: !ruby/object:Gem::Version
-  version: 1.3.6
+  version: 1.3.7
 platform: ruby
 authors:
 - Tom de Grunt
@@ -497,6 +497,7 @@ files:
 - app/services/nuntius/deliver_inbound_message_service.rb
 - app/services/nuntius/retrieve_inbound_mail_service.rb
 - app/tables/nuntius/application_table.rb
+- app/tables/nuntius/campaign_messages_table.rb
 - app/tables/nuntius/campaigns_table.rb
 - app/tables/nuntius/layouts_table.rb
 - app/tables/nuntius/lists_table.rb