DorkasV / jira-to-github-issues

0 stars 0 forks source link

Pop-up i forbindelse med tryk på "Godkend-knappen" #181

Open sync-by-unito[bot] opened 2 months ago

sync-by-unito[bot] commented 2 months ago

Det kunne være rart, hvis der ved tryk på ”Godkend-knappen” kom en boks op, hvor man kan sige ja/nej til om man vil godkende, så man ikke får godkendt i stedet for f.eks. at gemme. Et problem der lidt ofte kan opstå, da de to knapper sidder ved siden af hinanden. (Brugerønske fra RH)

Note 31/3/2022 (RH) - Der er spurgt efter ændringsønsket igen

Note 16/1/2023 (RH) - Der er spurgt efter ændringsønsket igen

Note 27-04-2023 (RH) - Der er spurgt til ændringsønsket igen. Problemet er, at vi tegner rigtig meget.

Note 5/6-2023 (RH) - Der er spurgt ind til ændringsønsket igen

┆Attachments: image-20230627-130952.png | image-20231122-173730.png | image-20231122-173959.png | image-20231122-174301.png | image-20231122-174338.png | image-20231219-112437.png | image-20231219-112611.png | image-20231219-113007.png | image-20240126-130235.png | image-20240126-130349.png | image-20240126-130420.png | image-20240126-130637.png | image-20240426-133727.png

sync-by-unito[bot] commented 2 months ago

➤ Kim Jacobsen commented:

Hej Anders Rosengreen

Jeg vil meget gerne tale med jer ang. denne issue.

mvh

Kim

sync-by-unito[bot] commented 2 months ago

➤ Filip Bruman commented:

Kim Jacobsen

Er det en udfordring med at knapperne sidder for tæt på hinanden vil vi anbefale at afstanden bliver gjort større.

Eventuelt at der tilføjes beskrivenede tekst på knappen så man inden man klikker kan læse hvad knappen gør - uden at man skal vente for at at se knappens tooltip.

Man bør så vidt muligt undgå at indføre en ekstra dialog til denne problemstilling.

sync-by-unito[bot] commented 2 months ago

➤ Marianna Engberg Pedersen commented:

Hej

Jeg mindes, at vi tidligere også har talt om knappernes placeringer, og at der ikke var stemning for at ændre dette. Jeg syntes blot, at vi skal gå med pop-up. På samme måde, som når man f.eks. afviser en kladde. Når man afviser en kladde, så får man følgende pop-up

!image-20230627-130952.png|width=349,height=104!

Så når man trykker på knappen godkend, så skal der blot komme en pop-up med:

❓ Godkende kladde?

Er du sikker på, at du vil godkende kladden?

[OK] [Annuller]

Hvorvidt der skal komme en pop-up eller ej, skal være konfigurbart.

sync-by-unito[bot] commented 2 months ago

➤ Filip Bruman commented:

Marianna Engberg Pedersen

Ok, så er næste skridt at JAR-samarbejdet prioriterer opgaven.

sync-by-unito[bot] commented 2 months ago

➤ Anders Rosengreen commented:

Kim Jacobsen

Hej Kim,

Mht. JAR-1039, så skal der tilføjes en pop-op, og det vil være en mindre opgave at udvikle. Dog skal der inkluderes en konfiguration, hvor der skal laves en forespørgsel til databasen, for at læse konfigurationen, som skal tilføjes databasen. Der skal være test og review, da der indføres en konfiguration, og konfigurationen skal med i både releasedokumentationen og løsningsbeskrivelsen, som begge skal gennem review. Da det er en relativ simpel opgave, er der ikke inkluderet tid til udarbejdelse af BDD og tech. spec.

På baggrund af ovenstående, har vi følgende implementeringsestimat til din godkendelse: Seniorkonsulent: 15 timer Projektleder: 1 timer Pris: 14012 DKK 

Mvh

Anders

sync-by-unito[bot] commented 2 months ago

➤ Kim Jacobsen commented:

Hej Marianna Engberg Pedersen

Se venligst beskrivelse og estimat og giv din vurdering om evt. igangsættelse.

mvh

Kim

sync-by-unito[bot] commented 2 months ago

➤ Marianna Engberg Pedersen commented:

