DorkasV / jira-to-github-issues

0 stars 0 forks source link

Mangler X i Overfladevand på fanebladet STOFFER #48

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

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

!image-20230509-085505 (bfb56ab2-b898-4840-9ccb-5fe7f26bad8f).png|width=1084,height=216!

!image-20230509-085536 (6320023b-99ac-4769-b07c-9d7e86647d0b).png|width=1425,height=94!

Hvis man tilknytter et stof til vandløb på en fase så bliver den ikke vist med et X i Overfladevand på fanebladet STOFFER.

BDD Test

Titel: Ny valgliste for mediekolonner i tabellen "Alle stoffer på lokalitetens faser" på fanebladet "3. Stoffer" Som en administrator Ønsker jeg at have en ny valgliste "Medie_på_FB_stoffer"

Scenarie 1: Valgliste "Medie_på_FB_stoffer". Givet at valgliste "Medie_på_FB_stoffer" er valgt i dropdown på Administration / Systemdata / Valglister Når tabellen for valglisten vises har tabellen følgende 6 rækker: "Jord", "Poreluft", "Grundvand", "Overfladevand", "Indeklima" og "Udeluft".

Titel: Ny sammenknytning af kolonner for fanen "3. Stoffer" og medier, så kun tilknyttede medier tages i betragtning når der skal sættes X i en kolonne. Som en administrator Ønsker jeg at have en ny sammenknytning "Medie-fasemedie" jeg kan lave sammenknytning mellem kolonnerne "Jord", "Poreluft", "Grundvand", "Overfladev.", "Indeklima", "Udeluft" i tabellen "Alle stoffer på lokalitetens faser" på fanebladet STOFFER og indhold fra valglisten "Medie".

Scenarie 1: Sammenknytning mellem kolonner og medier (valglisterne "Medie_på_FB_stoffer" og "Medie"). Givet at sammenknytning "Medie-fasemedie" er valgt i dropdown på Administration / Systemdata / Sammenknytning vises i tabellen "Relationsbetegnelse" indholdet af valglisten "Medie_på_FB_stoffer" Når en af rækkerne "Jord", "Poreluft", "Grundvand", "Overfladevand", "Indeklima" eller "Udeluft" vælges vises i tabellen "Mulige valg" indholdet af valglisten "Medie" Og jeg kan se/ændre de aktuelle sammenknytninger.

Scenarie 2: Standardopsætning sammenknytninger mellem kolonner og medier (valglisterne "Medie_på_FB_stoffer" og "Medie"). Givet at sammenknytning "Medie-fasemedie" er valgt i dropdown på Administration / Systemdata / Sammenknytning vises i tabellen "Relationsbetegnelse" indholdet af valglisten "Medie_på_FB_stoffer" ("Jord", "Poreluft", "Grundvand", "Overfladevand", "Indeklima" og "Udeluft") Når der i tabellen "Relationsbetegnelse" vælges "Jord" er som minimum mediet "Jord" (0000.001) valgt Når der i tabellen "Relationsbetegnelse" vælges "Grundvand" er som minimum mediet "Grundvand" (0000.002) valgt Når der i tabellen "Relationsbetegnelse" vælges "Poreluft" er som minimum mediet "Poreluft" (0000.003) valgt Når der i tabellen "Relationsbetegnelse" vælges "Overfladevand" er som minimum mediet "Overfladevand" (0000.004) valgt Når der i tabellen "Relationsbetegnelse" vælges "Indeklima" er som minimum mediet "Indeklima" (0000.005) valgt Når der i tabellen "Relationsbetegnelse" vælges "Udeluft" er som minimum mediet "Udeluft" (0000.006) valgt

Medie 4, 7, 8, 9 og 10 svarer til 0000.004, 0000.007, 0000.008, 0000.009 og 0000.010. Titel: Mangler X i Overfladevand på fanebladet STOFFER for medierne 7, 8, 9 eller 10 Som bruger Ønsker jeg for kolonnen "Overfladev." i tabellen "Alle stoffer på lokalitetens faser" på fanebladet "3. Stoffer", at flere medier tages i betragtning, når det afgøres om der skal sættes X. alle de medier som er valgt i sammenknytningen "Medie-fasemedie" for "Overfladevand" tages i betragtning.

Scenarie 1: X i kolonnen Overfladevand på fanebladet STOFFER. Givet en lokalitet indeholdende en fase Og der er oprettet sammenknytning for kolonnen "Overfladevand" til medierne 4, 7, 8, 9 eller 10 Og fasen kun indeholder stoffer på et eller flere af medierne 4, 7, 8, 9 eller 10 Når fanebladet "3. Stoffer" vises er der X i kolonnen "Overfladev." i tabellen "Alle stoffer på lokalitetens faser" for fasen.

