dokno 1.1.1 → 1.4.1

data/app/views/dokno/categories/index.html.erb

@@ -1,26 +1,33 @@
+<% if !current_page?(up_for_review_path) && (@category.blank? || @search_term.present?) %>
+  <div class="text-center m-auto mb-10 w-full max-w-screen-xl">
+    <% if @search_term.present? %>
+      <div class="text-gray-600 text-2xl uppercase">
+        <%= @total_records.positive? ? "#{@total_records} #{'article'.pluralize(@total_records)}" : 'No articles' %>
+        found containing the search term
+        <div class="text-4xl leading-tight"><span class="font-serif">&ldquo;</span> <%= @search_term %> <span class="font-serif">&rdquo;</span> </div>
+      </div>
+    <% else %>
+      <div class="text-gray-600 text-2xl">
+        Browse or search
+        <% if (article_count = Dokno::Article.count) > 1 %>
+          <%= number_with_delimiter(article_count, delimiter: ',') %> articles in
+        <% end %>
+        the
+      </div>
+      <div class="text-gray-800 text-4xl leading-tight uppercase"><%= Dokno.config.app_name %> knowledgebase</div>
+    <% end %>
+  </div>
+<% end %>
+
 <% if @category&.parent.present? %>
   <div class="text-gray-500 mb-5">
-    <div><%= @category.breadcrumb %></div>
+    <div><span class="text-gray-500 mr-1">Under</span> <%= @category.breadcrumb(search_term: @search_term, order: @order, hide_self: true) %></div>
   </div>
 <% end %>
 
-<div class="flex items-center mb-10">
-  <% if Dokno::Category.exists? %>
-    <div class="w-1/2 pr-5">
-
-      <select aria-label="Category" name="category" id="category" size="1" class="rounded text-xl shadow-inner bg-gray-100 p-2 w-full max-w-full" onchange="changeCategory(this.value, elem('#search_term').value, '<%= @order %>');">
-        <option value="">Select a category</option>
-        <% cache Dokno::Category do %>
-          <%= Dokno::Category.select_option_markup.html_safe %>
-        <% end %>
-      </select>
-
-    </div>
-  <% end %>
-  <div class="w-<%= Dokno::Category.exists? ? '1/2 pl-5' : 'full' %>">
-    <input onsearch="search(this.value, '<%= @order %>');" placeholder="Search article content, titles, and slugs" type="search" name="search_term" id="search_term" value="<%= @search_term %>" class="rounded text-xl shadow-inner bg-gray-100 p-2 w-full" />
-  </div>
-</div>
+<% if !current_page?(up_for_review_path) && (Dokno::Category.exists? || Dokno::Article.exists?) %>
+  <%= render 'partials/category_header' %>
+<% end %>
 
 <% if @articles.blank? %>
 
@@ -46,10 +53,10 @@
     </div>
     <div class="w-1/3 text-right">
       <i data-feather="corner-right-down" class="h-5 inline-block" title="Sort order"></i>
-      <a class="ml-3 pb-1 <%= 'border-b-2 border-blue-900' if @order == 'updated' %>" href="?search_term=<%= CGI.escape @search_term.to_s %>&order=updated">Updated</a>
-      <a class="ml-3 pb-1 <%= 'border-b-2 border-blue-900' if @order == 'newest' %>" href="?search_term=<%= CGI.escape @search_term.to_s %>&order=newest">Newest</a>
-      <a class="ml-3 pb-1 <%= 'border-b-2 border-blue-900' if @order == 'views' %>" href="?search_term=<%= CGI.escape @search_term.to_s %>&order=views">Views</a>
-      <a class="ml-3 pb-1 <%= 'border-b-2 border-blue-900' if @order == 'alpha' %>" href="?search_term=<%= CGI.escape @search_term.to_s %>&order=alpha">Title</a>
+      <a id="dokno-order-link-updated" class="ml-3 pb-1 <%= 'border-b-2 border-blue-900' if @order == 'updated' %>" href="?search_term=<%= CGI.escape @search_term.to_s %>&order=updated">Updated</a>
+      <a id="dokno-order-link-newest" class="ml-3 pb-1 <%= 'border-b-2 border-blue-900' if @order == 'newest' %>" href="?search_term=<%= CGI.escape @search_term.to_s %>&order=newest">Newest</a>
+      <a id="dokno-order-link-views" class="ml-3 pb-1 <%= 'border-b-2 border-blue-900' if @order == 'views' %>" href="?search_term=<%= CGI.escape @search_term.to_s %>&order=views">Views</a>
+      <a id="dokno-order-link-alpha" class="ml-3 pb-1 <%= 'border-b-2 border-blue-900' if @order == 'alpha' %>" href="?search_term=<%= CGI.escape @search_term.to_s %>&order=alpha">Title</a>
     </div>
   </div>
 
