Refine the consistency of Product import actions #241 (#268)

Signed-off-by: tdruez <[email protected]>

Author: tdruez

CHANGELOG.rst

@@ -109,6 +109,17 @@ Release notes
   Owner not found in the Dataspace are now automatically created.
   https://github.com/aboutcode-org/dejacode/issues/239
 
+- Updated the label of the following Product actions.
+  The labels were updated everywhere in the UI (page title, documentation,
+  import log, etc...) for consistency:
+  - Import data from Scan -> Import ScanCode scan results
+  - Load Packages from SBOMs -> Import SBOM
+  - Import Packages from manifests -> Import Package manifests
+  - Pull ScanCode.io Project data -> Import ScanCode.io project
+  Improve the rendering and layout of the Import related forms for consistency,
+  simplicity, and readability.
+  https://github.com/aboutcode-org/dejacode/issues/241
+
 ### Version 5.2.1
 
 - Fix the models documentation navigation.

component_catalog/templates/component_catalog/includes/scan_status.html

@@ -3,6 +3,8 @@
     <div class="progress-bar {% if has_errors %}bg-warning{% else %}bg-success{% endif %}" role="progressbar" style="width: 100%" aria-label="Scan progress" aria-valuenow="100" aria-valuemin="0" aria-valuemax="100"></div>
   {% elif status == 'failure' or status == "stopped" or status == "stale" %}
     <div class="progress-bar bg-danger" role="progressbar" style="width: 100%" aria-label="Scan progress" aria-valuenow="100" aria-valuemin="0" aria-valuemax="100"></div>
+  {% elif status == 'warning' %}
+    <div class="progress-bar bg-warning" role="progressbar" style="width: 100%" aria-label="Scan progress" aria-valuenow="100" aria-valuemin="0" aria-valuemax="100"></div>
   {% elif status == 'running' %}
     <div class="progress-bar" role="progressbar" style="width: 40%" aria-label="Scan progress" aria-valuenow="40" aria-valuemin="0" aria-valuemax="100"></div>
   {% elif status == 'not_started' or status == 'queued' %}

dejacode/static/css/dejacode_bootstrap.css

@@ -590,10 +590,12 @@ div.awesomplete {
 [data-bs-theme=dark] .awesomplete > ul {
   background: var(--bs-black);
 }
-
 #div_id_component .awesomplete {
   display: inline-block !important;
 }
+label.requiredField {
+  font-weight: bolder;
+}
 
 /* -- Products comparison -- */
 body.product-comparison tr.unchanged {

dje/tasks.py

@@ -197,7 +197,7 @@ def pull_project_data_from_scancodeio(scancodeproject_uuid):
     )
 
     if scancode_project.type == scancode_project.ProjectType.LOAD_SBOMS:
-        notification_verb = "Load Packages from SBOMs"
+        notification_verb = "Import SBOM"
     else:
         notification_verb = "Import packages from ScanCode.io"
 

dje/templates/includes/form_errors_alert.html

@@ -1,6 +1,5 @@
 {% if form.errors %}
-  <div class="alert alert-danger alert-dismissible" role="alert">
-    <button type="button" class="btn-close" data-bs-dismiss="modal" aria-label="Close"></button>
+  <div class="alert alert-danger" role="alert">
     <p class="m-0">Please correct the error{{ form.errors|pluralize }} below.</p>
   </div>
 {% endif %}

dje/tests/__init__.py

@@ -14,6 +14,7 @@
 from django.conf import settings
 from django.contrib.auth import get_user_model
 from django.contrib.auth.models import Permission
+from django.core.files.uploadedfile import TemporaryUploadedFile
 from django.db import DEFAULT_DB_ALIAS
 from django.db import connections
 from django.test.runner import DiscoverRunner
@@ -124,3 +125,20 @@ def assertMaxQueries(self, num, func=None, *args, using=DEFAULT_DB_ALIAS, **kwar
 
         with context:
             func(*args, **kwargs)
+
+
+def wrap_as_temp_uploaded_file(file_path):
+    """Wrap an existing file as a Django TemporaryUploadedFile"""
+    temp_file = TemporaryUploadedFile(
+        name=file_path.name,
+        content_type="application/json",
+        size=file_path.stat().st_size,
+        charset="utf-8",
+    )
+
+    # Read and copy file content into the temporary file
+    with open(file_path, "rb") as f:
+        temp_file.write(f.read())
+
+    temp_file.seek(0)  # Reset pointer after writing
+    return temp_file

docs/reference-1.rst

@@ -5,7 +5,7 @@ Reference 1 - Declared License Expression and License Clarity Scoring
 =====================================================================
 
 When you scan a Package from DejaCode, you can view the Scan Results in a
-:guilabel:`Action` tab on the Package details user view. DejaCode presents a selection of
+:guilabel:`Actions` tab on the Package details user view. DejaCode presents a selection of
 scan details with an emphasis on license detection. You can also download the
 complete :guilabel:`Scan Results` in .json format.
 

docs/tutorial-1.rst

@@ -29,8 +29,8 @@ Alternatively, you can conveniently download one of the provided examples from
 the following
 `GitHub repository <https://github.com/aboutcode-org/dejacode/tree/main/docs/sboms/>`_.
 
-On the Product details page, from the :guilabel:`Action` dropdown, select
-:guilabel:`Load Packages from SBOMs`:
+On the Product details page, from the :guilabel:`Actions` dropdown, select
+:guilabel:`Import SBOM`:
 
 * Click the :guilabel:`Choose File` button on the **SBOM file or zip archive** field.
 * Select your SBOM (.cdx.json or .spdx.json) and click the :guilabel:`Open` button.

docs/tutorial-2.rst

@@ -21,7 +21,7 @@ Click the green :guilabel:`Add` button.
 
 DejaCode presents the new Package.
 
-- Refresh your browser page to see the :guilabel:`Action` tab.
+- Refresh your browser page to see the :guilabel:`Actions` tab.
 - Explore the scan results.
 - Select values to apply to the new Package definition, such as one or more licenses,
   a Copyright statement, and a Primary langauge.
@@ -109,7 +109,7 @@ Improve Package Data by Scanning
 Select :guilabel:`Packages` from the main menu bar.
 
 Identify and select a Package that needs to be improved.
-Click the :guilabel:`Action` button on the Package details form.
+Click the :guilabel:`Actions` button on the Package details form.
 
 Optionally follow the progress of the Scan by selecting the :guilabel:`Scans`
 option from the :guilabel:`Tools` dropdown on the main menu bar.

docs/tutorial-4-vulnerabilities.rst

@@ -26,8 +26,8 @@ Load Scan Results to your Product
 
    `<https://github.com/aboutcode-org/dejacode/tree/main/docs/sboms/starship_engine_2.0_scan_results.json>`_.
 
-2. On the Product details page, from the :guilabel:`Action` dropdown, select
-   :guilabel:`Import from Scan`:
+2. On the Product details page, from the :guilabel:`Actions` dropdown, select
+   :guilabel:`Import ScanCode scan results`:
 
    * Click the :guilabel:`Choose File` button under the **Upload file** field.
    * Select the **starship_engine_2.0_scan_results.json** file and click the