Updates from 3rd review

Altinn · Oct 17, 2024 · 1a05b9e · 1a05b9e
1 parent 7ae920d
commit 1a05b9e
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 13 deletions.
diff --git a/backend/src/Designer/Services/Implementation/OptionsService.cs b/backend/src/Designer/Services/Implementation/OptionsService.cs
@@ -76,7 +76,7 @@ public async Task<List<Option>> UploadNewOption(string org, string repo, string
         IEnumerable<Option> result = deserializedOptions.Where(option => string.IsNullOrEmpty(option.Value) || string.IsNullOrEmpty(option.Label));
         if (result.Any())
         {
-            throw new JsonException("Uploaded file is missing one of the following attributes for a option: value or label.");
+            throw new JsonException("Uploaded file is missing one of the following attributes for an option: value or label.");
         }
 
         var altinnAppGitRepository = _altinnGitRepositoryFactory.GetAltinnAppGitRepository(org, repo, developer);

diff --git a/frontend/language/src/nb.json b/frontend/language/src/nb.json
@@ -1593,7 +1593,7 @@
   "ux_editor.modal_text": "Tekst",
   "ux_editor.modal_text_resource_body": "Tekstinnhold",
   "ux_editor.modal_text_resource_body_add": "Legg til tekst",
-  "ux_editor.model_properties_code_list_filename_error": "Filnavnet er ugyldig. Du kan bruke tall, understrek, punktum, bindestrek, og store/små bokstaver fra det norske alfabetet. Filnavnet må starte med en engelsk bokstav",
+  "ux_editor.model_properties_code_list_filename_error": "Filnavnet er ugyldig. Du kan bruke tall, understrek, punktum, bindestrek, og store/små bokstaver fra det norske alfabetet. Filnavnet må starte med en engelsk bokstav.",
   "ux_editor.multi_page_warning": "Denne siden inneholder grupper med flere sider. Denne funksjonaliteten er på nåværende tidspunkt ikke støttet i Altinn Studio. Du kan se og redigere komponentene, men ikke sideinformasjonen. Hvis en komponent legges til eller flyttes i en slik gruppe, blir den automatisk plassert på samme side som komponenten over.",
   "ux_editor.no_components_selected": "Velg en side for å se forhåndsvisningen",
   "ux_editor.no_text": "Ingen tekst",

diff --git a/...kages/ux-editor/src/components/config/editModal/EditOptions/EditCodeList/EditCodeList.tsx b/...kages/ux-editor/src/components/config/editModal/EditOptions/EditCodeList/EditCodeList.tsx
@@ -47,19 +47,18 @@ export function EditCodeList<T extends SelectionComponentType>({
   const onSubmit = (file: File) => {
     const fileNameError = findFileNameError(optionListIds, file.name);
     if (fileNameError) {
-      handleInvalidFileName(file, fileNameError);
+      handleInvalidFileName(fileNameError);
     } else {
       handleUpload(file);
     }
   };
 
-  const handleInvalidFileName = (file: File, fileNameError: FileNameError) => {
-    if (fileNameError == 'invalidFileName') {
-      alert(t('ux_editor.model_properties_code_list_filename_error'));
-    }
-
-    if (fileNameError == 'fileExists') {
-      toast.error(t('ux_editor.modal_properties_code_list_upload_duplicate_error'));
+  const handleInvalidFileName = (fileNameError: FileNameError) => {
+    switch (fileNameError) {
+      case 'invalidFileName':
+        return toast.error(t('ux_editor.model_properties_code_list_filename_error'));
+      case 'fileExists':
+        return toast.error(t('ux_editor.modal_properties_code_list_upload_duplicate_error'));
     }
   };
 
@@ -152,9 +151,9 @@ function CodeListSelectorWithData<T extends SelectionComponentType>({
           <option hidden value=''>
             {t('ux_editor.modal_properties_code_list_helper')}
           </option>
-          {optionListIds.map((option) => (
-            <option key={option} value={option}>
-              {option}
+          {optionListIds.map((optionListId) => (
+            <option key={optionListId} value={optionListId}>
+              {optionListId}
             </option>
           ))}
         </StudioNativeSelect>