@@ -58,32 +65,43 @@
       <section class="border-t border-gray-300 py-10 text-xl flex">
         <div class="w-1/3 pr-10">
           <div class="flex">
-            <div class="no-print w-10 text-gray-300"><i data-feather="chevron-right" class="inline-block"></i></div>
+            <div title="<%= 'Starred article' if article.starred %>" class="no-print w-10 text-gray-300"><i data-feather="<%= article.starred ? 'star' : 'chevron-right' %>" class="inline-block"></i></div>
             <div class="w-full">
-              <a class="" href="<%= article_path article.slug %>?search_term=<%= @search_term %>" title="View article"><%= article.title %></a>
+              <a class="dokno-article-title <% unless article.active %>text-gray-500 italic<% end %>" href="<%= article_path article.slug %>?search_term=<%= @search_term %>&cat_code=<%= @category&.code %>&order=<%= @order %>" title="View article"><%= article.title %></a>
             </div>
           </div>
         </div>
         <div class="dokno-article-summary w-2/3 <% unless article.active %>text-gray-500 italic<% end %>">
-          <% unless article.active %>
-            <div class="bg-yellow-700 p-4 mb-5 rounded text-lg border-t-4 border-yellow-900 text-white font-base not-italic">
-              <i data-feather="alert-circle" class="inline-block"></i> This article is no longer active
-            </div>
-          <% end %>
+          <div class="dokno-article-content-highlight mb-2"><%= article.summary.presence || 'No summary provided' %></div>
 
-          <span class="dokno-article-content-highlight"><%= article.summary.presence || 'No summary provided' %></span>
+          <div class="text-base text-gray-500">
+            <%= article.category_name_list(context_category_id: @category&.id, order: @order, search_term: @search_term) %>
+          </div>
 
           <% unless @order == 'alpha' %>
-            <div class="text-base mt-2">
+            <div class="text-base">
               <% if @order == 'views' %>
-                <div class="text-gray-500">Viewed <%= number_with_delimiter(article.views, delimiter: ',') %> <%= 'time'.pluralize(article.views) %></div>
+                <div class="text-gray-500">This article was viewed <%= number_with_delimiter(article.views, delimiter: ',') %> <%= 'time'.pluralize(article.views) %></div>
               <% elsif @order == 'updated' %>
-                <div class="text-gray-500">Last updated <%= time_ago_in_words article.updated_at %> ago</div>
+                <div class="text-gray-500">This article was last updated <%= time_ago_in_words article.updated_at %> ago</div>
               <% elsif @order == 'newest' %>
-                <div class="text-gray-500">Added <%= time_ago_in_words article.created_at %> ago</div>
+                <div class="text-gray-500">This article was added <%= time_ago_in_words article.created_at %> ago</div>
               <% end %>
             </div>
           <% end %>
