[chores:ui] Minor improvements to metric collection consent UI

(cherry picked from commit 5a32721)

# Conflicts:
#	openwisp_utils/metric_collection/helper.py
#	openwisp_utils/metric_collection/migrations/0001_initial.py
#	openwisp_utils/metric_collection/models.py
#	openwisp_utils/metric_collection/tests/test_consent.py

Author: nemesifier

openwisp_utils/admin_theme/static/admin/css/openwisp-info.css

@@ -8,3 +8,7 @@
     display: block;
     margin-top: 8px;
 }
+
+#metric-consent {
+  margin-top: 50px;
+}

openwisp_utils/admin_theme/templates/admin/openwisp_info.html

@@ -21,16 +21,29 @@ <h2>{% trans "OS Information" %}</h2>
 <p><strong>{% trans "Kernel version" %}:</strong> {{ system_info.kernel_version }}</p>
 <p><strong>{% trans "Hardware platform" %}:</strong> {{ system_info.hardware_platform }}</p>
 
+<div id="metric-consent">
 {% block metric_collect_consent %}
-    {% if metric_collection_installed %}
-        <h2>Metric collection</h2>
-        <form method="POST" id="id_metric_collection_consent_form">
-            {% csrf_token %}
-            {{ metric_consent_form }}
-            <p><input type="submit" style="display:none;"></p>
-        </form>
-    {% endif %}
+  {% if metric_collection_installed %}
+    <h2>Metric collection</h2>
+    <form method="POST" id="id_metric_collection_consent_form">
+      {% csrf_token %}
+      {% for field in metric_consent_form %}
+        <p>
+          {% for error in field.errors %}
+            <span class="error">{{ error }}</span>
+          {% endfor %}
+          {{ field.label_tag }}
+          {{ field }}
+        </p>
+        {% if field.help_text %}
+        <p class="helptext">{{ field.help_text|safe }}</p>
+        {% endif %}
+      {% endfor %}
+      <p><input type="submit" style="display:none;"></p>
+    </form>
+  {% endif %}
 {% endblock metric_collect_consent %}
+</div>
 {% endblock content %}
 
 {% block extrahead %}

openwisp_utils/metric_collection/helper.py

@@ -37,13 +37,17 @@ def show_consent_info(cls, request):
         consent = cls._get_consent()
 
         if not consent.shown_once:
-            messages.warning(
+            messages.info(
                 request,
                 mark_safe(
                     _(
-                        'We gather anonymous usage '
-                        'metrics to enhance OpenWISP. '
-                        'You can opt out from the '
+                        "<strong>Congratulations for installing "
+                        "OpenWISP successfully!</strong><br>"
+                        "Use the navigation menu on the left to explore "
+                        "the interface and begin deploying your network.<br>"
+                        "Keep in mind: we gather anonymous usage "
+                        "metrics to improve OpenWISP. "
+                        "You can opt out from the "
                         '<a href="{url}">System Information page</a>.'
                     ).format(url=reverse('admin:ow-info'))
                 ),

openwisp_utils/metric_collection/migrations/0001_initial.py

@@ -45,13 +45,17 @@ class Migration(migrations.Migration):
                     "user_consented",
                     models.BooleanField(
                         default=True,
+                        verbose_name=(
+                            "Help improve OpenWISP by allowing the collection "
+                            "of anonymous usage metrics."
+                        ),
                         help_text=(
-                            'Allow OpenWISP to collect and share anonymous usage metrics to improve'
-                            ' the software. Before opting-out kindly consider reading'
-                            ' <a href="https://openwisp.io/docs/user/usage-metric-collection.html"'
-                            ' target="_blank">why we collect metrics</a>.'
+                            "These statistics help us prioritize features, "
+                            " fix bugs, and better support real-world usage,"
+                            " all without collecting any personal data. "
+                            '<a href="https://openwisp.io/docs/user/usage-metric-collection.html"'
+                            ' target="_blank">Learn more about why we collect metrics</a>.'
                         ),
-                        verbose_name="Allow collecting anonymous usage metrics",
                     ),
                 ),
             ],

openwisp_utils/metric_collection/models.py

@@ -192,11 +192,15 @@ class Consent(TimeStampedEditableModel):
     # usage metrics with the OpenWISP project.
     user_consented = models.BooleanField(
         default=True,
-        verbose_name=_('Allow collecting anonymous usage metrics'),
+        verbose_name=_(
+            "Help improve OpenWISP by allowing the "
+            "collection of anonymous usage metrics."
+        ),
         help_text=_(
-            'Allow OpenWISP to collect and share anonymous usage metrics to improve'
-            ' the software. Before opting-out kindly consider reading'
-            ' <a href="https://openwisp.io/docs/user/usage-metric-collection.html"'
-            ' target="_blank">why we collect metrics</a>.'
+            "These statistics help us prioritize features, "
+            " fix bugs, and better support real-world usage,"
+            " all without collecting any personal data. "
+            '<a href="https://openwisp.io/docs/user/usage-metric-collection.html"'
+            ' target="_blank">Learn more about why we collect metrics</a>.'
         ),
     )

openwisp_utils/metric_collection/tests/test_consent.py

@@ -21,10 +21,13 @@ def _get_user(self, **kwargs):
 
     def test_info_message(self):
         expected_message = (
-            '<li class="warning">We gather anonymous usage metrics '
-            'to enhance OpenWISP. You can opt out from the '
-            '<a href="/admin/openwisp-system-info/">System '
-            'Information page</a>.</li>'
+            '<li class="info"><strong>Congratulations for installing '
+            "OpenWISP successfully!</strong><br>Use the navigation "
+            "menu on the left to explore the interface and begin "
+            "deploying your network.<br>Keep in mind: we gather "
+            "anonymous usage metrics to improve OpenWISP. You can "
+            'opt out from the <a href="/admin/openwisp-system-info/">'
+            "System Information page</a>.</li>"
         )
         non_superuser = self._get_user(is_staff=True, is_superuser=False)
         superuser1 = self._get_user(