bible270 0.17.0 → 0.17.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f2e0821b84cb0bc7ece339c39775d9b7b814b4f274e721f112cc393aabfc5093
-  data.tar.gz: 53f7e5d9386bfd4f9ae14ed6ac56daedab5df0139c0043e4bac6576bde257ed2
+  metadata.gz: 9e3c27f2336f3e2a809205242bbc42c46d241e49b9ea2b1bf49ca288523d7c88
+  data.tar.gz: 8e8c25b59eafde481a17546ac14b563527dd591ca64d9cf7541d95a01dd1f36c
 SHA512:
-  metadata.gz: ef8060ffe3f6a7c99e35978886609c580e091a0bb0b6f86b809ef7bf8afa8bdda919f7e94c97f04ca5bf69e30b5b70b79b37333fb0850073864d40ed3bf8b909
-  data.tar.gz: c17db24c8ca3ab97e950f2fa516f5366b9d98dcbaa0422a4130acab6031bdeeb1a7faefed1d28e6f4ea44f77404ea3a50a28c041b8bba48edab4936bf158313c
+  metadata.gz: 06af092550c30ded960e3611301d941e10fdea25e79fcfcab8aa97c4d68fee7b3780961f22bc4f81be4ba3f065ebe57642386ba318c69f80e2f1bd1e07ce0dec
+  data.tar.gz: ed69c75af0e88e8dfeb347b756c6c10692f46b6e90196e96851aecc026a664752ce6f2fbb3de3b324734eff316555c229d19aa15270d26061b370b4db643f3ae

data/Rakefile

@@ -2,6 +2,7 @@
 
 require 'bundler/gem_tasks' # provides build / install / release
 require 'rake/testtask'
+require 'coveralls/rake/task'
 
 Rake::TestTask.new(:test) do |t|
   t.libs << 'test' << 'lib'
@@ -9,4 +10,6 @@ Rake::TestTask.new(:test) do |t|
   t.warning = false
 end
 
+Coveralls::RakeTask.new
+
 task default: :test

data/app/controllers/bible270/admin_controller.rb

@@ -8,6 +8,9 @@ module Bible270
   # neither set the panel is unreachable, and every action 404s rather than 403s
   # so its existence isn't advertised.
   class AdminController < ApplicationController
+    LAST_BROADCAST_AT = 'last_broadcast_at'
+    LAST_BROADCAST_SUBJECT = 'last_broadcast_subject'
+
     before_action :require_admin!
     before_action :load_reader,
                   only: %i[show destroy update_start update_profile remove_avatar complete_through toggle_day]
@@ -32,6 +35,9 @@ module Bible270
       counts = Checkoff.group(:reader_id, :day).count
       @days_completed = Hash.new(0)
       counts.each { |(rid, day), n| @days_completed[rid] += 1 if n >= Plan.total_parts(day) }
+      @reachable = Reader.where.not(email: [nil, '']).count
+      @last_broadcast_at = parsed_time(Setting.read(LAST_BROADCAST_AT))
+      @last_broadcast_subject = Setting.read(LAST_BROADCAST_SUBJECT)
     end
 
     def show
@@ -75,6 +81,47 @@ module Bible270
     end
 
     # Put the reader on a given day as of today, or set an explicit start date.
+    def broadcast
+      subject = params[:subject].to_s.strip
+      body = params[:body].to_s.strip
+
+      if subject.empty? || body.empty?
+        return redirect_to(admin_path, alert: 'A message needs both a subject and something to say.')
+      end
+
+      recipients = Reader.where.not(email: [nil, '']).order(:id)
+      return redirect_to(admin_path, alert: 'Nobody has an email address.') if recipients.empty?
+
+      sent = deliver_broadcast(recipients, subject, body)
+      Setting.write(LAST_BROADCAST_AT, Time.current.iso8601)
+      Setting.write(LAST_BROADCAST_SUBJECT, subject)
+
+      redirect_to admin_path,
+                  notice: "Sent to #{sent} #{'reader'.pluralize(sent)}."
+    end
+
+    def update_bible_version
+      reader = Reader.find_by(id: params[:id])
+      return redirect_to(admin_path, alert: 'No such reader.') if reader.nil?
+
+      code = params[:bible_version].to_s
+      # Blank means "no preference": the reader follows the site default, which is
+      # not the same as choosing whichever translation the site happens to use now.
+      if code.empty?
+        reader.update(bible_version: nil)
+        return redirect_to admin_reader_path(reader),
+                           notice: "#{reader.display_name} now follows the site default " \
+                                   "(#{Bible270.config.bible_version})."
+      end
+
+      if reader.update_bible_version(code)
+        redirect_to admin_reader_path(reader),
+                    notice: "#{reader.display_name} now reads the #{reader.bible_version_label}."
+      else
+        redirect_to admin_reader_path(reader), alert: 'That is not a translation on the list.'
+      end
+    end
+
     def update_start
       if params[:start_date].present?
         # set_start_date!, not update_start_date!: an admin is not subject to