Hej Kim

Jeg tænker GO og det er jo så Region H som betaler for denne.

Som COWI skriver, så er det en relativ simpel opgave.

Der skal komme en pop-up, når man trykker på Godkend-knappen (på samme måde, som når man f.eks. afviser en kladde, se nedenstående billede).

!image-20230627-130952.png|width=50%!

Så når man trykker på knappen Godkend, så skal der blot komme en pop-up med følgende tekst:

❓ Godkende kladde?

Er du sikker på, at du vil godkende kladden?

[OK] [Annuller]

Hvorvidt der skal komme en pop-up eller ej, skal være konfigurbart. Jeg har lige talt med Filip, som foreslog, at jeg kunne komme med et forslag til hvad konfigurationen skulle hedde, hvis jeg ville have lidt indflydelse på det. Jeg tænker følgende parameter under Systemkonfiguration:

PopupGodkend_Lokalitet

True = Der kommer et pop-up-vindue, som beskrevet ovenfor False = Der kommer IKKE et pop-up-vindue (dvs. at JAR fungerer som i dag)

OBS! - Når ændringen kommer med i en release, og en region kører releasen på eget JAR-miljø, så skal parameteren være sat til “False” (så er det kun RH, som skal ind og ændre i systemkonfigurationen)

sync-by-unito[bot] commented 2 months ago

➤ Marianna Engberg Pedersen commented:

Anders Rosengreen

Der er GO til denne jira

sync-by-unito[bot] commented 2 months ago

➤ Anders Rosengreen commented:

Kim Jacobsen

Hej Kim,

Denne er godkendt fra Marianna, kan du godkende vores implementeringsestimat?

Mvh

Anders

sync-by-unito[bot] commented 2 months ago

➤ Kim Jacobsen commented:

Anders Rosengreen

Hermed godkendes implementeringsarbejde svarende til:

Seniorkonsulent: 15 timer Projektleder: 1 timer Pris: 14012 DKK 

mvh

Kim

sync-by-unito[bot] commented 2 months ago

➤ Marianna Engberg Pedersen commented:

Følgende er testet på referencemiljøet - version 4.10.013584

Godkender en kladde og får følgende pop-up:

!image-20231122-173730.png|width=511,height=187!

Trykker på Annuller → Lokaliteten godkendes ikke (OK)

Trykker på Ok → Lokaliteten godkendes (OK)

Går herefter over i systemkonfigurationen - og sætter parameteren til false

!image-20231122-173959.png|width=596,height=67!

Sætter en lokalitet i kladde og godkender lokaliteten

Får nu følgende Pop-up (Bemærk, at nedenstående pop-up fik jeg ikke i forbindelse med, at systemparameteren var sat til true) → Filip Bruman Dette skal lige drøftes.

!image-20231122-174301.png|width=686,height=617!

Trykker “Ja”

Får nu følgende pop-up

!image-20231122-174338.png|width=459,height=140!

Fejl → Når systemparameteren PopupGodkend_Lokalitet er sat til false, så skal ovenstående pop-up ikke komme frem.

sync-by-unito[bot] commented 2 months ago

➤ John Ryan Pedersen commented:

valideringspopup skal komme i begge tilfælde når denne funktionalitet er aktiveret i konfig.

sync-by-unito[bot] commented 2 months ago

➤ Marianna Engberg Pedersen commented:

Testet på 4.10.0.13735 på referencemiljøet

Testet - Hvis jeg sætter parameteren Validering_VedLokalitetGodkendt til false, så kommer der ikke et pop-up-vindue vedrørende validering, og det er uanset om parameteren Pop_up_Godkend_Lokalitet er sat til false eller true.

Bemærkning.

Pop-up-vinduerne som er relateret til hhv. parameteren Validering_VedLokalitetGodkendt og parameteren Pop_up_Godkend_Lokalitet, skal fungere uafhængig af hinanden.

sync-by-unito[bot] commented 2 months ago

➤ Marianna Engberg Pedersen commented:

Gentestet og der ses stadig fejl. Valideringsvinduet kommer aldrig frem. Se beskrivelserne under test 3 og test 4

Test 1 - ValideringVedLokalitetGodkendt til false + PopupGodkendLokalitet er sat til false

