From 4b2b313249b3d3190bcb5351408d403138ef4c5f Mon Sep 17 00:00:00 2001
From: Troels Ugilt Jensen <6103205+tuj@users.noreply.github.com>
Date: Mon, 25 Mar 2024 11:10:27 +0100
Subject: [PATCH 1/3] 969: Added onChange to checkbox

---
 src/components/slide/content/media-selector-list.jsx | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/components/slide/content/media-selector-list.jsx b/src/components/slide/content/media-selector-list.jsx
index 7ffb0bb2..2455758f 100644
--- a/src/components/slide/content/media-selector-list.jsx
+++ b/src/components/slide/content/media-selector-list.jsx
@@ -106,6 +106,7 @@ function MediaSelectorList({ multiple, selectedMediaIds, onItemClick }) {
                         aria-label={t("checkbox-form-aria-label", {
                           this: data.title,
                         })}
+                        onChange={() => {}}
                       />
                     )}
 

From 2024a71597ba6a9430b51e1c86ffeb44b86c01b0 Mon Sep 17 00:00:00 2001
From: Troels Ugilt Jensen <6103205+tuj@users.noreply.github.com>
Date: Mon, 25 Mar 2024 11:12:39 +0100
Subject: [PATCH 2/3] 969: Added activation code helptext. Fixed issues raised
 when compiling

---
 .../activation-code/activation-code-form.jsx  | 46 ++++++++++++-------
 .../activation-code/activation-code-list.jsx  |  6 ++-
 .../slide/content/media-selector-list.jsx     |  5 +-
 src/components/user/oidc-login.jsx            | 10 ++--
 src/translations/da/common.json               |  4 +-
 5 files changed, 44 insertions(+), 27 deletions(-)

diff --git a/src/components/activation-code/activation-code-form.jsx b/src/components/activation-code/activation-code-form.jsx
index f21ef28a..68067443 100644
--- a/src/components/activation-code/activation-code-form.jsx
+++ b/src/components/activation-code/activation-code-form.jsx
@@ -44,29 +44,41 @@ function ActivationCodeForm({
     },
   ];
 