+
+          <% if !article.active %>
+            <div class="bg-yellow-700 p-4 mt-5 rounded text-lg border-t-4 border-yellow-900 text-white font-base not-italic">
+              <i data-feather="info" class="inline-block mr-1"></i> This article is no longer active
+            </div>
+          <% end %>
+
+          <% if article.up_for_review? %>
+            <div class="bg-<%= article.review_due_days.negative? ? 'red' : 'gray' %>-800 p-4 mt-5 rounded text-lg border-t-4 border-<%= article.review_due_days.negative? ? 'red' : 'gray' %>-900 text-white font-base not-italic">
+              <i data-feather="bell" class="inline-block mr-1"></i> <%= article.review_due_days_string %>
+            </div>
+          <% end %>
+
         </div>
       </section>
     <% end %>
@@ -97,11 +115,6 @@
   </div>
 <% end %>
 
-<script>
-  // Client-side select of cached select list
-  selectOption('category', '<%= j @category&.code %>');
-</script>
-
-<% if @search_term.present? %>
-  <script> highlightTerm(['<%= j @search_term.strip %>'], 'dokno-article-content-highlight'); </script>
+<% if @search_term.present? && @articles.blank? %>
+  <script> handleSearchHotKey(); </script>
 <% end %>

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

@@ -2,6 +2,8 @@
 <html>
 <head>
   <title><%= Dokno.config.app_name %> KNOWLEDGEBASE</title>
+
+  <meta name="viewport" content="width=device-width,initial-scale=1">
   <%= csrf_meta_tags %>
   <%= csp_meta_tag %>
 
@@ -12,7 +14,8 @@
     var dokno__base_path = '<%= root_path %>';
   </script>
 </head>
-<body class="bg-white font-sans font-light subpixel-antialiased">
+<body class="bg-white font-sans font-light subpixel-antialiased text-lg">
+
 
   <nav id="dokno-nav-container" class="bg-blue-900 text-white py-10 px-16 text-lg">
     <div class="flex items-center m-auto w-full max-w-screen-xl">
@@ -24,12 +27,12 @@
       <div class="w-2/3 text-right">
         <% if can_edit? %>
           <% if action_name != 'new' %>
-            <button title="Add a new article" class="bg-gray-700 text-gray-300 hover:text-white hover:bg-gray-900 rounded ml-3 py-2 px-3 font-bold text-base" onclick="location.href='<%= new_article_path %>/?category_code=<%= @category&.code %>';"><i data-feather="plus-circle" class="inline h-5"></i> ARTICLE</button>
-            <button title="Add a new category" class="bg-gray-700 text-gray-300 hover:text-white hover:bg-gray-900 rounded ml-3 py-2 px-3 font-bold text-base" onclick="location.href='<%= new_category_path %>/?parent_category_code=<%= @category&.code %>';"><i data-feather="plus-circle" class="inline h-5"></i> CATEGORY</button>
+            <button title="Add a new article" class="bg-gray-700 text-gray-300 hover:text-white hover:bg-gray-900 rounded ml-3 py-2 px-3 font-bold text-base" onclick="location.href='<%= new_article_path %>/?category_code=<%= @category&.code %>';"><i data-feather="plus" class="inline h-5"></i> ARTICLE</button>
+            <button title="Add a new category" class="bg-gray-700 text-gray-300 hover:text-white hover:bg-gray-900 rounded ml-3 py-2 px-3 font-bold text-base" onclick="location.href='<%= new_category_path %>/?parent_category_code=<%= @category&.code %>';"><i data-feather="plus" class="inline h-5"></i> CATEGORY</button>
           <% end %>
 
           <% if @category&.persisted? && action_name != 'edit' %>
-            <button title="Edit this category" class="bg-gray-700 text-gray-100 hover:text-white hover:bg-gray-900 rounded ml-3 py-2 px-3 font-bold text-base" onclick="location.href='<%= edit_category_path(@category) %>';"><i data-feather="edit" class="inline h-5"></i> CATEGORY</button>
+            <button title="Edit this category" class="bg-gray-700 text-gray-100 hover:text-white hover:bg-gray-900 rounded ml-3 py-2 px-3 font-bold text-base" onclick="location.href='<%= edit_category_path(@category) %>';"><i data-feather="edit-2" class="inline h-5"></i> CATEGORY</button>
           <% end %>
         <% end %>
 