Der kommer ingen pop-up-vinduer → Resultat OK

Test 2 - ValideringVedLokalitetGodkendt til false + PopupGodkendLokalitet er sat til true

Pop-up-vindue fremkommer

!image-20231219-112437.png|width=429,height=122!

Trykker Annuller → Lokaliteten godkendes ikke, dvs. den forbliver i kladde

Trykker igen på godkend knappen og pop-up-vindue fremkommer

!image-20231219-112611.png|width=421,height=118!

Trykker Ok → Lokaliteten godkendes

Resultat OK

Test 3 - ValideringVedLokalitetGodkendt til true + PopupGodkendLokalitet er sat til false

Lokaliteten blev ikke valideret, dvs. Pop-up-vinduet kom ikke frem → FEJL

Test 4 - ValideringVedLokalitetGodkendt til true + PopupGodkendLokalitet er sat til true

Godkender lokaliteten

!image-20231219-113007.png|width=423,height=114!

Trykker OK

Valideringsvinduet kom ikke frem → FEJL

sync-by-unito[bot] commented 2 months ago

➤ Marianna Engberg Pedersen commented:

Testet på Master JAR version 4.10.0.14709

Test 1 - ValideringVedLokalitetGodkendt til false + PopupGodkendLokalitet er sat til false

Der kommer ingen pop-up-vinduer → Resultat OK

Test 2 - ValideringVedLokalitetGodkendt til false + PopupGodkendLokalitet er sat til true

Pop-up-vindue fremkommer

!image-20231219-112437.png|width=50%!

Trykker Annuller → Lokaliteten godkendes ikke, dvs. den forbliver i kladde

Trykker igen på godkend knappen og pop-up-vindue fremkommer

!image-20231219-112611.png|width=50%!

Trykker Ok → Lokaliteten godkendes

Resultat OK

Test 3 - ValideringVedLokalitetGodkendt til true + PopupGodkendLokalitet er sat til false

Godkender lokaliteten

!image-20240126-130637.png|width=694,height=686!

Trykker Nej → Lokaliteten forbliver i kladde (resultat ok)

Godkender lokaliteten

!image-20240126-130637.png|width=694,height=686!

Trykker Ja → Lokaliteten bliver godkendt (resultat ok)

Test 4 - ValideringVedLokalitetGodkendt til true + PopupGodkendLokalitet er sat til true

Godkender lokaliteten

!image-20240126-130235.png|width=687,height=710!

Trykker Nej → Lokaliteten forbliver i kladde (resultat ok)

Trykker “Godkend lokalitet” igen

!image-20240126-130349.png|width=692,height=715!

Trykker Ja

!image-20240126-130420.png|width=448,height=139!

Trykker Annuller → Lokaliteten forbliver i kladde (resultat ok)

Trykker “Godkend lokalitet” igen

!image-20240126-130349.png|width=692,height=715!

Trykker Ja

!image-20240126-130420.png|width=448,height=139!

Trykker Ok → Lokaliteten bliver godkendt (resultat ok)

Efter denne test lukkes Jira

sync-by-unito[bot] commented 2 months ago

➤ Lone Oster commented:

Filip Bruman

I systemkonfigurationen er “Pop_up_Godkend_Lokalitet” sat til “true”.

En lokalitet har sagsbehandler_1 som hovedsagsbehandler. Der er oprettet en kladde af Sagsbehandler_2 . Sagsbehandler_3 vil godkende lokaliteten og trykker derfor på “Godkend”. Sagsbehandler_3 får følgende popup:

!image-20240426-133727.png|width=385,height=125,alt="image-20240426-133727.png"!

og trykker “Ok” - og der sker ingenting - lokaliteten forbliver i kladde. Det er altså når der er 3 forskellige der er Hovedsagsbehandler, opretter af kladde og godkender, at det går skævt.

Hvis der trykkes “Annuller” så bliver “timeglasset” ved med at snurre rundt og låser JAR.

Hvis jeg sætter “Pop_up_Godkend_Lokalitet” til “false”, så virker det som tidligere - lokaliteten godkendes når man har trykket “Ok” til Popup-spørgsmålet ovenfor.

Testet i MasterJAR version 4.10.0.15099 og i RSj.