+  // 123
+
   return (
     <>
       <LoadingComponent isLoading={isLoading} loadingMessage={loadingMessage} />
       <Form>
         <h1 id="h1UserDisplayName">{headerText}</h1>
         <ContentBody>
-          <FormInput
-            title="display-name"
-            type="text"
-            label={t("display-name-label")}
-            placeholder={t("display-name-placeholder")}
-            value={activationCode.displayName}
-            onChange={handleInput}
-            name="displayName"
-            required
-          />
-          <RadioButtons
-            radioGroupName="role"
-            handleChange={handleInput}
-            options={roles}
-            label={t("role-label")}
-            selected={activationCode.role}
-          />
+          <div className="mb-2">
+            <FormInput
+              title="display-name"
+              type="text"
+              label={t("display-name-label")}
+              placeholder={t("display-name-placeholder")}
+              value={activationCode.displayName}
+              onChange={handleInput}
+              name="displayName"
+              required
+            />
+          </div>
+          <div className="mb-2">
+            <RadioButtons
+              radioGroupName="role"
+              handleChange={handleInput}
+              options={roles}
+              label={t("role-label")}
+              selected={activationCode.role}
+            />
+            <div>
+              <small>{t("role-external-user-helptext")}</small>
+            </div>
+            <div>
+              <small>{t("role-external-user-admin-helptext")}</small>
+            </div>
+          </div>
         </ContentBody>
         <ContentFooter>
           <Button
diff --git a/src/components/activation-code/activation-code-list.jsx b/src/components/activation-code/activation-code-list.jsx
index ef024099..2ac3503a 100644
--- a/src/components/activation-code/activation-code-list.jsx
+++ b/src/components/activation-code/activation-code-list.jsx
@@ -128,7 +128,7 @@ function ActivationCodeList() {
   const dispatch = useDispatch();
 
   const refreshCallback = (id) => {
-    const item = items.filter((e) => e['@id'] === id);
+    const item = items.filter((e) => e["@id"] === id);
 
     if (item.length !== 1) {
       return;
@@ -136,7 +136,9 @@ function ActivationCodeList() {
 
     dispatch(
       api.endpoints.postV1UserActivationCodesRefresh.initiate({
-        userActivationCodeActivationCode: JSON.stringify({ activationCode: item[0].code }),
+        userActivationCodeActivationCode: JSON.stringify({
+          activationCode: item[0].code,
+        }),
       })
     )
       .then((response) => {
diff --git a/src/components/slide/content/media-selector-list.jsx b/src/components/slide/content/media-selector-list.jsx
index 2455758f..5448421f 100644
--- a/src/components/slide/content/media-selector-list.jsx
+++ b/src/components/slide/content/media-selector-list.jsx
@@ -91,10 +91,11 @@ function MediaSelectorList({ multiple, selectedMediaIds, onItemClick }) {
               const selected = selectedMediaIds.includes(data["@id"]);
               return (
                 <div key={data["@id"]} className="col mb-3">
-                  <div
+                  <button
                     className={`card bg-light h-100 media-item ${
                       selected ? " selected" : ""
                     }`}
+                    type="button"
                     onClick={() => onItemClick(data)}
                   >
                     <FilePreview fileEntry={data} />
@@ -122,7 +123,7 @@ function MediaSelectorList({ multiple, selectedMediaIds, onItemClick }) {
                         </div>
                       </div>
                     </div>
-                  </div>
+                  </button>
                 </div>
               );
             })}
diff --git a/src/components/user/oidc-login.jsx b/src/components/user/oidc-login.jsx
index 873068cd..458fdaa9 100644
--- a/src/components/user/oidc-login.jsx
+++ b/src/components/user/oidc-login.jsx
@@ -1,5 +1,5 @@
-import { React, useEffect, useState } from "react";
-import { Alert, Spinner } from "react-bootstrap";
+import { React, useState } from "react";
+import { Alert } from "react-bootstrap";
 import { useTranslation } from "react-i18next";
 import "./login.scss";
 import PropTypes from "prop-types";
@@ -22,7 +22,6 @@ function OIDCLogin({ config }) {
   const { provider, label, icon } = config;
 
   // State
-  const [oidcAuthUrl, setOidcAuthUrl] = useState("");
   const [oidcAuthLoadingError, setOidcAuthLoadingError] = useState("");
 
   let labelText = label;
@@ -79,16 +78,17 @@ function OIDCLogin({ config }) {
 
   return (
     <>
-      <div
+      <button
         onClick={onClick}
         className="margin-right-button btn btn-primary btn-lg d-flex justify-content-center align-items-center"
         style={{ minWidth: "160px" }}
+        type="button"
         aria-label={t("login-with-oidc-aria-label")}
         aria-describedby="ad-explanation"
       >
         {iconRender}
         {labelText}
-      </div>
+      </button>
       {oidcAuthLoadingError !== "" && (
         <Alert variant="danger mt-2">{oidcAuthLoadingError}</Alert>
       )}
diff --git a/src/translations/da/common.json b/src/translations/da/common.json
index fdf61f50..c59ff732 100644
--- a/src/translations/da/common.json
+++ b/src/translations/da/common.json
@@ -962,7 +962,9 @@
     "save-button": "Gem aktiveringskode",
     "role-external-user-admin": "Ekstern brugeradministrator",
     "role-external-user": "Ekstern bruger",
-    "role-label": "Brugerens rolle"
+    "role-label": "Brugerens rolle",
+    "role-external-user-helptext": "Hvis brugeren kun skal oprette indhold, vælg \"Ekstern bruger\".",
+    "role-external-user-admin-helptext": "Hvis brugeren skal kunne tilføje andre eksterne brugere til området, så vælg \"Ekstern brugeradministrator\"."
   },
   "activation-code-create": {
     "header": "Tilkobling til nyt område",

From d21d46f7e1b53093153a6691326af55ab33a2c34 Mon Sep 17 00:00:00 2001
From: Troels Ugilt Jensen <6103205+tuj@users.noreply.github.com>
Date: Mon, 25 Mar 2024 11:15:06 +0100
Subject: [PATCH 3/3] 969: Updated changelog

---
 CHANGELOG.md | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index bb2f3837..8d938deb 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -4,6 +4,9 @@ All notable changes to this project will be documented in this file.
 
 ## [Unreleased]
 
+- [#236](https://github.com/os2display/display-admin-client/pull/236)
+  - Add help text to activation code form.
+  - Fixed warnings raised when compiling.
 - [#235](https://github.com/os2display/display-admin-client/pull/235)
   - Added roles to users list.
 - [#233](https://github.com/os2display/display-admin-client/pull/233)