i18n_feedback 0.8.0 → 0.8.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 675e56ff8164412361ecd15886a56ef5eb7a97fdda9a766246b0d690c602d2e1
-  data.tar.gz: 36c511550f1b042de8052e995afeab43c27645410d0dea5246a4ffde9e9d598f
+  metadata.gz: 7961f050a8f18c3136f8bac4be58014c6d137b408539d9bbb589e3756c024674
+  data.tar.gz: eb772bb6ca0dbd76d2638f6d5de2fb455a3c1ac737b640856fff79677f50f873
 SHA512:
-  metadata.gz: fc37ff253e157cc2c2b619e8007bdbe698a1669a96c19c5243b5e326db8185344ee1fa49bdaf40f355240fb6322d90d84bc2e7944bef84b19c77bc381bdcf859
-  data.tar.gz: f9136776bb4abeb8bb1a6271e8f924178486fad4ab3923eed3e9826110daa56501ca8773962ffc8caee9b340c9b294c6747b500c38b989ed6e412ada137d6237
+  metadata.gz: a586da5d40cda89a58312e02bdeaaa27af64bc9c7013fca26cbd94bf7dd01da8109dfb2ce7417bf07fae3525837740d85441a6e47e3c65cebcb85dfe4125c53f
+  data.tar.gz: 1978b54ebb78dadfbc90d09e97d4a8e656c2f77d34d134301e0844efbcf3848c13d95f8b2ea5311ef8f872f21439852c6c67564c981b3b42d08f2eee085376bc

data/CHANGELOG.md

@@ -2,6 +2,16 @@
 
 ## [Unreleased]
 