@@ -40,40 +43,22 @@
     </div>
   </nav>
 
-  <main class="py-10 px-16">
-    <% if @article.blank? && (@category.blank? || @search_term.present?) %>
-      <div class="text-center m-auto mb-10 w-full max-w-screen-xl">
-        <% if @search_term.present? %>
-          <div class="text-gray-600 text-2xl uppercase">
-            <%= @total_records.positive? ? "#{@total_records} #{'article'.pluralize(@total_records)}" : 'No articles' %>
-            found containing the search term
-            <div class="text-4xl leading-tight"><span class="font-serif">&ldquo;</span> <%= @search_term %> <span class="font-serif">&rdquo;</span> </div>
-          </div>
-        <% else %>
-          <div class="text-gray-600 text-2xl">
-            Browse or search
-            <% if (article_count = Dokno::Article.count) > 1 %>
-              <%= number_with_delimiter(article_count, delimiter: ',') %> articles in
-            <% end %>
-            the
-          </div>
-          <div class="text-gray-800 text-4xl leading-tight uppercase"><%= Dokno.config.app_name %> knowledgebase</div>
-        <% end %>
-      </div>
-    <% end %>
 
-    <div id="dokno-content-container" class="w-full max-w-screen-xl m-auto print-this">
-      <% flash.each do |color, msg| %>
-        <div class="bg-<%= color %>-700 p-4 mb-5 rounded text-lg border-t-4 border-<%= color %>-900 text-white font-base">
-          <i data-feather="<%= (color == 'green' ? 'check-circle' : (color == 'yellow' ? 'alert-circle' : 'x-circle')) %>" class="inline mr-2"></i>
-          <%= msg %>
-        </div>
-      <% end %>
+  <% flash.each do |type, msg| %>
+    <div class="bg-<%= type %>-800 text-lg text-white font-base py-10 px-16 text-center">
+      <i data-feather="<%= (type == 'green' ? 'smile' : (type == 'yellow' ? 'info' : (type == 'gray' ? 'bell' : 'frown'))) %>" class="inline mr-1"></i>
+      <%= sanitize(msg, tags: %w[a], attributes: %w[href class]) %>
+    </div>
+  <% end %>
 
+
+  <main class="py-10 px-16">
+    <div id="dokno-content-container" class="w-full max-w-screen-xl m-auto print-this">
       <%= yield %>
     </div>
   </main>
 
+
   <footer id="dokno-footer-container">
     <% if @article.present? && action_name == 'show' %>
       <div id="dokno-article-log-container" data-category-id="<%= @category&.id %>" data-article-id="<%= @article.id %>">
@@ -81,10 +66,24 @@
       </div>
     <% end %>
 
+    <% if @show_up_for_review && (up_for_review_count = Dokno::Article.up_for_review.count).positive? %>
+      <div id="dokno-articles-up-for-review-container">
+        <div class="py-10 px-16 bg-gray-900">
+          <div class="w-full max-w-screen-xl m-auto">
+            <div class="text-xl text-white cursor-pointer" onclick="location.href='<%= up_for_review_path %>';">
+              <i data-feather="bell" class="inline mr-1"></i>
+              There <%= "#{up_for_review_count == 1 ? 'is' : 'are'} #{up_for_review_count}" %> <%= 'article'.pluralize(up_for_review_count) %> up for accuracy / relevance review
+            </div>
+          </div>
+        </div>
+      </div>
+    <% end %>
+
     <div class="py-10 px-16 text-gray-400 bg-blue-900">
       <div class="w-full max-w-screen-xl m-auto flex">
         <div class="w-1/2">
-          <a target="_blank" href="https://github.com/cpayne624/dokno" title="Knowledgebase">dokno</a>
+          <i data-feather="github" class="inline mr-1"></i>
+          <a target="_blank" href="https://github.com/cpayne624/dokno" title="Dokno on GitHub">dokno</a>
         </div>
         <div class="w-1/2 text-right">
           <% if user.present? %>
