From 917a6a8649a9a69287842d907447f3d5cfd1cba0 Mon Sep 17 00:00:00 2001
From: Jakub Kadlčík <jkadlcik@redhat.com>
Date: Jan 22 2017 18:17:08 +0000
Subject: [PATCH 1/5] [frontend] suggest dnf to enable module


---

diff --git a/frontend/coprs_frontend/coprs/templates/coprs/detail/module.html b/frontend/coprs_frontend/coprs/templates/coprs/detail/module.html
index fee10b7..0b9068d 100644
--- a/frontend/coprs_frontend/coprs/templates/coprs/detail/module.html
+++ b/frontend/coprs_frontend/coprs/templates/coprs/detail/module.html
@@ -115,9 +115,8 @@
 
         <div class="highlight autumn">
 
-          <pre><span class="c1"># Add module repofile to your system (this will be automated by dnf in the future)</span>
-wget {{ copr_url('coprs_ns.generate_module_repo_file', copr, _external=True) }} -O <span class="se">\</span>
-     /etc/fm.modules.d/_copr_{{ module.full_name |replace("/", "-") }}-{{ module.id }}.cfg
+          <pre><span class="c1"># Add module repofile to your system</span>
+dnf copr enable {{ module.copr.full_name }}
 
 <span class="c1"># Install the module</span>
 dnf module <span class="nb">enable</span> {{ module.name }}</pre>

From feed5e75345d7aa1b5f27f769fbb12cb7e8d1091 Mon Sep 17 00:00:00 2001
From: Jakub Kadlčík <jkadlcik@redhat.com>
Date: Jan 22 2017 18:17:08 +0000
Subject: [PATCH 2/5] [frontend] print info when there are no packages


---

diff --git a/frontend/coprs_frontend/coprs/templates/coprs/create_module.html b/frontend/coprs_frontend/coprs/templates/coprs/create_module.html
index 87902c6..5cfbe21 100644
--- a/frontend/coprs_frontend/coprs/templates/coprs/create_module.html
+++ b/frontend/coprs_frontend/coprs/templates/coprs/create_module.html
@@ -60,6 +60,8 @@
           </div>
         </div>
 
+        {% set no_packages = 'No successfully built packages in this project yet' %}
+
         <div class="col-sm-4">
           <div class="panel panel-default">
             <div class="panel-heading">
@@ -77,6 +79,8 @@
                   {{ render_checkbox("filter-{}".format(loop.index0), package, (package in form.filter.data or not form.is_submitted())) }}
                 </label>
               </div>
+              {% else %}
+                <p class="text-muted"><span class="pficon pficon-warning-triangle-o"></span> {{ no_packages }}</p>
               {% endfor %}
             </div>
           </div>
@@ -100,6 +104,8 @@
                     {{ render_checkbox("api-{}".format(loop.index0), package, (package in form.api.data)) }}
                   </label>
                 </div>
+                {% else %}
+                  <p class="text-muted"><span class="pficon pficon-warning-triangle-o"></span> {{ no_packages }}</p>
                 {% endfor %}
               </div>
             </div>
@@ -129,6 +135,8 @@
                       {{ render_checkbox("profile_pkgs-{}-{}".format(i, loop.index0), package, (package in form.profile_pkgs[i].data)) }}
                     </label>
                   </div>
+                  {% else %}
+                    <p class="text-muted"><span class="pficon pficon-warning-triangle-o"></span> {{ no_packages }}</p>
                   {% endfor %}
                 </div>
               </li>
diff --git a/frontend/coprs_frontend/coprs/templates/coprs/detail/module.html b/frontend/coprs_frontend/coprs/templates/coprs/detail/module.html
index 0b9068d..cdfac3d 100644
--- a/frontend/coprs_frontend/coprs/templates/coprs/detail/module.html
+++ b/frontend/coprs_frontend/coprs/templates/coprs/detail/module.html
@@ -54,6 +54,8 @@
     </div>
   </div>
 
+  {% set no_packages = 'No packages in module\'s %s' %}
+
   <div class="col-sm-4">
     <div class="panel panel-default">
       <div class="panel-heading">
@@ -63,6 +65,8 @@
         <ul>
           {% for package in module.modulemd.filter.rpms %}
             <li>{{ package }}</li>
+          {% else %}
+            <p>{{ no_packages | format('filter')}}</p>
           {% endfor %}
         </ul>
       </div>
@@ -78,6 +82,8 @@
         <ul>
         {% for package in module.modulemd.api.rpms %}
           <li>{{ package }}</li>