@@ -138,6 +185,33 @@ module Bible270
       end
     end
 
+    # Failures are counted rather than raised: one bad address must not stop the
+    # rest of the message going out, and the admin is told how many were sent.
+    # A stored timestamp that cannot be read is treated as absent rather than
+    # blowing up the admin page.
+    def parsed_time(value)
+      return nil if value.blank?
+
+      Time.parse(value.to_s)
+    rescue ArgumentError, TypeError
+      nil
+    end
+
+    def deliver_broadcast(recipients, subject, body)
+      later = Bible270.config.registration_notice_deliver_later
+      sent = 0
+
+      recipients.find_each do |reader|
+        mail = NoticeMailer.broadcast(reader_id: reader.id, subject: subject, body: body)
+        later ? mail.deliver_later : mail.deliver_now
+        sent += 1
+      rescue StandardError => e
+        Rails.logger.error("[bible270] broadcast to reader #{reader.id} failed: #{e.class}: #{e.message}")
+      end
+
+      sent
+    end
+
   private
 
     def require_admin!

data/app/controllers/bible270/comments_controller.rb

@@ -92,7 +92,9 @@ module Bible270
       end
 
       ids = roots.sort_by { |_root, at| -at.to_i }.first(wanted).map(&:first)
-      threads = Comment.approved.where(id: ids).includes(:reader, replies: :reader).index_by(&:id)
+      threads = Comment.approved.where(id: ids)
+        .includes(:reader, { likes: :reader }, { replies: [:reader, { likes: :reader }] })
+        .index_by(&:id)
       ids.filter_map { |id| threads[id] }
     end
 

data/app/controllers/bible270/likes_controller.rb

@@ -0,0 +1,22 @@
+# frozen_string_literal: true
+
+module Bible270
+  # A heart on someone's reflection. One route, because liking and unliking are
+  # the same gesture — tapping the heart again takes it back.
+  class LikesController < ApplicationController
+    def toggle
+      return unless require_reader!
+
+      @comment = Comment.approved.find_by(id: params[:id])
+      head :not_found and return unless @comment
+
+      @comment.toggle_like!(current_reader)
+      @comment.reload
+
+      respond_to do |format|
+        format.turbo_stream
+        format.html { redirect_to day_path(@comment.day, anchor: "comment-#{@comment.id}") }
+      end
+    end
+  end
+end

data/app/controllers/bible270/sessions_controller.rb

@@ -36,7 +36,9 @@ module Bible270
       remember_reader!(reader)
       @current_reader = reader
 
-      redirect_to destination, notice: "Welcome #{reader.first_name}."
+      # first_name is filled in for every route now, but a bridged host user may
+      # still arrive without one.
+      redirect_to destination, notice: "Welcome #{reader.first_name.presence || reader.display_name}."
     end
 
     def destroy

data/app/helpers/bible270/plan_helper.rb

@@ -137,6 +137,18 @@ module Bible270
       end
     end
 
+    # Timestamps in the plan's zone rather than UTC. Dates — started_on and the
+    # like — need no conversion and are formatted inline.
+    def b270_date(time)
+      local = Bible270.local_time(time)
+      local&.strftime('%b %-d, %Y')
+    end
+
+    def b270_datetime(time)
+      local = Bible270.local_time(time)
+      local&.strftime('%b %-d, %Y at %-I:%M %p')
+    end
+
     # Only the writer may change their own words. An admin can remove a reflection
     # but not rewrite it: putting words in someone's mouth is worse than taking
     # them away, and taking them away is already the moderator's job.

data/app/mailers/bible270/broadcast.html.erb

@@ -0,0 +1,17 @@
+<!DOCTYPE html>
+<html>
+  <body style="margin:0;padding:24px;background:#ece6da;font-family:-apple-system,Segoe UI,Roboto,Helvetica,Arial,sans-serif;color:#231f18">
+    <div style="max-width:520px;margin:0 auto;background:#f4efe6;border:1px solid #d7cfbf;border-radius:14px;padding:28px">
+      <p style="margin:0 0 16px;font-size:15px">
+        <%= @reader.first_name.presence || @reader.display_name %>,
+      </p>
+
+      <div style="font-size:15px;line-height:1.6"><%= simple_format(@body) %></div>
+
+      <p style="margin:24px 0 0;padding-top:16px;border-top:1px solid #d7cfbf;font-size:12px;color:#9a9082">
+        <%= @app_name %><% if @plan_url %> ·
+        <a href="<%= @plan_url %>" style="color:#a5812c"><%= @plan_url %></a><% end %>
+      </p>
+    </div>
+  </body>
+</html>

data/app/mailers/bible270/broadcast.text.erb