@@ -101,5 +100,9 @@
   </footer>
 
   <%= javascript_include_tag 'init' %>
+
+  <% if @search_term.present? %>
+    <script> highlightTerm(['<%= j @search_term.strip %>'], 'dokno-article-content-highlight'); </script>
+  <% end %>
 </body>
 </html>

data/app/views/partials/_category_header.html.erb

@@ -0,0 +1,30 @@
+<div class="no-print flex items-center mb-10">
+  <% if Dokno::Category.exists? %>
+    <div class="w-1/2 pr-5">
+      <select aria-label="Category" name="category" id="category" size="1" class="rounded text-xl shadow-inner bg-gray-100 p-2 w-full max-w-full" onchange="applyCategoryCriteria(this.value, elem('#search_term').value, '<%= @order %>');">
+        <option value="">Uncategorized</option>
+
+        <optgroup label="Categories">
+          <% cache Dokno::Category do %>
+            <%= Dokno::Category.select_option_markup.html_safe %>
+          <% end %>
+        </optgroup>
+      </select>
+    </div>
+
+    <script>
+      // Client-side select of cached select list
+      selectOption('category', '<%= j @category&.code %>');
+    </script>
+  <% end %>
+
+  <% if Dokno::Article.exists? %>
+    <div class="relative w-<%= Dokno::Category.exists? ? '1/2 pl-5' : 'full' %>">
+      <i data-feather="search" class="absolute ml-4 mt-3 inline-block text-gray-300" title="Search"></i>
+      <input title="Press / to search" onsearch="applyCategoryCriteria('<%= @category&.code %>', this.value, '<%= @order %>');" onfocus="disableSearchHotkey();" onblur="enableSearchHotkey();" placeholder="Search<%= @category.present? ? ' under this category' : ', hotkey /' %>" type="search" name="search_term" id="search_term" value="<%= @search_term %>" class="pl-12 pr-8 py-2 rounded text-xl shadow-inner bg-gray-100 w-full" />
+      <% if @category.present? %><div title="Press / to search" class="absolute -ml-6 mt-2 inline-block text-gray-300 font-semibold">/</div><% end %>
+    </div>
+  <% end %>
+</div>
+
+<script> enableSearchHotkey(); </script>

data/app/views/partials/_form_errors.html.erb

@@ -1,5 +1,4 @@
 <div id="error_explanation" class="bg-white mb-10">
-  <i data-feather="alert-octagon" class="float-right"></i>
   <h2 class="font-semibold">
     There
     <% if errors.count == 1 %>

data/app/views/partials/_logs.html.erb

@@ -2,7 +2,7 @@
   <div class="py-10 px-16 text-gray-200 bg-gray-900">
     <div class="w-full max-w-screen-xl m-auto">
       <div class="text-xl text-gray-600 cursor-pointer" onclick="toggleVisibility('change-log');">
-        Change log for this article
+        Change history for this article
 
         <div class="inline toggle-visibility-indicator-container change-log">
           <i data-feather="chevron-left" class="inline toggle-visibility-indicator change-log"></i>
@@ -20,7 +20,7 @@
                 <% end %>
 
                 <div class="text-gray-500 bg-gray-700 p-5 pr-10 rounded <%= 'cursor-pointer' if log.diff_left != log.diff_right %> flex items-center" onclick="toggleVisibility('article-diff-<%= log.id %>');" title="Show / Hide Diff">
-                  <div class="w-4/5">
+                  <div class="w-<%= log.diff_left != log.diff_right ? '11/12' : 'full' %>">
                     <%= time_ago_in_words log.created_at %> ago
                     <% if log.username.present? %>
                       by <%= log.username %>
@@ -31,9 +31,11 @@
                     <% end %>
                   </div>
 
