Make it possible to use manual editor to edit predefined codelists

[nkylstad]

When the user selects a predefined codelist, they should be able to use the codelist editor in the component config panel to edit the codelist just like if it were a codelist that is defined directly on the component.

Tasks

Preview Give feedback

1. Create/adjust endpoint for editing predefined codelists
2. Implement the codelist editor in the "select codelist" interface

[TomasEng]

Testet i dev. Det fungerer stort sett greit, men jeg opplever av og til å få en feilmelding i bakgrunnen mens jeg gjør endringer. Det skjer særlig når jeg legger til alternativer. Se video:
https://github.com/user-attachments/assets/c334d7b6-1237-4447-88a1-2d7a1d620c12
@standeren nevnte i går at det muligens er noen feil i hvordan backenden utfører validering, siden den ikke godtar tomme strenger. Det er kanskje relatert til det? Har vi en sak på det?

Ellers ser alt veldig fint ut, men det oppleves litt upraktisk at tabellen flytter på seg når jeg sletter rader. Andre seteder har vi løst den typen problemer ved å gi modalen en fast høyde. Kanskje vi bør gjøre det her også?

[Konrad-Simso]

Tusen takk for testing @TomasEng. Backend vil svare med 400 Bad Request fordi C# ikke godtar at et felt som er required slik som Value, har verdien null:

public class Option
{
    [Required]
    [JsonPropertyName("value")]
    [JsonConverter(typeof(OptionConverter))]
    public object Value { get; set; }

Verdien kan være en tom string '', men ikke null. Det er noen måter å komme seg rundt dette, men Andrea nevnte at dette kan bli tatt hånd om i et annet issue siden vi fortsatt er bak featureFlag. En løsning er å endre på valideringen til StudioCodeListEditor.

Jeg skal sette størrelsen på modalen 😄

[standeren]

Jeg jobber med å løse dette på backend! 🫡 Da skal det være mulig å sende inn tom streng. Mulig vi må ha noe validering på StudioCodeListEditor i tillegg slik at den ikke kaller onChange før value eller label er satt til gyldige verdier. Vet ikke hva den identifiserer som gyldige verdier per nå @TomasEng ?

[Konrad-Simso]

Backend godtar tomme strenger, men ikke null? er det noe jeg ikke har fått med meg?
Edit: Nvm, ser nå at det feiler med tomme strings også

[TomasEng]

Mulig vi må ha noe validering på StudioCodeListEditor i tillegg slik at den ikke kaller onChange før value eller label er satt til gyldige verdier. Vet ikke hva den identifiserer som gyldige verdier per nå @TomasEng ?

value og label skal automatisk bli satt til tomme strenger. Så hvis tom streng er gyldig, skal det ikke være mulig at disse blir ugyldige mens de blir behandlet i komponenten. Den eneste situasjonen hvor en av disse kan bli undefined er hvis de kommer inn som undefined/null fra backend og ikke blir endret av brukeren. Er dette noe som kan skje, eller blir det validert av backend før det sendes til frontend? Hva skjer hvis man for eksempel lager en ugyldig kodeliste manuelt i Gitea og åpner den i Studio etterpå? Slik det er nå, går i hvert fall frontend ut fra at dataen som kommer fra backend er gylig.

[TomasEng]

Testet igjen nå og ser at modalen har fått fast høyde. Da har jeg ikke mer å utsette på denne saken. Bra jobbet!