Scenarie 2: Ingen X i kolonnen Overfladevand på fanebladet STOFFER. Givet en lokalitet indeholdende en fase Og der er oprettet sammenknytning for kolonnen "Overfladevand" til medierne 4, 7, 8, 9 eller 10 Og fasen kun indeholder stoffer på andre medier end 4, 7, 8, 9 eller 10 Når fanebladet "3. Stoffer" vises er der IKKE X i kolonnen "Overfladev." i tabellen "Alle stoffer på lokalitetens faser" for fasen.

┆Attachments: image-20230509-085505.png | image-20230509-085505 (bfb56ab2-b898-4840-9ccb-5fe7f26bad8f).png | image-20230509-085536.png | image-20230509-085536 (6320023b-99ac-4769-b07c-9d7e86647d0b).png | image-20231116-133641.png | JAR-1754 TechSpec_DK.pdf

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

➤ Filip Bruman commented:

John Ryan Pedersen

Hvilket miljø er det at i ser dette issue?

Ses issue også i Master JAR/prod?

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

➤ John Ryan Pedersen commented:

4.7 mit driftmiljø og 4.7 Master JAR

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

➤ Filip Bruman commented:

John Ryan Pedersen

Hvilken lokalitet er det du har lavet dette på? Bare så vi kan se den.

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

➤ John Ryan Pedersen commented:

På master er det 671-00202

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

➤ Filip Bruman commented:

John Ryan Pedersen

Den eneste måde der kan være et X i Overfladevand-kolonnen er at have et medie med kode-værdi 0000.004. Det samme gør sig gældende i v2.6.5, her er navnet på kolonnen blot Recipient.

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

➤ John Ryan Pedersen commented:

Så skal det laves så medier med koderne:

7, 8, 9, 10, 11 og 12 også giver er X i kolonnen.

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

➤ John Ryan Pedersen commented:

Men er nok et ÆØ

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

➤ Filip Bruman commented:

John Ryan Pedersen

Tænker at JAR-samarbejdet finder sammen og prioriterer opgaven. Jag laver Jira om til Task.

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

➤ Anders Rosengreen commented:

Kim Jacobsen

Hej Kim,

Undersøgelse viste at dette ikke var en bug.

Hermed timeopgørelse og pris til din godkendelse:

Konsulent: 10 timer

Seniorkonsulent: 1 time

Projektleder: 1 time

Pris: 7866 DKK

Mvh

Anders

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

➤ Kim Jacobsen commented:

Anders Rosengreen

Hermed godkendes arbejder svarende til:

Konsulent: 10 timer

Seniorkonsulent: 1 time

Projektleder: 1 time

Pris: 7866 DKK

mvh

Kim

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

➤ Filip Bruman commented:

John Ryan Pedersen

Skal 7, 8, 9, 10, 11 og 12 give er X i Overfladevand-kolonnen eller i en anden/andre kolonne(r)?

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

➤ John Ryan Pedersen commented:

det skal være 4, 7, 8, 9, 10

det smarteste ville være hvis det var en liste-liste så man selv kunne vælge. Også hvis der kommer nye til.

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

➤ Filip Bruman commented:

John Ryan Pedersen

{quote}det smarteste ville være hvis det var en liste-liste{quote}

Du mener en sammenknytning?

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

➤ John Ryan Pedersen commented:

jeps. En liste-liste funktion 🙂

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

➤ Filip Bruman commented:

John Ryan Pedersen

  1. Hvilket navn skal sammenknytningen have?

  2. Relationsbetegnelse: antager vi er valgliste Medie

  3. Mulige valg: antager vi er en ny valgliste med Tekst: Jord, Poreluft, Grundvand, Overfladevand, Indeklima og Udeluft

    1. Hvilket navn skal denne nye valgliste have?
  4. Kan vi antager at Kode og Sortering for elementerne i den nye valgliste kan besluttes af COWI?

  5. Skal der være særlig logik når et medie eller en af mulighederne i den nye valgliste har en udløbsdato ældre end dags dato?

  6. Skal implementering af sammenknytningen have en standard konfiguration eller skal det være op til den enkelte region at beslutte om man ønsker at aktivere sammenknytningen?

!image-20231116-133641.png|width=436,height=245!

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

