From 96096cb96fd46661fe3b5be16740decc7fa0d9b2 Mon Sep 17 00:00:00 2001
From: Pierre-Yves Chibon <pingou@pingoured.fr>
Date: Feb 07 2020 15:58:46 +0000
Subject: [PATCH 1/3] Only enable the "Take" button once we know the package is active


Otherwise the package may actually be retired and the "take" button will
not work.

Fixes https://pagure.io/pagure-dist-git/issue/99

Signed-off-by: Pierre-Yves Chibon <pingou@pingoured.fr>

---

diff --git a/pagure/themes/srcfpo/templates/repo_master_sidebar.html b/pagure/themes/srcfpo/templates/repo_master_sidebar.html
index d2670bc..ff231fd 100644
--- a/pagure/themes/srcfpo/templates/repo_master_sidebar.html
+++ b/pagure/themes/srcfpo/templates/repo_master_sidebar.html
@@ -123,7 +123,7 @@
     <h6>Orphaned:</h6>
     <div class="btn-group">
       <a href="#" title="Adopt this orphaned package"
-          class="btn btn-sm btn-outline-primary"
+          class="btn btn-sm btn-outline-primary disabled"
           id="take-orphan-button">
         <i id="take-orphan-icon" class="fa fa-fw fa-hand-paper-o"></i>
         Take
@@ -341,8 +341,8 @@
         type: 'GET',
         dataType: 'json',
         success: function(res) {
+          var _btn = $("#take-orphan-button");
           if (!res.active){
-            var _btn = $("#take-orphan-button");
             _btn.off("click");
             _btn.click(function(){
                 window.open(
@@ -351,6 +351,8 @@
             });
             _btn.prop( "title", "Package retired - Open a releng ticket to adopt it" );
             _btn.html("Retired");
+          } else {
+            _btn.removeClass('disabled');
           }
         }
       });

From 8b5fb42f90135650a7f8039a693dbc29e29ee805 Mon Sep 17 00:00:00 2001
From: Pierre-Yves Chibon <pingou@pingoured.fr>
Date: Feb 07 2020 16:47:46 +0000
Subject: [PATCH 2/3] Make the "Issue" link in the info page point to bugzilla


Fixes https://pagure.io/pagure-dist-git/issue/98

Signed-off-by: Pierre-Yves Chibon <pingou@pingoured.fr>

---

diff --git a/pagure/themes/srcfpo/templates/repo_info.html b/pagure/themes/srcfpo/templates/repo_info.html
index 4c74d7e..19d902d 100644
--- a/pagure/themes/srcfpo/templates/repo_info.html
+++ b/pagure/themes/srcfpo/templates/repo_info.html
@@ -341,13 +341,20 @@
                 </div>
 
                 <div class="btn-group">
-                  <a class="bugz"
-                    href="https://apps.fedoraproject.org/packages/{{ repo.name }}/bugs">
-                    <button type="button"
+                  {% if repo.namespace in ['rpms', 'container', 'modules'] %}
+                  {% if repo.namespace == 'rpms' %}
+                  <a class="bugz" href="https://bugzilla.redhat.com/buglist.cgi?bug_status=NEW&bug_status=ASSIGNED&classification=Fedora&product=Fedora&component={{ repo.name }}">
+                  {% elif repo.namespace == 'container' %}
+                  <a class="bugz" href="https://bugzilla.redhat.com/buglist.cgi?bug_status=NEW&bug_status=ASSIGNED&classification=Fedora&product=Fedora Container Images&component={{ repo.name }}">
+                  {% elif repo.namespace == 'modules' %}
+                  <a class="bugz" href="https://bugzilla.redhat.com/buglist.cgi?bug_status=NEW&bug_status=ASSIGNED&classification=Fedora&product=Fedora Modules&component={{ repo.name }}">
+                  {% endif %}
+                  <button type="button"
                       class="btn btn-sm btn-outline-primary font-weight-bold">
                       Bug Reports
                     </button>
                   </a>
+                  {% endif %}
                 </div>
 
                 <div class="btn-group">

From efe79432a9dfcc669432635e327692ec1817711c Mon Sep 17 00:00:00 2001
From: Pierre-Yves Chibon <pingou@pingoured.fr>
Date: Feb 07 2020 16:47:51 +0000
Subject: [PATCH 3/3] Provide some feedback to the user when changing monitoring worked


Basically, a small checked mark will appear next to the drop-down when
the ajax request succeeded.

Fixes https://pagure.io/pagure-dist-git/issue/79

Signed-off-by: Pierre-Yves Chibon <pingou@pingoured.fr>

---

diff --git a/pagure/themes/srcfpo/templates/repo_master_sidebar.html b/pagure/themes/srcfpo/templates/repo_master_sidebar.html
index ff231fd..9a94910 100644
--- a/pagure/themes/srcfpo/templates/repo_master_sidebar.html
+++ b/pagure/themes/srcfpo/templates/repo_master_sidebar.html
@@ -108,6 +108,7 @@
         </div>
       </a>
     </div>
+    <div id="monitoring_feedback"></div>
     {% else %}
     <button title="Monitoring status" class="btn btn-sm btn-outline-primary disabled"
         id="monitoring-button">
@@ -246,6 +247,7 @@
 
       {% if g.authenticated %}
       $(".monitoring-menu a").click(function(){
+        $("#monitoring_feedback").attr('class', '');
         var selectedValue = $(this).attr('id');
         var _status = "no-monitoring";
         if (selectedValue === "monitoring_option_button") {
@@ -263,6 +265,7 @@
           dataType: 'json',
           success: function(res) {
             set_up_monitoring(res.monitoring)
+            $("#monitoring_feedback").attr("class", "fa fa-fw fa-check")
           },
           error: function() {
             alert('Unable to change the monitoring status!');