From ccb36dc9b424fef8b9b5296eae0f773291f12e8d Mon Sep 17 00:00:00 2001
From: Juzer Shakir <juzershakir.webdev@gmail.com>
Date: Tue, 13 Aug 2024 19:28:02 +0530
Subject: [PATCH] Show only Open Bounties by default (#68)

* show `'open'` bounties by default at root path

* set filter to '`none`' to fetch all bounties

* show custom msg if no open bounties are found
---
 app/controllers/bounties_controller.rb |  1 +
 app/views/bounties/index.html.erb      | 20 +++++++++++++++-----
 2 files changed, 16 insertions(+), 5 deletions(-)

diff --git a/app/controllers/bounties_controller.rb b/app/controllers/bounties_controller.rb
index 13abfef..39a4327 100644
--- a/app/controllers/bounties_controller.rb
+++ b/app/controllers/bounties_controller.rb
@@ -5,6 +5,7 @@ class BountiesController < ApplicationController
 
   # GET /bounties or /bounties.json
   def index
+    params[:filter] = "open" if params[:filter].nil?
     @bounties = Bounty.sorted
     @filter, @bounties = params[:filter], @bounties.filtered(params[:filter]) if Bounty::STATUSES.include?(params[:filter])
     @pagy, @bounties = pagy(@bounties)
diff --git a/app/views/bounties/index.html.erb b/app/views/bounties/index.html.erb
index 0f90da4..abbce0a 100644
--- a/app/views/bounties/index.html.erb
+++ b/app/views/bounties/index.html.erb
@@ -5,15 +5,25 @@
 <div class="my-8 flex gap-2 items-center justify-between">
   <h2 class="m-0"><%= (@filter || "All").titleize %> Bounties</h2>
   <nav class="flex gap-2">
-    <%= link_to "All", bounties_path, class: class_names("font-bold" => @filter.blank?) %>
+    <%= link_to "All", bounties_path(filter: "none"), class: class_names("font-bold" => @filter.blank?) %>
     <% Bounty::STATUSES.each do |status| %>
       <%= link_to status.titleize, bounties_path(filter: status), class: class_names("font-bold" => status == @filter) %>
     <% end %>
   </nav>
 </div>
 
-<div id="bounties">
-  <%= render @bounties %>
-</div>
+<% if @bounties.present? %>
+  <div id="bounties">
+    <%= render @bounties %>
+  </div>
 
-<%== pagy_nav(@pagy).html_safe %>
+  <%== pagy_nav(@pagy).html_safe %>
+<% else %>
+  <% if @filter == "open" %>
+    <p class="m-8 md:m-12 text-center italic text-lg md:text-xl">
+      There are no open bounties right now.
+      <br>
+      Spread the word that we need people to post new Beginner Bounties!
+    </p>
+  <% end %>
+<% end %>