➤ John Ryan Pedersen commented:

  1. Medie-fasemedie
    1. Ja
    2. Medie_på_FB_stoffer
    3. Det skal jeg nok gøre
    4. Nej. Skal fortsat vises på FB_stoffer tabellen
    5. Det er ikke en funktionalitet som man kan fravælge
sync-by-unito[bot] commented 4 months ago

➤ Kent Nielsen commented:

John Ryan Pedersen

Vil du læse BDD igennem og bekræfte at opgavens forståelse og omfang er korrekt.

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

➤ John Ryan Pedersen commented:

BDD ok

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

➤ Kent Nielsen commented:

John Ryan Pedersen

I forbindelse med vores indledende analyse har vi et spørgsmål til tabellen "Alle stoffer på lokalitetens faser" på fanebladet "3. Stoffer", som vi gerne vil have afklaret inden vi kan fortsætte.

Det viser sig at mediekolonnerne i tabellen "Alle stoffer på lokalitetens faser" på fanebladet "3. Stoffer" egentlig viser koncentration af stoffet på fasen. Værdien hentes fra feltet [max_konc] i tabellen [dbo].[r_fase_stoffer]. Hvis [max_konc] for et stof på fasen er 0, skrives i stedet for koncentrationen blot et X. I vores databaser ser det ud til med kun én undtagelse at være tilfældet for alle stoffer på alle faser.

Hvis der på en fase er flere stoffer med samme medietype (samme kolonne i tabellen), tjekkes kun det første stof, og det er koncentrationen for det stof (eller et et X) som vises i tabellen. Der ser ikke ud til at være nogen sortering når stofferne på fasen hentes fra databasen (stored procedure jar_hent_stoffer_lokalitet_sp).

Altså, hvis der er flere stoffer med samme medietype på en fase, vises kun koncentrationen for det første stof.

Er det i overensstemmelse med jeres opfattelse af hvad der skal vises i mediekolonnerne i tabellen "Alle stoffer på lokalitetens faser" på fanebladet "3. Stoffer"?

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

➤ John Ryan Pedersen commented:

Hverken Mængde, dybde fra eller til på Mediet eller mængde og enhed på stoffer skal have noget at gøre med hvilke stoffer og medier der vises på FB3.

tabellen Stof/mængde på FB 3 skal afspejle hvis der er tastet stof og mængde ind i den tilsvarende tabel på FB6

Og jeg er usikker på hvad i mener med “koncentration”. Der tastes ikke og vises ikke koncentrationer på Faser

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

➤ John Ryan Pedersen commented:

Du er velkommen til at indkalde til et teamsmøde hvis vi skal drøfte det

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

➤ Kent Nielsen commented:

John Ryan Pedersen

Ok, er det så korrekt forstået at der ved visning i tabellen på FB3 altid kun skal anvendes et X (og aldrig koncentration)?

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

➤ John Ryan Pedersen commented:

Det er korrekt. Altid kun et X

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

➤ Anders Rosengreen commented:

John Ryan Pedersen Kim Jacobsen

Hej John,

Teknisk specifikation er vedhæftet til din gennemgang og godkendelse.

Mvh

Anders

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

➤ Anders Rosengreen commented:

Kim Jacobsen

Hej Kim,

Kan du godkende timer brugt til BDD, analyse og udarbejdelse af teknisk specifikation: Seniorkonsulent: 19 timer Projektleder: 2 timer Pris: 18692 DKK

Mvh

Anders

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

➤ Kim Jacobsen commented:

Hej Anders Rosengreen ,

Hermed godkendes arbejder med BDD, analyse og udarbejdelse af teknisk specifikation svarende til: Seniorkonsulent: 19 timer Projektleder: 2 timer Pris: 18692 DKK

Mvh

Kim

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

➤ Filip Bruman commented:

John Ryan Pedersen

Teknisk specifikation er vedhæftet til din gennemgang og godkendelse.

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

➤ John Ryan Pedersen commented:

Kim Jacobsen Beskrivelsesdelen er OK. der er lige en af DB folkene der skal sige god for den

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

➤ Kim Jacobsen commented:

Peter Riis

Har du mulighed for at lave review denne issue?

mvh

Kim

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

➤ Peter Riis commented:

Umiddelbart ser det fint ud. Husk dog at benytte danske navne. F.eks. skal der ikke være en tekst i e_liste som hedder 'Indeclim'. Det må være Indeklima.

Jeg er i tvivl om hvorledes relationen bliver i mellem e_liste og e_liste_liste. Er den identisk med eksisterende relation mellem tabellerne for kode_type = ‘Medie’? Hvis Cowi vil beskrive dette, så er det fint.