@@ -0,0 +1,7 @@
+<%= @reader.first_name.presence || @reader.display_name %>,
+
+<%= @body %>
+
+--
+<%= @app_name %><% if @plan_url %>
+<%= @plan_url %><% end %>

data/app/mailers/bible270/notice_mailer.rb

@@ -31,6 +31,22 @@ module Bible270
            subject: "#{@app_name}: #{@author.display_name} mentioned you on day #{@comment.day}"
     end
 
+    # A message from whoever runs the plan to one reader. Sent individually rather
+    # than as one email with everyone in bcc: a bcc list of a hundred addresses is
+    # a spam signal, and it means nobody can be greeted by name or unsubscribe
+    # meaningfully.
+    def broadcast(reader_id:, subject:, body:)
+      @reader = Reader.find_by(id: reader_id)
+      return message.perform_deliveries = false if @reader.nil? || @reader.email.blank?
+      return message.perform_deliveries = false if subject.to_s.strip.empty? || body.to_s.strip.empty?
+
+      @body = body.to_s
+      @app_name = Bible270.config.app_name
+      @plan_url = plan_url
+
+      mail to: @reader.email, subject: subject.to_s.strip
+    end
+
   private
 
     # A mailer has no request, so the host is config.mailer_host or nothing. No
@@ -41,6 +57,10 @@ module Bible270
     # config.mount_at was worse than useless — ignored when blank, and overriding
     # the true prefix when not, so a config that disagreed with routes.rb would
     # have produced links to nowhere.
+    def plan_url
+      engine_url(:root_url)
+    end
+
     def day_url_for(day)
       engine_url(:day_url, day)
     end

data/app/models/bible270/comment.rb

@@ -3,6 +3,15 @@
 module Bible270
   # A publicly visible reflection attached to a plan day (optionally a track).
   class Comment < ApplicationRecord
+    # One heart for both states. An unliked one is the same emoji shown grey by
+    # CSS, so it cannot drift in size from a real like, and hovering it previews
+    # exactly what clicking will produce.
+    #
+    # The white heart this replaced was legible on white but nearly invisible on
+    # the paper background; an outline glyph like U+2661 is drawn by the font
+    # rather than the emoji set and needs hand-tuning to match.
+    HEART = '❤️'
+
     self.table_name = 'bible270_comments'
 
     belongs_to :reader, class_name: 'Bible270::Reader'
@@ -12,6 +21,7 @@ module Bible270
     belongs_to :parent, class_name: 'Bible270::Comment', optional: true
     has_many :replies, class_name: 'Bible270::Comment', foreign_key: :parent_id,
                        dependent: :destroy, inverse_of: :parent
+    has_many :likes, class_name: 'Bible270::Like', dependent: :destroy, inverse_of: :comment
 
     validates :day, inclusion: { in: 1..Plan::DAYS }
     validates :body, presence: true, length: { maximum: 4000 }
@@ -37,7 +47,9 @@ module Bible270
     # Only the reflections themselves: replies are rendered under their parent, so
     # listing them again at the top level would show each twice.
     scope :for_day,  ->(d) { approved.where(day: d, parent_id: nil).order(created_at: :asc) }
-    scope :threads_for_day, ->(d) { for_day(d).includes(:reader, replies: :reader) }
+    scope :threads_for_day, ->(d) {
+      for_day(d).includes(:reader, { likes: :reader }, { replies: [:reader, { likes: :reader }] })
+    }
     scope :recent, -> { approved.order(created_at: :desc) }
 
     # Every reflection is visible when written; these are the moderation actions.
@@ -46,9 +58,30 @@ module Bible270
     def hidden?  = !approved
     def reply?   = parent_id.present?
 
+    def liked_by?(reader)
+      return false if reader.nil?
+
+      likes.any? { |like| like.reader_id == reader.id }
+    end
+
+    # Toggling returns the new state, so a caller can say what happened without
+    # asking again.
+    def toggle_like!(reader)
+      existing = likes.find { |like| like.reader_id == reader.id }
+      if existing
+        existing.destroy
+        likes.reload
+        false
+      else
+        likes.create(reader: reader)
+        likes.reload
+        true
+      end
+    end
+
     # A second's grace: created_at and updated_at differ by microseconds on
     # insert, which would mark every reflection as edited.
-    def edited?  = updated_at - created_at > 1
+    def edited? = updated_at - created_at > 1
 
     # Visible replies, oldest first: a conversation reads forwards.
     def visible_replies = replies.approved.order(created_at: :asc)

data/app/models/bible270/like.rb

@@ -0,0 +1,14 @@
+# frozen_string_literal: true
+
+module Bible270
+  # A reader's heart on a reflection. There is nothing to a like but who gave it
+  # and when — the count is the number of rows, and the hover text is the name.
+  class Like < ApplicationRecord
+    self.table_name = 'bible270_likes'
+
+    belongs_to :reader, class_name: 'Bible270::Reader'
+    belongs_to :comment, class_name: 'Bible270::Comment'
+
+    validates :reader_id, uniqueness: { scope: :comment_id }
+  end
+end

