Merge branch 'release/5.11.0' into 'master'

Release 5.11.0

See merge request os2borgerpc/os2borgerpc-admin-site!393

Author: Marcus Funch

.gitlab-ci.yml

@@ -43,8 +43,6 @@ variables:
   stage: lint
   needs: []
   image: python:3.8
-  tags:
-      - docker
 
 Lint Python:
     <<: *lint-default
@@ -76,8 +74,6 @@ Build app image:
   image:
     name: gcr.io/kaniko-project/executor:debug-v0.16.0
     entrypoint: [""]
-  tags:
-    - docker
   script:
     # Login to GitLab registry
     - echo "{\"auths\":{\"$CI_REGISTRY\":{\"username\":\"$CI_REGISTRY_USER\",\"password\":\"$CI_REGISTRY_PASSWORD\"}}}" > /kaniko/.docker/config.json
@@ -98,8 +94,6 @@ Build app image:
   variables:
     GIT_STRATEGY: none # We do not need the source code
   dependencies: []
-  tags:
-    - docker
   before_script:
     # install skopeo
     - apk add skopeo

NEWS.rst

@@ -1,3 +1,8 @@
+Version 5.11.0, February 23, 2023
+---------------------------------
+
+- Make computers/json API work again.
+
 Version 5.10.3, February 2, 2023
 --------------------------------
 

VERSION

@@ -1 +1 @@
-5.10.3
+5.11.0

admin_site/static/docs/Guide_til_brug_af_strømbesparingsfunktioner.pdf

@@ -2989,8 +2989,14 @@ class JSONSiteSummary(JSONResponseMixin, SiteView):
         "location",
     ]
 
+    def render_to_response(self, context, **response_kwargs):
+        # It is necessary to overwrite the standard render_to_response method
+        # when we want to return something that should not be rendered
+        # via a Django template
+        return self.render_to_json_response(context, **response_kwargs)
+
     def get_context_data(self, **kwargs):
-        pcs = []
+        pcs = {}
         for p in self.object.pcs.all():
             pc = {}
             for pn in JSONSiteSummary.interesting_properties:
@@ -3010,7 +3016,7 @@ def get_context_data(self, **kwargs):
                 else:
                     pv = str(pv)
                 pc[pn] = pv
-            pcs.append(pc)
+            pcs[pc["id"]] = pc
         return pcs
 
 

admin_site/system/views.py

@@ -44,6 +44,7 @@
   </ie11-browser-warning>
 
   <script src="/static/js/bootstrap-table.min.js"></script>
+  <script src="/static/js/bootstrap-table-da-DK.min.js"></script>
   <script src="/static/js/bootstrap.bundle.min.js"></script>
   {% block javascripts %}{% endblock %}
 </body>

admin_site/templates/base.html

@@ -95,11 +95,11 @@ <h2>Vigtige bemærkninger</h2>
     den først begynde at følge den nye/opdaterede tidsplan, når den har været tændt.
   </p>
   <p>
-    Hvis en computer modtager en tidsplan, der siger at den skal være slukket på nuværende tidspunkt,
-    vil der gå 20 minutter fra den modtager tidsplanen til den faktisk slukker. Ligeledes,
-    hvis den manuelt tændes på et tidspunkt, hvor den skal være slukket ifølge tidsplanen, vil
-    der gå 20 minutter før den slukker. Dette er valgt, så man har tid til at ændre tidsplanen,
-    inden computeren slukker.
+    Hvis en computer manuelt tændes efter dens sluk-tidspunkt, eller på en dag hvor den er
+    sat til at være konstant slukket, vil den først slukke automatisk på
+    sluk-tidspunktet for næste dag. Hvis der ikke findes et sluk-tidspunkt for næste dag,
+    vil den automatisk slukke 24 timer efter den blev tændt. Hvis computeren manuelt
+    slukkes igen, vil den vågne efter tidsplanen.
   </p>
   <h3>Indstilling af BIOS/UEFI</h3>
   <ol>