-                  <div class="w-1/5 text-right toggle-visibility-indicator-container article-diff-<%= log.id %>">
-                    <% if log.diff_left != log.diff_right %><i data-feather="chevron-left" class="inline toggle-visibility-indicator article-diff-<%= log.id %>"></i><% end %>
-                  </div>
+                  <% if log.diff_left != log.diff_right %>
+                    <div class="w-1/12 text-right toggle-visibility-indicator-container article-diff-<%= log.id %>">
+                      <i data-feather="chevron-left" class="inline toggle-visibility-indicator article-diff-<%= log.id %>"></i>
+                    </div>
+                  <% end %>
                 </div>
 
                 <% if log.diff_left != log.diff_right %>

data/app/views/partials/_pagination.html.erb

@@ -1,28 +1,31 @@
-<span class="mr-5">
-  <% if @page > 1 %>
-    <span class="mr-1 inline-block"><a href="?search_term=<%= CGI.escape @search_term.to_s %>&order=<%= @order %>&page=<%= (@page - 1) %>"><i data-feather="arrow-left" class="h-5 inline-block" title="Previous page"></i></a></span>
-  <% end %>
+<% if @total_pages > 1 %>
+  <span class="mr-5">
+    <% if @page > 1 %>
+      <span class="mr-1 inline-block"><a href="?search_term=<%= CGI.escape @search_term.to_s %>&order=<%= @order %>&page=<%= (@page - 1) %>"><i data-feather="arrow-left" class="h-5 inline-block" title="Previous page"></i></a></span>
+    <% end %>
 
-  <span class="mr-1 inline-block">Page</span>
+    <span class="mr-1 inline-block">Page</span>
 
-  <%= form_with(url: article_index_path(@category&.code), method: :get, class: 'inline') do %>
-    <input type="hidden" name="search_term" value="<%= @search_term %>">
-    <input type="hidden" name="order" value="<%= @order %>">
-    <input aria-label="Page" type="text" name="page" value="<%= @page %>" onclick="this.select();" class="w-10 text-center bg-gray-200 rounded" />
+    <%= form_with(url: article_index_path(@category&.code), method: :get, class: 'inline') do %>
+      <input type="hidden" name="search_term" value="<%= @search_term %>">
+      <input type="hidden" name="order" value="<%= @order %>">
+      <input aria-label="Page" type="text" name="page" value="<%= @page %>" onclick="this.select();" class="w-10 text-center bg-gray-200 rounded" />
 
-    <span class="mx-1 inline-block">of</span>
-    <span class="text-center inline-block"><%= @total_pages %></span>
-  <% end %>
+      <span class="mx-1 inline-block">of</span>
+      <span class="text-center inline-block"><%= @total_pages %></span>
+    <% end %>
 
-  <% if @page < @total_pages %>
-    <span class="ml-1 inline-block"><a href="?search_term=<%= CGI.escape @search_term.to_s %>&order=<%= @order %>&page=<%= (@page + 1) %>"><i data-feather="arrow-right" class="h-5 inline-block" title="Next page"></i></a></span>
-  <% end %>
-</span>
+    <% if @page < @total_pages %>
+      <span class="ml-1 inline-block"><a href="?search_term=<%= CGI.escape @search_term.to_s %>&order=<%= @order %>&page=<%= (@page + 1) %>"><i data-feather="arrow-right" class="h-5 inline-block" title="Next page"></i></a></span>
+    <% end %>
+  </span>
+<% end %>
 
 <span class="text-gray-400">
   <%= @total_records %>
-  <%= 'uncategorized' if @category.blank? && @search_term.blank? %>
+  <%= 'uncategorized' if !current_page?(up_for_review_path) && @category.blank? && @search_term.blank? %>
   <%= 'article'.pluralize(@total_records) %>
+  <%= 'up for review' if current_page?(up_for_review_path) %>
 
   <% if @search_term.present? %>
     containing <span class="font-serif">&ldquo;</span><%= @search_term %><span class="font-serif">&rdquo;</span>

data/config/routes.rb

@@ -3,8 +3,8 @@ Dokno::Engine.routes.draw do
   resources :articles
 
   get '/(:cat_code)',          to: 'categories#index',     as: :article_index