data/app/models/bible270/reader.rb

@@ -8,6 +8,7 @@ module Bible270
 
     has_many :checkoffs, class_name: 'Bible270::Checkoff', dependent: :destroy
     has_many :comments,  class_name: 'Bible270::Comment',  dependent: :destroy
+    has_many :likes,     class_name: 'Bible270::Like',     dependent: :destroy, inverse_of: :reader
     belongs_to :owner, polymorphic: true, optional: true
 
     # Active Storage is optional: an app may have it disabled, and the engine has
@@ -44,10 +45,37 @@ module Bible270
       image = first_present(dig_auth(info, :image), dig_auth(info, :avatar_url))
       reader.email      = email if email.present?
       reader.avatar_url = image if image.present?
+      assign_names_from(reader, info)
       reader.save
       reader
     end
 
+    # Providers hand back one display name far more often than two fields, so the
+    # name is split when they do not. Without this an OmniAuth reader had no
+    # first_name — which made the greeting read "Welcome ." and, more seriously,
+    # made them unmentionable: Reader.mentioned_in only considers readers who have
+    # one.
+    #
+    # Only filled in when empty, so a reader who has since corrected their name on
+    # their profile does not have it overwritten at every sign-in.
+    def self.assign_names_from(reader, info)
+      return if reader.first_name.present?
+
+      given = dig_auth(info, :first_name)
+      family = dig_auth(info, :last_name)
+
+      if given.present?
+        reader.first_name = given
+        reader.last_name = family
+        return
+      end
+
+      split = Names.split_display_name(reader.display_name)
+      reader.first_name = split ? split[:first_name] : reader.display_name.to_s.split.first
+      reader.last_name = split ? split[:last_name] : nil
+    end
+    private_class_method :assign_names_from
+
     # Find or create the reader behind a verified email address. Uses the same
     # provider/uid identity columns as OmniAuth, with provider "email", so an
     # email reader is indistinguishable from any other downstream.

data/app/views/bible270/admin/comments.html.erb

@@ -18,7 +18,7 @@
         <div class="b270-cmeta">
           <strong><%= comment.reader.display_name %></strong>
           · <%= link_to "day #{comment.day}", day_path(comment.day) %>
-          · <%= comment.created_at.strftime('%b %-d, %Y') %>
+          · <%= b270_date(comment.created_at) %>
           <% if comment.reply? %>· reply to <strong><%= comment.parent&.reader&.display_name || 'a deleted reflection' %></strong><% end %>
           <% if comment.hidden? %><span class="b270-badge">Hidden</span><% end %>
         </div>

data/app/views/bible270/admin/index.html.erb

@@ -9,7 +9,7 @@
   <% if @enrollment_closed %>
     <% if @enrollment_closed_at %>
       <p class="b270-startline">Enrollment closed
-      on <%= @enrollment_closed_at.strftime('%B %-d, %Y') %><% end %>.
+      on <%= b270_date(@enrollment_closed_at) %><% end %>.
       The <%= @readers.size %> reader<%= 's' unless @readers.size == 1 %> already taking part can
       still sign in and carry on.</p>
     <%= button_to 'Open to new readers', admin_enrollment_path, method: :patch,
@@ -47,4 +47,31 @@
   <p style="color:var(--faint)">No readers yet.</p>
 <% end %>
 
+<div class="b270-panel">
+  <h2 class="b270-subhead">Write to everyone</h2>
+
+  <% if @reachable.zero? %>
+    <p class="b270-hint" style="margin-top:0">No reader has an email address yet.</p>
+  <% else %>
+    <p class="b270-hint" style="margin-top:0">
+      Goes to <strong><%= @reachable %></strong>
+      <%= @reachable == 1 ? 'reader' : 'readers' %> individually, not as one message with
+      everyone in bcc — so each is greeted by name.
+      <% if @last_broadcast_at.present? %>
+        <br>Last sent <%= b270_datetime(@last_broadcast_at) %><%
+          %><% if @last_broadcast_subject.present? %>: <em><%= @last_broadcast_subject %></em><% end %>.
+      <% end %>
+    </p>
+
+    <%= form_with url: admin_broadcast_path, method: :post, class: "b270-form",
+          data: { turbo_confirm: "Send this to #{@reachable} #{@reachable == 1 ? 'reader' : 'readers'}?" } do %>
+      <%= text_field_tag :subject, params[:subject], placeholder: "Subject", class: "b270-input" %>
+      <%= text_area_tag :body, params[:body], placeholder: "What would you like to say?", rows: 6 %>
+      <div class="row">
+        <%= submit_tag "Send", class: "b270-btn" %>
+      </div>
+    <% end %>
+  <% end %>
+</div>
+
 <p style="margin-top:18px"><%= link_to '← Back to the plan', root_path %></p>