+        {% else %}
+          <p>{{ no_packages | format('API')}}</p>
         {% endfor %}
         </ul>
       </div>
@@ -98,6 +104,8 @@
                 <li>{{ package }}</li>
               {% endfor %}
             </ul>
+          {% else %}
+            <p>No profiles in module</p>
           {% endfor %}
         </ul>
       </div>

From ef6b961a2c8fd121230544fbee9e937b762e5174 Mon Sep 17 00:00:00 2001
From: Jakub Kadlčík <jkadlcik@redhat.com>
Date: Jan 22 2017 18:17:08 +0000
Subject: [PATCH 3/5] [frontend] allow to have multiple info lines per field


---

diff --git a/frontend/coprs_frontend/coprs/templates/_helpers.html b/frontend/coprs_frontend/coprs/templates/_helpers.html
index 9165824..7ffda83 100644
--- a/frontend/coprs_frontend/coprs/templates/_helpers.html
+++ b/frontend/coprs_frontend/coprs/templates/_helpers.html
@@ -9,9 +9,11 @@
         {{ field(class="form-control", **kwargs)|safe }}
         <ul class="list-unstyled">
         {% if info %}
+          {% for line in (info if info is not string else [info]) %}
           <li class="help-block">
-            <small class="text-muted pficon pficon-info"></small> {{info|safe}}
+            <small class="text-muted pficon pficon-info"></small> {{line|safe}}
           </li>
+          {% endfor %}
         {% endif %}
         {% if field.errors %}
           {% for error in field.errors %}

From 3f68291c0c8452e30cec0d42d6339f6aa6fce013 Mon Sep 17 00:00:00 2001
From: Jakub Kadlčík <jkadlcik@redhat.com>
Date: Jan 22 2017 18:17:08 +0000
Subject: [PATCH 4/5] [frontend] add info what to do with modulemd


---

diff --git a/frontend/coprs_frontend/coprs/templates/coprs/detail/module.html b/frontend/coprs_frontend/coprs/templates/coprs/detail/module.html
index cdfac3d..3b1e82a 100644
--- a/frontend/coprs_frontend/coprs/templates/coprs/detail/module.html
+++ b/frontend/coprs_frontend/coprs/templates/coprs/detail/module.html
@@ -142,6 +142,7 @@ dnf module <span class="nb">enable</span> {{ module.name }}</pre>
         <h3 class="panel-title">Modulemd</h3>
       </div>
       <div class="panel-body">
+        <p class="text-muted"><span class="pficon pficon-info"></span> You may want to use this modulemd as a template and then submit it via Copr API or <code>copr-cli</code> to be build.</p>
         {{ yaml | safe }}
       </div>
     </div>

From 156180a8d5af67aefae54c23025fe5a65d41785a Mon Sep 17 00:00:00 2001
From: Jakub Kadlčík <jkadlcik@redhat.com>
Date: Jan 22 2017 18:17:08 +0000
Subject: [PATCH 5/5] [frontend] show a quick guide how to install 'dnf module' command


---

diff --git a/frontend/coprs_frontend/coprs/templates/coprs/detail/module.html b/frontend/coprs_frontend/coprs/templates/coprs/detail/module.html
index 3b1e82a..c0d7e4e 100644
--- a/frontend/coprs_frontend/coprs/templates/coprs/detail/module.html
+++ b/frontend/coprs_frontend/coprs/templates/coprs/detail/module.html
@@ -118,11 +118,18 @@
         <h3 class="panel-title">How to use</h3>
       </div>
       <div class="panel-body">
-        <p>First, see the documentation for <code>copr module</code> command on it's <a href="https://pagure.io/fm-dnf-plugin">project page</a>.<br>
-        To install this module, please run following commands as root:</p>
+        <p>First, see the documentation for <code>dnf module</code> command on it's <a href="https://pagure.io/fm-dnf-plugin">project page</a>.
+           Quick installation guide for this command:</p>
 
         <div class="highlight autumn">
+          <pre>dnf copr enable @modularity/fm
+dnf copr enable @modularity/modulemd
+dnf copr enable @modularity/modulemd-resolver
+dnf install python3-fm-dnf-plugin</pre>
+        </div>
 
+        <p>To install this module, please run following commands as root:</p>
+        <div class="highlight autumn">
           <pre><span class="c1"># Add module repofile to your system</span>
 dnf copr enable {{ module.copr.full_name }}