Add basic Topic index and show page for articles

Author: rossta

app/content/models/sitepress_article.rb

@@ -5,12 +5,16 @@ class SitepressArticle < Sitepress::Model
   collection glob: "articles/*.html*"
   data :title, :published, :updated, :summary, :description, :tags, :image
 
-  delegate :mime_type, :handler, to: :page
+  delegate :resource_path, :mime_type, :handler, to: :page
 
   def self.published(params = {})
-    all
+    take_published(all)
+  end
+
+  def self.take_published(articles, params = {})
+    articles
       .filter { |article| article.published?(preview: params[:preview]) }
-      .sort { |a, b| b.published_on <=> a.published_on } # DESC order
+      .sort { |a, b| b.published_on <=> a.published_on }
   end
 
   def self.draft

app/controllers/topics_controller.rb

@@ -0,0 +1,10 @@
+class TopicsController < ApplicationController
+  def index
+    @topics = Topic.approved.with_pages.order(name: :asc)
+    @topics = @topics.where("lower(slug) LIKE ?", "#{params[:letter].downcase}%") if params[:letter].present?
+  end
+
+  def show
+    @topic = Topic.find_by!(slug: params[:slug])
+  end
+end

app/javascript/css/utilities/custom.css

@@ -119,3 +119,11 @@
 .step-2 {
   font-size: var(--step-2);
 }
+
+.text-subtle {
+  color: var(--joy-text-subtle);
+}
+
+.text-faint {
+  color: var(--joy-text-faint);
+}

app/models/page.rb

@@ -21,6 +21,14 @@ class Page < ApplicationRecord
   has_many :topics, through: :page_topics
   has_many :approved_topics, -> { approved }, through: :page_topics, source: :topic, inverse_of: :pages
 
+  def self.as_published_articles
+    SitepressArticle.take_published(all.map { |page| SitepressArticle.new(page.resource) })
+  end
+
+  def sitepress_article
+    SitepressArticle.new(resource)
+  end
+
   def resource = Sitepress.site.get(request_path)
 
   def body_text = Nokogiri::HTML(SitepressPage.render_html(resource)).text.squish

app/views/topics/index.html.erb

@@ -0,0 +1,21 @@
+
+<%= render Pages::Header.new(title: "Topics") %>
+<div class="section-content container py-gap mb-3xl">
+  <% if @topics.blank? %>
+  <div class="only:inherit grid grid-row-mid">
+    <h2>No topics have been created yet!</h2>
+    <div>
+      <p>Coming soon ;)</p>
+    </div>
+  </div>
+  <% else %>
+    <% @topics.each do |topic| %>
+      <%= tag.div id: dom_id(topic) do %>
+        <h3>
+          <%= link_to topic.name, topic_path(topic) %>
+          <span class="text-faint">(<%= topic.pages_count %>)</span>
+        </h3>
+      <% end %>
+    <% end %>
+  <% end %>
+</div>

app/views/topics/show.html.erb

@@ -0,0 +1,16 @@
+
+<%= render Pages::Header.new(title: "Topic: #{@topic.name}") %>
+<div class="section-content container py-gap mb-3xl">
+  <% if @topic.pages.blank? %>
+  <div class="only:inherit grid grid-row-mid">
+    <h2>No articles have been added for this topic yet!</h2>
+    <div>
+      <p>Coming soon. Maybe ;)</p>
+    </div>
+  </div>
+  <% else %>
+    <%= render Pages::List.new(@topic.pages.as_published_articles) %>
+  <% end %>
+
+  <div><%= link_to "Back to Topics list", topics_path %></div>
+</div>

config/routes.rb

@@ -183,6 +183,8 @@
 
   match "search", to: "searches#show", via: [:get, :post]
 
+  resources :topics, param: :slug, only: [:index, :show]
+
   namespace :share do
     resources :snippets do
       resource :screenshot, only: [:new, :create, :show], controller: "snippet_screenshots"

spec/requests/topics_spec.rb

@@ -0,0 +1,35 @@
+require "rails_helper"
+
+RSpec.describe "Topics", type: :request do
+  describe "GET /index" do
+    it "returns http success" do
+      article = Page.find_or_create_by!(request_path: "/articles/introducing-joy-of-rails")
+
+      topic_1 = FactoryBot.create(:topic, :approved)
+      topic_2 = FactoryBot.create(:topic, :approved)
+
+      article.topics = [topic_1, topic_2]
+
+      get topics_path
+
+      expect(response).to have_http_status(:success)
+
+      expect(page).to have_content(topic_1.name)
+      expect(page).to have_content(topic_2.name)
+    end
+  end
+
+  describe "GET /show" do
+    it "returns http success" do
+      article = Page.find_or_create_by!(request_path: "/articles/introducing-joy-of-rails")
+      topic = FactoryBot.create(:topic, :approved)
+      article.topics << topic
+
+      get topic_path(topic)
+
+      expect(response).to have_http_status(:success)
+      expect(response.body).to include(topic.name)
+      expect(response.body).to include("Introducing Joy of Rails")
+    end
+  end
+end