data/app/views/bible270/admin/show.html.erb

@@ -92,6 +92,28 @@
   <%= render "bible270/shared/reflections", comments: @comments, moderate: true %>
 </div>
 
+<div class="b270-panel">
+  <h2 class="b270-subhead">Which translation they read</h2>
+  <p class="b270-hint" style="margin-top:0">
+    Reading links open in this translation.
+    <% if @reader.bible_version.blank? %>
+      They have expressed no preference, so they follow the site default
+      (<%= Bible270.config.bible_version %>).
+    <% end %>
+  </p>
+
+  <%= form_with url: admin_reader_version_path(@reader), method: :patch, class: "b270-form" do %>
+    <div class="row">
+      <%= select_tag :bible_version,
+            options_for_select([["Site default (#{Bible270.config.bible_version})", ""]] +
+                               Bible270::Translations.options,
+                               @reader.bible_version.to_s),
+            class: "b270-select", style: "width:auto" %>
+      <%= submit_tag "Save", class: "b270-btn" %>
+    </div>
+  <% end %>
+</div>
+
 <div class="b270-panel">
   <h2 class="b270-subhead">Remove</h2>
   <p class="b270-hint" style="margin-top:0">Deletes the reader together with every check-off and

data/app/views/bible270/comments/_comment.html.erb

@@ -11,9 +11,9 @@
   <div style="flex:1">
     <div class="b270-cmeta">
       <strong><%= link_to comment.reader.display_name, reader_path(comment.reader), style: "text-decoration:none" %></strong>
-      · <%= comment.created_at.strftime("%b %-d, %Y") %>
+      · <%= b270_date(comment.created_at) %>
       <% if comment.track.present? %>· <%= b270_track(comment.track)[:label] %><% end %>
-      <% if comment.edited? %>· <span class="b270-edited" title="<%= comment.updated_at.strftime('%b %-d, %Y at %-I:%M %p') %>">edited</span><% end %>
+      <% if comment.edited? %>· <span class="b270-edited" title="<%= b270_datetime(comment.updated_at) %>">edited</span><% end %>
       <% if b270_can_edit?(comment) %>
         <%= link_to "edit", day_path(comment.day, edit: comment.id, anchor: "comment-#{comment.id}"),
               class: "b270-cedit" %>
@@ -37,13 +37,16 @@
 
     <%# Only a top-level reflection can be replied to: one level deep, so a reply
         offers no Reply of its own. %>
-    <% unless reply %>
-      <% if signed_in? && comment.reader_id != current_reader.id %>
-        <div class="b270-creply">
-          <%= link_to "Reply", day_path(comment.day, reply_to: comment.id, anchor: "new_comment_form"),
-                class: "b270-linkbtn" %>
-        </div>
+    <div class="b270-cactions">
+      <%= render partial: "bible270/likes/likes", locals: { comment: comment } %>
+      <%# A reply may be liked but not replied to: one level deep. %>
+      <% if !reply && signed_in? && comment.reader_id != current_reader.id %>
+        <%= link_to "Reply", day_path(comment.day, reply_to: comment.id, anchor: "new_comment_form"),
+              class: "b270-linkbtn" %>
       <% end %>
+    </div>
+
+    <% unless reply %>
 
       <div class="b270-replies" id="replies-<%= comment.id %>">
         <%= render partial: "bible270/comments/comment",

data/app/views/bible270/comments/index.html.erb

@@ -2,7 +2,7 @@
 <% content_for :hide_day_index, true %><%# this page has its own, filtered grid %>
 
 <p class="b270-eyebrow">Reflections</p>
-<h1>What people are saying</h1>
+<h1>Shared gleanings</h1>
 
 <% if @days_with_reflections.any? %>
   <div class="b270-panel">

data/app/views/bible270/likes/_likes.html.erb

@@ -0,0 +1,27 @@
+<%# locals: (comment:) %>
+<%#
+  One heart per like, each carrying its giver's name — so a row of hearts is both
+  the count and, on hover, the list of who.
+
+  Your own like is not drawn in the row: the button is your heart, filled, and
+  saying so. Drawing both gave a reader two hearts for one like.
+%>
+<%
+  liked = signed_in? && comment.liked_by?(current_reader)
+  others = signed_in? ? comment.likes.reject { |like| like.reader_id == current_reader.id } : comment.likes
+%>
+<span class="b270-likes" id="likes-<%= comment.id %>">
+  <% others.each do |like| %><span class="b270-heart"
+      title="<%= like.reader.display_name %>"><%= Bible270::Comment::HEART %></span><% end %>
+
+  <% if signed_in? %>
+    <%= button_to like_comment_path(comment), method: :post, class: "b270-likebtn#{' liked' if liked}",
+          form: { style: "display:inline", data: { turbo_stream: true } },
+          aria: { label: liked ? "Take back your like" : "Like this reflection" },
+          title: liked ? "You liked this — tap to take it back" : "Like this reflection" do %>
+      <%= Bible270::Comment::HEART %>
+    <% end %>
+  <% elsif comment.likes.any? %>
+    <span class="b270-likecount"><%= comment.likes.size %></span>
+  <% end %>
+</span>

