rojeras / tpinfo-frontend

Obsolet!
GNU Lesser General Public License v3.0
1 stars 3 forks source link

Hippos vy uppdateras inte efter nytt val i datumlistan #4

Closed Marie-LouiseAndersson closed 3 years ago

Marie-LouiseAndersson commented 3 years ago

Scenario: 1 Per idag 2021-03-10, Jag väljer tjänstekonsument "...-9N3V" Obstetrix

2 Jag väljer datum 2021-02-22

3 Jag försöker välja 2021-03-10, dagens datum saknas i datumlistan. Det senaste datum jag kan välja är 2021-03-09, jag väljer det, behörigheter som avtakats efter 2021-02-22 försvinner inte från vyn (vyn uppdateras inte).

rojeras commented 3 years ago

Confirmed! Can you please add a screen shot highlighting the problem.

Marie-LouiseAndersson commented 3 years ago

Bifogar skärmprintar för varje steg.

Marie-LouiseAndersson commented 3 years ago

Funkar inte att bifoga skärmprintar, jag har försökt både genom att klippa in i kommentaren med Ctrl-V såväl som att jag försöker välja ut fil via Utforskaren, det tar inte! :(

rojeras commented 3 years ago

Provar att lägga in skärmbilderna.

Hippo, felrapport datumlista, konsument 9N3V, steg 1 Hippo, felrapport datumlista, konsument 9N3V, steg 2

Ser ut att gå bra. Jag bara drog in de två första filerna från utforskaren.

Hippo, felrapport datumlista, konsument 9N3V, steg 3

Och det gick bra att lägga in den tredje filen genom att klicka på texten "Attach files by..." nedan och välja i utforskaren. Märkligt.

leoroj commented 3 years ago

Kan jag lägga in en bild från mitt andra testkonto månne? Testar en paste

image

Ser också ut att gå bra.

Marie-LouiseAndersson commented 3 years ago
Hippo, felrapport datumlista, konsument 9N3V, steg 1

Testar igen, att lägga in dublett av print, för steg 1. Tidigare använde jag Edge, nu använder jag Chrome.

Marie-LouiseAndersson commented 3 years ago

Aah, ok, nu gick det bra. Då var nog orsaken att jag jobbade i Edge, ska ta för vana att alltid använda Chrome.

rojeras commented 3 years ago

Three fixes applied:

  1. Ensure that the current date (= the last update date in TPDB) always is added to the date selection list.
  2. When the URL parameter "filter" is parsed, missing start- and end dates should be converted to current date in the Bookmark object. They were left as null.
  3. Ensure that the currently selected date is always included in the date selection list.

With these changes the problem reported in this issue is solved.

rojeras commented 3 years ago

@Marie-LouiseAndersson , fix för denna issue är installerad i QA. Kan du pls verifiera att det problem du anmälde är löst.

Marie-LouiseAndersson commented 3 years ago

Aah, ja, jag måste vara lite trött och tankspridd, jag testade i produktion... Nu när jag testade i QA gick det mycket bättre, behövde inte trycka ctrl + R (ursäkta förvirringen). Så, ja, fixen är validerad ok.

rojeras commented 3 years ago

Då beställer jag installation.

MvH/LEO

On Fri, Mar 12, 2021 at 6:00 PM Marie-LouiseAndersson < @.***> wrote:

Aah, ja, jag måste vara lite trött och tankspridd, jag testade i produktion... Nu när jag testade i QA gick det mycket bättre, behövde inte trycka ctrl + R (ursäkta förvirringen). Så, ja, fixen är validerad ok.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/rojeras/tpinfo-frontend/issues/4#issuecomment-797622997, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADJVEDQRZS5WMQPIZKTHEF3TDJCC5ANCNFSM4Y5QJVHA .

--

Lars Erik Röjerås

@.***

www.skoview.se

Tel: 0705-58 90 08

Marie-LouiseAndersson commented 3 years ago

Ja, gör det!

Vänliga hälsningar

Marie-Louise


Marie-Louise Andersson FO Informationsinfrastruktur, Förvaltningsledare IT Avdelning Applikation, SLL IT Mobilnummer: +46 (0) 704 200 027 E-post: @.***

Region Stockholm SLL IT Box 22550, 104 22 Stockholm Norra Stationsgatan 69 Telefon: 08-737 25 00 www.sll.sehttp://www.sll.se/

Stockholms Läns Landsting blev 1 januari 2019 Region Stockholm

Från: rojeras @.> Skickat: den 15 mars 2021 07:29 Till: rojeras/tpinfo-frontend @.> Kopia: Marie-Louise Andersson @.>; Mention @.> Ämne: Re: [rojeras/tpinfo-frontend] Hippos vy uppdateras inte efter nytt val i datumlistan (#4)

Då beställer jag installation.

MvH/LEO

On Fri, Mar 12, 2021 at 6:00 PM Marie-LouiseAndersson < @.<mailto:@.>> wrote:

Aah, ja, jag måste vara lite trött och tankspridd, jag testade i produktion... Nu när jag testade i QA gick det mycket bättre, behövde inte trycka ctrl + R (ursäkta förvirringen). Så, ja, fixen är validerad ok.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/rojeras/tpinfo-frontend/issues/4#issuecomment-797622997, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADJVEDQRZS5WMQPIZKTHEF3TDJCC5ANCNFSM4Y5QJVHA .

--

Lars Erik Röjerås

@.<mailto:@.>

www.skoview.sehttp://www.skoview.se

Tel: 0705-58 90 08

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/rojeras/tpinfo-frontend/issues/4#issuecomment-799151288, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AS5C3ZZI6EXDU5F7J3OYQJTTDWSMFANCNFSM4Y5QJVHA.

rojeras commented 3 years ago

Fixed with 7.1.1