admin_site/templates/documentation/wake_plans/index.html

@@ -7,7 +7,7 @@
 {% block specific_content %}
   <div class="container-fluid main sublevelmain" style="overflow-x: auto;">
     <h2 class="divideheader">
-      Computere 
+      Computere
       <span class="ms-3 badge bg-secondary">Aktiverede: {{ activated_pcs }} / {{ total_pcs }}</span>
       {# Originally this was online / activated, but it turns out a computer that isn't activated can still be online #}
       <span class="ms-3 badge bg-secondary">Online: {{ online_pcs }} / {{ total_pcs }}</span>
@@ -25,8 +25,8 @@ <h2 class="divideheader">
         </tr>
       </thead>
       <tbody>
-          {% for pc in ls_pcs %}
-        <tr class="click-list--item">
+        {% for pc in ls_pcs %}
+          <tr class="click-list--item">
             <td>
               <span style="display: none;">{{pc.name}}</span> <!-- Used to ensure sorting by pcname rather than uid -->
               <strong><a href="{% url 'computer' site_uid=site.uid pc_uid=pc.uid %}">{{ pc.name }}</a></strong>
@@ -40,22 +40,22 @@ <h2 class="divideheader">
             <td>{{ pc.location }}</td>
             <td>
               {% if pc.online %}
-              <span class="badge bg-success">Online</span>
+                <span class="badge bg-success">Online</span>
               {% elif pc.is_activated %}
-              <span class="badge bg-danger">Offline</span>
+                <span class="badge bg-danger">Offline</span>
               {% endif %}
             </td>
             <td>{% if pc.last_seen %}{{ pc.last_seen|timesince }} siden ({{ pc.last_seen }}) {% else %}Aldrig{% endif %}</td>
             <td>
-            {% if pc.security_events.latest_event %}
-	            {{ pc.security_events.latest_event.reported_time }}
-            {% else %}
-              Aldrig
-            {% endif %}
+              {% if pc.security_events.latest_event %}
+                {{ pc.security_events.latest_event.reported_time }}
+              {% else %}
+                Aldrig
+              {% endif %}
             </td>
-        </tr>
+          </tr>
         {% endfor %}
       </tbody>
-    </table> 
+    </table>
   </div>
 {% endblock %}

admin_site/templates/system/site_status.html

@@ -73,6 +73,7 @@ COPY --from=frontend \
   /code/nodejs/node_modules/jquery/dist/jquery.min.js \
   /code/nodejs/node_modules/qrcodejs/qrcode.min.js \
   /code/nodejs/node_modules/bootstrap-table/dist/bootstrap-table.min.js \
+  /code/nodejs/node_modules/bootstrap-table/dist/locale/bootstrap-table-da-DK.min.js \
   /frontend/js/
 
 # Unfortunately, `collectstatic` requires all settings to be set. We include a

docker/Dockerfile

@@ -9,7 +9,7 @@
     "build-sass": "node_modules/.bin/sass src/custom.scss:dist/css/bootstrap.css --style=compressed --source-map",
     "watch-sass": "node_modules/.bin/sass src/custom.scss:/code/admin_site/static/css/bootstrap.css --style=compressed --source-map --watch",
     "copy-bootstrap-css": "cp dist/css/bootstrap.* node_modules/bootstrap-table/dist/bootstrap-table.min.css /code/admin_site/static/css",
-    "copy-bootstrap-js": "cp node_modules/bootstrap/dist/js/bootstrap.bundle.min.js* node_modules/bootstrap-table/dist/bootstrap-table.min.js /code/admin_site/static/js",
+    "copy-bootstrap-js": "cp node_modules/bootstrap/dist/js/bootstrap.bundle.min.js* node_modules/bootstrap-table/dist/bootstrap-table.min.js node_modules/bootstrap-table/dist/locale/bootstrap-table-da-DK.min.js /code/admin_site/static/js",
     "test": "echo \"Error: no test specified\" && exit 1"
   },
   "author": "Magenta ApS",