data/app/views/bible270/likes/toggle.turbo_stream.erb

@@ -0,0 +1,3 @@
+<%= turbo_stream.replace "likes-#{@comment.id}" do %>
+  <%= render partial: "bible270/likes/likes", locals: { comment: @comment } %>
+<% end %>

data/app/views/bible270/notice_mailer/broadcast.html.erb

@@ -0,0 +1,17 @@
+<!DOCTYPE html>
+<html>
+  <body style="margin:0;padding:24px;background:#ece6da;font-family:-apple-system,Segoe UI,Roboto,Helvetica,Arial,sans-serif;color:#231f18">
+    <div style="max-width:520px;margin:0 auto;background:#f4efe6;border:1px solid #d7cfbf;border-radius:14px;padding:28px">
+      <p style="margin:0 0 16px;font-size:15px">
+        <%= @reader.first_name.presence || @reader.display_name %>,
+      </p>
+
+      <div style="font-size:15px;line-height:1.6"><%= simple_format(@body) %></div>
+
+      <p style="margin:24px 0 0;padding-top:16px;border-top:1px solid #d7cfbf;font-size:12px;color:#9a9082">
+        <%= @app_name %><% if @plan_url %> ·
+        <a href="<%= @plan_url %>" style="color:#a5812c"><%= @plan_url %></a><% end %>
+      </p>
+    </div>
+  </body>
+</html>

data/app/views/bible270/notice_mailer/broadcast.text.erb

@@ -0,0 +1,7 @@
+<%= @reader.first_name.presence || @reader.display_name %>,
+
+<%= @body %>
+
+--
+<%= @app_name %><% if @plan_url %>
+<%= @plan_url %><% end %>

data/app/views/bible270/notice_mailer/new_reader.html.erb

@@ -11,7 +11,7 @@
         <tr><td style="padding:3px 12px 3px 0;color:#6d6558">Name</td><td><%= @reader.display_name %></td></tr>
         <tr><td style="padding:3px 12px 3px 0;color:#6d6558">Email</td><td><%= @reader.email.presence || '(none given)' %></td></tr>
         <tr><td style="padding:3px 12px 3px 0;color:#6d6558">Signed in with</td><td><%= @reader.provider.presence || 'the host application' %></td></tr>
-        <tr><td style="padding:3px 12px 3px 0;color:#6d6558">Joined</td><td><%= @reader.created_at.strftime('%B %-d, %Y at %-I:%M %p') %></td></tr>
+        <tr><td style="padding:3px 12px 3px 0;color:#6d6558">Joined</td><td><%= Bible270.local_time(@reader.created_at).strftime('%B %-d, %Y at %-I:%M %p') %></td></tr>
       </table>
 
       <p style="margin:18px 0 0;font-size:14px;color:#6d6558">

data/app/views/bible270/notice_mailer/new_reader.text.erb

@@ -3,7 +3,7 @@
   Name:        <%= @reader.display_name %>
   Email:       <%= @reader.email.presence || '(none given)' %>
   Signed in with: <%= @reader.provider.presence || 'the host application' %>
-  Joined:      <%= @reader.created_at.strftime('%B %-d, %Y at %-I:%M %p') %>
+  Joined:      <%= Bible270.local_time(@reader.created_at).strftime('%B %-d, %Y at %-I:%M %p') %>
 
 That makes <%= @total %> reader<%= 's' unless @total == 1 %> taking part.
 <% if @admin_url %>

data/app/views/bible270/readers/index.html.erb

@@ -11,7 +11,7 @@
         <%= b270_avatar(r, size: 40) %>
         <div>
           <div class="name"><%= link_to r.display_name, reader_path(r), style: "text-decoration:none" %></div>
-          <div class="sub"><%= @days_completed[r.id] %> of <%= Bible270::Plan::DAYS %> days · joined <%= r.created_at.strftime("%b %Y") %></div>
+          <div class="sub"><%= @days_completed[r.id] %> of <%= Bible270::Plan::DAYS %> days · joined <%= Bible270.local_time(r.created_at).strftime("%b %Y") %></div>
         </div>
         <div class="stat">
           <span class="big"><%= (@days_completed[r.id].to_f / Bible270::Plan::DAYS * 100).round %>%</span>

data/app/views/bible270/shared/_header.html.erb

@@ -12,11 +12,11 @@
     <!--<%= link_to "Plan", root_path %>-->
     <%= link_to "People", community_path %>
     <%# Public, like the community page: anyone may read reflections. %>