+  get '/up_for_review',        to: 'categories#index',     as: :up_for_review
   get 'article_panel/(:slug)', to: 'articles#panel',       as: :panel
-  post 'article_log',          to: 'articles#article_log', as: :article_log
   post 'article_preview',      to: 'articles#preview',     as: :preview
   post 'article_status',       to: 'articles#status',      as: :status
   root 'categories#index'

data/db/migrate/20201203190330_baseline.rb

@@ -13,12 +13,12 @@ class Baseline < ActiveRecord::Migration[6.0]
       t.string "slug"
       t.string "title"
       t.text "markdown"
-      t.datetime "created_at", precision: 6, null: false
-      t.datetime "updated_at", precision: 6, null: false
       t.text "summary"
       t.boolean "active", default: true
       t.bigint "views", default: 0
       t.datetime "last_viewed_at"
+      t.datetime "created_at", precision: 6, null: false
+      t.datetime "updated_at", precision: 6, null: false
       t.index ["slug"], name: "index_dokno_articles_on_slug", unique: true
     end
 
@@ -32,10 +32,10 @@ class Baseline < ActiveRecord::Migration[6.0]
 
     create_table "dokno_categories", force: :cascade do |t|
       t.string "name"
-      t.datetime "created_at", precision: 6, null: false
-      t.datetime "updated_at", precision: 6, null: false
       t.bigint "category_id"
       t.string "code", null: false
+      t.datetime "created_at", precision: 6, null: false
+      t.datetime "updated_at", precision: 6, null: false
       t.index ["category_id"], name: "index_dokno_categories_on_category_id"
       t.index ["code"], name: "index_dokno_categories_on_code", unique: true
       t.index ["name"], name: "index_dokno_categories_on_name", unique: true

data/db/migrate/20201211192306_add_review_due_at_to_articles.rb

@@ -0,0 +1,6 @@
+class AddReviewDueAtToArticles < ActiveRecord::Migration[6.0]
+  def change
+    add_column :dokno_articles, :review_due_at, :datetime
+    add_index :dokno_articles, :review_due_at
+  end
+end

data/db/migrate/20201213165700_add_starred_to_article.rb

@@ -0,0 +1,5 @@
+class AddStarredToArticle < ActiveRecord::Migration[6.0]
+  def change
+    add_column :dokno_articles, :starred, :boolean, default: false
+  end
+end

data/lib/dokno/config/config.rb

@@ -1,4 +1,8 @@
 module Dokno
+  module Error
+    class Config < StandardError; end
+  end
+
   def self.configure
     yield config
     config.validate
@@ -8,62 +12,71 @@ module Dokno
     @config ||= Config.new
   end
 
-  def self.config=(val)
-    @config = val
-  end
-
   class Config
-    # (String) Host application name for display within the mounted dashboard
-    attr_accessor :app_name
+    # Dokno configuration options
+    #
+    # app_name (String)
+    #   Host app name for display within the mounted dashboard
+    # tag_whitelist (Enumerable)
+    #   Determines which HTML tags are allowed in Article markdown
+    # attr_whitelist (Enumerable)
+    #   Determines which HTML attributes are allowed in Article markdown
+    # app_user_object (String)
+    #   Host app's user object
+    # app_user_auth_method (Symbol)
+    #   Host app's user object method to be used for edit authorization.
+    #   Should return boolean
+    # app_user_name_method (Symbol)
+    #   Host app's user object method that returns the authenticated user's name or other
+    #   identifier that will be included in change log events.
+    #   Should return a string
+    # article_review_period (ActiveSupport::Duration)
+    #   The amount of time before articles should be reviewed for accuracy/relevance
+    # article_review_prompt_days (Integer)
+    #   The number of days prior to an article being up for review that users should be prompted
 
-    # (Enumerable) Determines which HTML tags are allowed in Article markdown
+    attr_accessor :app_name
     attr_accessor :tag_whitelist
-
-    # (Enumerable) Determines which HTML attributes are allowed in Article markdown
     attr_accessor :attr_whitelist
-
-    # (String) Host application's user object
     attr_accessor :app_user_object