+## [0.8.1]
+
+- Make the dashboard **read-only**. Apply / Reject / Reopen / Delete only ever
+  flipped the stored `status` — they never touched the host's locale files, so
+  offering them implied a capability the gem doesn't have. The buttons and their
+  `update` / `destroy` routes and actions are removed; the dashboard now just
+  lists suggestions for review, and you make the edits in your own locale files.
+  A suggestion's `status` still exists on the model for out-of-band tracking and
+  as groundwork for a future "apply to locale file" feature.
+
 ## [0.8.0]
 
 - Add a built-in **triage dashboard**, mounted at the engine root (your

data/README.md

@@ -237,13 +237,19 @@ blue accent stays the same in both.
 
 ## Reviewing suggestions
 
-### Triage dashboard
+### Review dashboard
 
 Mounted at your `mount_path` (default `/i18n_feedback`), the engine root is a
-built-in triage dashboard: pending / applied / rejected tabs with counts, a
-per-locale filter, each suggestion shown as current-vs-proposed, and one-click
-**Apply** / **Reject** / **Reopen** / **Delete**. It's plain server-rendered
-HTML with its own styling — no host assets or JS framework needed.
+built-in **read-only** review board: pending / applied / rejected tabs with
+counts, a per-locale filter, and each suggestion shown as current-vs-proposed
+with its comment and author. It's plain server-rendered HTML with its own
+styling — no host assets or JS framework needed.
+
+It is deliberately read-only. The gem never writes to your locale files, so it
+doesn't pretend to: you review the suggestions here, then make the edits in your
+own `config/locales/*.yml`. (A suggestion's `status` still exists on the model
+for your own tracking — set it from the console — and is the groundwork for a
+future "apply to locale file" feature.)
 
 It has its own gate, `config.authorize_admin`, **defaulting to development
 only** — so a fresh install never exposes it in production. Point it at your own

data/app/controllers/i18n_feedback/suggestions_controller.rb

@@ -4,10 +4,9 @@ module I18nFeedback
   class SuggestionsController < ApplicationController
     PER_PAGE = 50
 
-    # Public widget API is available-gated; the triage dashboard is admin-gated.
+    # Public widget API is available-gated; the read-only dashboard is admin-gated.
     before_action :require_available, only: %i[context create]
-    before_action :require_admin, only: %i[index update destroy]
-    before_action :set_suggestion, only: %i[update destroy]
+    before_action :require_admin, only: :index
 
     layout 'i18n_feedback/application', only: :index
 
@@ -40,19 +39,6 @@ module I18nFeedback
       @suggestions = rows.first(PER_PAGE)
     end
 
-    def update
-      # status arrives from a dashboard button; ignore anything not a real state
-      # (an AR enum would otherwise raise on an unknown value).
-      status = params[:status].to_s
-      @suggestion.update!(status: status) if Suggestion::STATUSES.include?(status)
-      redirect_back fallback_location: root_path, status: :see_other
-    end
-
-    def destroy
-      @suggestion.destroy!
-      redirect_back fallback_location: root_path, status: :see_other
-    end
-
     # --- widget API (public) -------------------------------------------------
 
     # Pending suggestions for one key/locale, shown as read-only context when the
@@ -81,10 +67,6 @@ module I18nFeedback
 
     private
 
-    def set_suggestion
-      @suggestion = Suggestion.find(params[:id])
-    end
-
     def attribute_author(suggestion)
       author = current_author
       return unless author

data/app/views/i18n_feedback/suggestions/index.html.erb

@@ -53,24 +53,6 @@
       <% if suggestion.comment.present? %>
         <div class="comment" dir="auto"><%= suggestion.comment %></div>
       <% end %>
-
-      <div class="actions">
-        <% unless suggestion.status_applied? %>
-          <%= button_to t('i18n_feedback.dashboard.apply', default: 'Apply'),
-                        suggestion_path(suggestion, status: 'applied'), method: :patch, class: 'primary' %>
-        <% end %>
-        <% unless suggestion.status_rejected? %>
-          <%= button_to t('i18n_feedback.dashboard.reject', default: 'Reject'),
-                        suggestion_path(suggestion, status: 'rejected'), method: :patch %>
-        <% end %>
-        <% unless suggestion.status_pending? %>
-          <%= button_to t('i18n_feedback.dashboard.reopen', default: 'Reopen'),
-                        suggestion_path(suggestion, status: 'pending'), method: :patch %>
-        <% end %>
-        <%= button_to t('i18n_feedback.dashboard.delete', default: 'Delete'),
-                      suggestion_path(suggestion), method: :delete, class: 'danger',
-                      form: { data: { turbo_confirm: t('i18n_feedback.dashboard.delete_confirm', default: 'Delete this suggestion?') } } %>
-      </div>
     </article>
   <% end %>
 

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

@@ -75,15 +75,11 @@
       dd .new { font-weight: 600; }
       .comment { margin-top: 10px; padding: 8px 10px; background: var(--code); border-radius: 8px; font-size: 14px; overflow-wrap: anywhere; }
 
-      .actions { display: flex; gap: 8px; flex-wrap: wrap; margin-top: 14px; }
-      .actions form { display: inline; margin: 0; }
       button {
         padding: 7px 13px; border: 1px solid var(--border); border-radius: 8px; cursor: pointer;
         background: var(--surface); color: var(--text); font: inherit; font-size: 14px; font-weight: 600;
       }
       button:hover { border-color: var(--muted); }
-      button.primary { background: var(--accent); border-color: var(--accent); color: var(--accent-text); }
-      button.danger { color: var(--rejected); border-color: transparent; }
       :focus-visible { outline: 2px solid var(--accent); outline-offset: 2px; }
 
       .empty { padding: 48px 16px; text-align: center; color: var(--muted); border: 1px dashed var(--border); border-radius: 12px; }

data/config/locales/i18n_feedback.en.yml

@@ -29,11 +29,6 @@ en:
       current: "Current"
       suggested: "Suggested"
       ago: "ago"
-      apply: "Apply"
-      reject: "Reject"
-      reopen: "Reopen"
-      delete: "Delete"
-      delete_confirm: "Delete this suggestion?"
       empty: "Nothing here yet."
       newer: "← Newer"
       older: "Older →"

data/config/routes.rb

@@ -1,9 +1,11 @@
 # frozen_string_literal: true
 
 I18nFeedback::Engine.routes.draw do
-  # index/update/destroy are the admin triage dashboard; create + the `context`
-  # collection route are the widget's public API (see SuggestionsController).
-  resources :suggestions, only: %i[index create update destroy] do
+  # index is the read-only admin dashboard; create + the `context` collection
+  # route are the widget's public API (see SuggestionsController). There is
+  # deliberately no update/destroy — the tool never mutates suggestions from the
+  # UI, since it can't write to the host's locale files.
+  resources :suggestions, only: %i[index create] do
     get :context, on: :collection
   end
 

data/lib/i18n_feedback/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module I18nFeedback
-  VERSION = '0.8.0'
+  VERSION = '0.8.1'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: i18n_feedback
 version: !ruby/object:Gem::Version
-  version: 0.8.0
+  version: 0.8.1
 platform: ruby
 authors:
 - Yaroslav Shmarov