-    <%= link_to "Reflections", reflections_path %>
+    <%= link_to "Posts", reflections_path %>
     <% if signed_in? %>
-      <% if Bible270.config.admin?(current_reader) %><%= link_to "Admin", admin_path %><% end %>
       <%= link_to "Progress", progress_path %>
       <%= link_to "Profile", profile_path %>
+      <% if Bible270.config.admin?(current_reader) %><%= link_to "Admin", admin_path %><% end %>
       <%= button_to "Sign out", sign_out_path, method: :delete, class: "b270-linkbtn",
             form: { style: "display:inline" } %>
     <% elsif Bible270.config.any_sign_in_method? %>

data/app/views/bible270/shared/_reflections.html.erb

@@ -19,7 +19,7 @@
       <div style="flex:1">
         <div class="b270-cmeta">
           <%= link_to "Day #{comment.day}", day_path(comment.day) %>
-          · <%= comment.created_at.strftime("%b %-d, %Y") %>
+          · <%= b270_date(comment.created_at) %>
           <% if moderate && comment.hidden? %><span class="b270-badge">Hidden</span><% end %>
         </div>
 

data/app/views/bible270/shared/_styles.html.erb

@@ -161,6 +161,17 @@
   .b270-replying strong{color:var(--ink)}
   .b270-mention{color:var(--teal);font-weight:600;text-decoration:none}
   .b270-mention:hover{text-decoration:underline}
+  .b270-cactions{display:flex;align-items:center;gap:12px;margin-top:6px;flex-wrap:wrap}
+  .b270-likes{display:inline-flex;align-items:center;gap:2px}
+  .b270-heart{font-size:13px;line-height:1;cursor:default}
+  /* Same size as a heart in the row: the button is one of the hearts, not a
+     control sitting beside them. Unliked, it is the same emoji desaturated —
+     visible against the paper, and hovering shows what clicking will give. */
+  .b270-likebtn{background:none;border:0;padding:0 2px;font-size:13px;line-height:1;cursor:pointer;
+                filter:grayscale(1);opacity:.5;transition:filter .12s,opacity .12s}
+  .b270-likebtn:hover{filter:none;opacity:1}
+  .b270-likebtn.liked{filter:none;opacity:1}
+  .b270-likecount{font-size:11.5px;color:var(--faint);margin-left:2px}
   .b270-creply{margin-top:6px}
   .b270-cbody{margin:3px 0 0;white-space:pre-wrap}
   .b270-editform{margin-top:6px}
@@ -172,6 +183,7 @@
   .b270-comment.b270-editing{background:rgba(165,129,44,.05);border-radius:10px;padding:12px}
   .b270-editing textarea{min-height:64px}
   .b270-cdel{font-size:12px;color:var(--oxblood);background:none;border:none;cursor:pointer;padding:0;margin-left:8px}