-
-    # (Symbol) Host application's user object method that should be used to authorize users to edit Dokno data
-    #   Should return boolean.
     attr_accessor :app_user_auth_method
-
-    # (Symbol) Host application's user object method that should return the authenticated user's name or other
-    #   identifier that will be included in change log events. Should return a string.
     attr_accessor :app_user_name_method
+    attr_accessor :article_review_period
+    attr_accessor :article_review_prompt_days
 
     # Defaults
-    TAG_WHITELIST        = %w[code img h1 h2 h3 h4 h5 h6 a em u i b strong ol ul li table thead tbody tfoot tr th td blockquote hr br p]
-    ATTR_WHITELIST       = %w[src alt title href target]
-    APP_USER_OBJECT      = 'current_user'
-    APP_USER_AUTH_METHOD = 'admin?'
-    APP_USER_NAME_METHOD = 'name'
+    TAG_WHITELIST              = %w[code img h1 h2 h3 h4 h5 h6 a em u i b strong ol ul li table thead tbody tfoot tr th td blockquote hr br p]
+    ATTR_WHITELIST             = %w[src alt title href target]
+    APP_USER_OBJECT            = 'current_user'
+    APP_USER_AUTH_METHOD       = :admin?
+    APP_USER_NAME_METHOD       = :name
+    ARTICLE_REVIEW_PERIOD      = 1.year
+    ARTICLE_REVIEW_PROMPT_DAYS = 30
 
     def initialize
-      self.app_name             = Rails.application.class.module_parent.name.underscore.humanize.upcase
-      self.tag_whitelist        = TAG_WHITELIST
-      self.attr_whitelist       = ATTR_WHITELIST
-      self.app_user_object      = APP_USER_OBJECT
-      self.app_user_auth_method = APP_USER_AUTH_METHOD
-      self.app_user_name_method = APP_USER_NAME_METHOD
+      self.app_name                   = Rails.application.class.module_parent.name.underscore.humanize.upcase
+      self.tag_whitelist              = TAG_WHITELIST
+      self.attr_whitelist             = ATTR_WHITELIST
+      self.app_user_object            = APP_USER_OBJECT
+      self.app_user_auth_method       = APP_USER_AUTH_METHOD
+      self.app_user_name_method       = APP_USER_NAME_METHOD
+      self.article_review_period      = ARTICLE_REVIEW_PERIOD
+      self.article_review_prompt_days = ARTICLE_REVIEW_PROMPT_DAYS
     end
 
     def validate
-      validate_tag_whitelist
-      validate_attr_whitelist
+      validate_config_option(option: 'tag_whitelist',              expected_class: Enumerable,              example: '%w[a p strong]')
+      validate_config_option(option: 'attr_whitelist',             expected_class: Enumerable,              example: '%w[class href]')
+      validate_config_option(option: 'app_user_object',            expected_class: String,                  example: 'current_user')
+      validate_config_option(option: 'app_user_auth_method',       expected_class: Symbol,                  example: ':admin?')
+      validate_config_option(option: 'app_user_name_method',       expected_class: Symbol,                  example: ':name')
+      validate_config_option(option: 'article_review_period',      expected_class: ActiveSupport::Duration, example: '1.year')
+      validate_config_option(option: 'article_review_prompt_days', expected_class: Integer,                 example: '30')
     end
 
-    def validate_tag_whitelist
-      return unless !tag_whitelist.is_a?(Enumerable)
-
-      raise "#{config_error_prefix} tag_whitelist must be Enumerable"
-    end
-
-    def validate_attr_whitelist
-      return unless !attr_whitelist.is_a?(Enumerable)
-
-      raise "#{config_error_prefix} attr_whitelist must be Enumerable"
+    def validate_config_option(option:, expected_class:, example:)
+      return unless !send(option.to_sym).is_a? expected_class
+      raise Error::Config, "#{config_error_prefix} #{option} must be #{expected_class}, e.g. #{example}"
     end
 
     def config_error_prefix