+  .b270-input{width:100%;padding:9px 11px;border:1px solid var(--line);border-radius:9px;background:#fff;font:inherit;font-size:14px;margin-bottom:8px}
   .b270-form textarea{width:100%;font:inherit;font-size:15px;border:1px solid var(--line);border-radius:10px;padding:11px 13px;background:#fff;resize:vertical;min-height:76px}
   .b270-form .row{display:flex;align-items:center;gap:10px;margin-top:10px}
   .b270 .b270-btn{border:none;border-radius:10px;padding:8px 16px;margin-top:4px;font:inherit;font-weight:600;font-size:14px;cursor:pointer;background:var(--ink);color:var(--paper);text-decoration:none;display:inline-block}

data/config/routes.rb

@@ -12,6 +12,7 @@ Bible270::Engine.routes.draw do
                                    constraints: { day: %r{\d+} }
   post   'day/:day/comments',      to: 'comments#create',  as: :day_comments,
                                    constraints: { day: %r{\d+} }
+  post   'comments/:id/like',      to: 'likes#toggle',     as: :like_comment
   patch  'comments/:id',           to: 'comments#update'
   delete 'comments/:id',           to: 'comments#destroy', as: :comment
 
@@ -21,10 +22,12 @@ Bible270::Engine.routes.draw do
   delete 'admin/readers/:id',            to: 'admin#destroy'
   patch  'admin/readers/:id/profile',    to: 'admin#update_profile',   as: :admin_reader_profile
   delete 'admin/readers/:id/avatar',     to: 'admin#remove_avatar',    as: :admin_reader_avatar
+  patch  'admin/readers/:id/version',    to: 'admin#update_bible_version', as: :admin_reader_version
   patch  'admin/readers/:id/start',      to: 'admin#update_start',     as: :admin_reader_start
   patch  'admin/readers/:id/through',    to: 'admin#complete_through', as: :admin_reader_through
   post   'admin/readers/:id/days/:day',  to: 'admin#toggle_day',       as: :admin_reader_day
 
+  post   'admin/broadcast',              to: 'admin#broadcast',        as: :admin_broadcast
   patch  'admin/enrollment',             to: 'admin#update_enrollment', as: :admin_enrollment
   get    'admin/comments',               to: 'admin#comments',         as: :admin_comments
   patch  'admin/comments/:id/hide',      to: 'admin#hide_comment',     as: :admin_hide_comment

data/db/migrate/20260101000013_create_bible270_likes.rb

@@ -0,0 +1,21 @@
+# frozen_string_literal: true
+
+class CreateBible270Likes < ActiveRecord::Migration[7.0]
+  def change
+    create_table :bible270_likes do |t|
+      t.references :reader, null: false, index: false,
+                            foreign_key: { to_table: :bible270_readers }
+      t.references :comment, null: false, index: false,
+                             foreign_key: { to_table: :bible270_comments }
+      t.timestamps
+    end
+
+    # One like per reader per reflection, enforced by the database as well as the
+    # model: a double-tap should not become two hearts.
+    add_index :bible270_likes, %i[reader_id comment_id], unique: true,
+                               name: 'idx_b270_likes_unique'
+    # Reading a thread asks for a comment's likes, so that is the index that earns
+    # its keep.
+    add_index :bible270_likes, :comment_id, name: 'idx_b270_likes_comment'
+  end
+end

data/lib/bible270/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Bible270
-  VERSION = '0.17.0'
+  VERSION = '0.17.2'
 end

data/lib/bible270.rb

@@ -22,10 +22,24 @@ module_function
   # it; Date.today follows the machine. A self-hosted reading plan wants the
   # latter, so that is the default, with config.time_zone to override.
   def today
-    zone = config.time_zone
-    return Date.today if zone.nil? || zone.to_s.empty?
+    local_time(Time.now).to_date
+  end
 
-    found = Time.respond_to?(:find_zone) ? Time.find_zone(zone) : nil
-    found ? found.today : Date.today
+  # A stored timestamp in the plan's own zone.
+  #
+  # created_at and friends come back in Rails' Time.zone, which is UTC unless the
+  # host app sets it — so a reflection written at 5pm in Los Angeles was being
+  # shown as the next day. This is the same rule Bible270.today follows: the
+  # machine's zone, or config.time_zone when one is given.
+  def local_time(time)
+    return nil if time.nil?
+
+    zone = config.time_zone
+    if zone.to_s.empty?
+      time.getlocal
+    else
+      found = Time.respond_to?(:find_zone) ? Time.find_zone(zone) : nil
+      found ? time.in_time_zone(found) : time.getlocal
+    end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: bible270
 version: !ruby/object:Gem::Version
-  version: 0.17.0
+  version: 0.17.2
 platform: ruby
 authors:
 - Andrew vonderLuft
@@ -85,16 +85,20 @@ files:
 - app/controllers/bible270/checkoffs_controller.rb
 - app/controllers/bible270/comments_controller.rb
 - app/controllers/bible270/days_controller.rb
+- app/controllers/bible270/likes_controller.rb
 - app/controllers/bible270/profiles_controller.rb
 - app/controllers/bible270/readers_controller.rb
 - app/controllers/bible270/sessions_controller.rb
 - app/helpers/bible270/plan_helper.rb
 - app/mailers/bible270/application_mailer.rb
+- app/mailers/bible270/broadcast.html.erb
+- app/mailers/bible270/broadcast.text.erb
 - app/mailers/bible270/notice_mailer.rb
 - app/mailers/bible270/sign_in_mailer.rb
 - app/models/bible270/application_record.rb
 - app/models/bible270/checkoff.rb
 - app/models/bible270/comment.rb
+- app/models/bible270/like.rb
 - app/models/bible270/reader.rb
 - app/models/bible270/setting.rb
 - app/models/bible270/sign_in_token.rb
@@ -115,6 +119,10 @@ files:
 - app/views/bible270/days/_start_date.html.erb
 - app/views/bible270/days/index.html.erb
 - app/views/bible270/days/show.html.erb
+- app/views/bible270/likes/_likes.html.erb
+- app/views/bible270/likes/toggle.turbo_stream.erb
+- app/views/bible270/notice_mailer/broadcast.html.erb
+- app/views/bible270/notice_mailer/broadcast.text.erb
 - app/views/bible270/notice_mailer/mentioned.html.erb
 - app/views/bible270/notice_mailer/mentioned.text.erb
 - app/views/bible270/notice_mailer/new_reader.html.erb
@@ -148,6 +156,7 @@ files:
 - db/migrate/20260101000010_add_remember_token_to_bible270_readers.rb
 - db/migrate/20260101000011_add_mention_notices_to_bible270_readers.rb
 - db/migrate/20260101000012_add_parent_to_bible270_comments.rb
+- db/migrate/20260101000013_create_bible270_likes.rb
 - lib/bible270.rb
 - lib/bible270/avatars.rb
 - lib/bible270/configuration.rb