Project

General

Profile

Bug #4329

Brugerprofil - ændre pinkode felter

Added by Nhi A Sy 10 months ago. Updated 5 months ago.

Status:
Need more info
Priority:
Normal
Assignee:
Target version:
Estimated time:
URL med eksempel:
Kategorier:
Min konto - Brugerprofil

Description

Test på Rel 30 (ding2-7.x-4.6.2)og Rel 30_1(ding2-7.x-4.7.0-rc6)

Når man logger ind og gå ind på sin brugerprofil (se billede 1) 

Når man ændre/tilføje fraværendeperiode, så ændres pinkode felter (se billede 2) og når man ændrer sit pinkode, så ændrer pinkode felt sig ti lnoget 3. (se billede 3)

Jeg er logget alm. med cpr og pinkode.

Login--brugerprofil_1.png (14 KB) Login--brugerprofil_1.png 1. Brugerprofil Nhi A Sy, 05/10/2019 01:58 PM
Login--brugerprofil efter ændring i fraværendeperiode_2.png (9.73 KB) Login--brugerprofil efter ændring i fraværendeperiode_2.png 2. Ændrer fraværendeperiode Nhi A Sy, 05/10/2019 01:58 PM
login-ændre_pinkode_3.png (10.7 KB) login-ændre_pinkode_3.png 3. Ændrer pinkode Nhi A Sy, 05/10/2019 01:59 PM
Skærmbillede 2019-06-05 kl. 14.03.32.png (23.4 KB) Skærmbillede 2019-06-05 kl. 14.03.32.png Gitte Barlach, 06/05/2019 02:08 PM
Skærmbillede 2019-06-05 kl. 14.01.55.png (33.5 KB) Skærmbillede 2019-06-05 kl. 14.01.55.png Gitte Barlach, 06/05/2019 02:08 PM
Skærmbillede 2019-09-09 kl. 10.32.21.png (172 KB) Skærmbillede 2019-09-09 kl. 10.32.21.png Gitte Barlach, 09/09/2019 10:32 AM

History

#1 Updated by Rolf Madsen 10 months ago

  • Status changed from New to Ready for development
  • Assignee set to Christel Krabbenhøft
  • Target version set to Release 31 - bugfixes

Det lader til at være noget med den responsive visning.

Afhængig af hvor bredt browservinduet er ser det enten forkert eller rigtigt ud.

Lad os få den forbi B14.

#2 Updated by Christel Krabbenhøft 10 months ago

  • Assignee changed from Christel Krabbenhøft to Rasmus Høymann Laursen

Hej Rasmus. Kan I kigge på denne i indeværende uge? //Christel

#3 Updated by Rasmus Høymann Laursen 10 months ago

Absolut ; )

#4 Updated by Rasmus Høymann Laursen 10 months ago

  • Assignee changed from Rasmus Høymann Laursen to Steen Holten-Andersen

#5 Updated by Steen Holten-Andersen 10 months ago

  • Status changed from Ready for development to Needs code review
  • Assignee changed from Steen Holten-Andersen to Christel Krabbenhøft

Hej - PR: https://github.com/ding2/ding2/pull/1430

Der er flere ting her.

1. Form description feltet mangler noget styling så det placerer sig korrekt - løst i ovenstående PR.

2. Efter formen er gemt, skifter ID'et på formen som stylingen er afhængig af, derfor styles elementet ikke længere side-by-side - det er løst i ovenstående PR.

3. Der er ydermere også en lidt underlig opførsel, som jeg ikke helt ved om er indbefattet af dette Issue - og som er sværere at gennemskue. Når jeg på min installation skifter en værdi (eksempelvis 'E-mail', 'Afhent reserveringer på'), forsvinder 'Bekræft gammel pinkode feltet' - (indtil jeg logger ud og logger ind igen). Jeg kan dog stadig godt ændre mine pinkoder. Ved ikke om denne opførsel er tiltænkt - og eftersom det er del mere vanskeligt at fejlfinde på, vil jeg anbefale at det flyttes til et seperat issue - så det ikke forsinker de andre fejl.

Steen

#6 Updated by Christel Krabbenhøft 10 months ago

  • Assignee changed from Christel Krabbenhøft to Gitte Barlach

#7 Updated by Gitte Barlach 9 months ago

  • Assignee changed from Gitte Barlach to Jørgen Nielsen

#8 Updated by Jørgen Nielsen 9 months ago

  • Status changed from Needs code review to Reviewed
  • Assignee changed from Jørgen Nielsen to Gitte Barlach

reviewet og godkendt

#9 Updated by Kasper Garnæs 9 months ago

  • Status changed from Reviewed to Technical test
  • Assignee changed from Gitte Barlach to Kasper Garnæs

Merged.

#10 Updated by Kasper Garnæs 9 months ago

  • Assignee changed from Kasper Garnæs to Gitte Barlach

#11 Updated by Gitte Barlach 9 months ago

Testet på vanilla-fbs med 7.x-4.8.0-rc1

Ændring af pinkode og kun pinkode fungerer ok
Ændring af onhold-periode fungerer ok
Ændring af tlf.nummer fungerer ok, men KUN hvis man selv skriver jbert tegn, der skal stå i feltet og dermed kke anvender browserens forslag fra senest indtastede. Gør man det  står der pludselig nogle tegn i feltet "Bekræft pinkode". Og dermed får jeg en fejl når jeg forsøger at gemme. 
Se skærmdumps. 

Ved ikke om det er forventeligt resultat?

#12 Updated by Steen Holten-Andersen 9 months ago

  • Assignee changed from Steen Holten-Andersen to Philip Birk-Jensen

#13 Updated by Philip Birk-Jensen 8 months ago

Fejlen består i browserens autocomplete selv skriver pinkoden i det første password felt. Første gang profilen gemmes er det første felt "nuværende pinkode" og alt virker som det skal.

Men næste gang profilen redigeres er "nuværende pinkode" væk og derfor er det ny pinkode feltet som udfyldes af browseren.

 

Løsningen er vi ikke skjuler "nuværende pinkode" efter den er blevet udfyldt første gang (mit gæt er det er for at være flink systemet ikke har behov for at godkende pinkode igen).

#14 Updated by Gitte Barlach 8 months ago

Okay, sålænge vi ikke skriver nuværende pinkode i klar tekst, vil det være en god løsning. Når feltet er helt tomt bliver man jo som bruger i tvivl om man i det hele taget har en pinkode. 

#15 Updated by Christel Krabbenhøft 8 months ago

Philip, vil du gå videre med sagen i denne uge?

#16 Updated by Philip Birk-Jensen 8 months ago

  • Status changed from Reviewed - Needs info/rework to Need more info
  • Assignee changed from Philip Birk-Jensen to Gitte Barlach

Vi kan ikke genskabe fejlen i den nyeste master længere. Eskistere den stadig?

#17 Updated by Gitte Barlach 7 months ago

  • Assignee changed from Gitte Barlach to Nhi A Sy

Hej Nhi
Kan du verificere om fejlen stadig eksisterer?

#18 Updated by Gitte Barlach 6 months ago

Gentestet på upgrade-fbs med 7.x-4.8.0-rc5

Samme resultat som beskrevet i kommentar 11: 

Ændring af pinkode og kun pinkode fungerer ok
Ændring af onhold-periode fungerer ok
Ændring af tlf.nummer fungerer ok, men KUN hvis man selv skriver jbert tegn, der skal stå i feltet og dermed kke anvender browserens forslag fra senest indtastede. Gør man det  står der pludselig nogle tegn i feltet "Bekræft pinkode". Og dermed får jeg en fejl når jeg forsøger at gemme. 

#19 Updated by Christel Krabbenhøft 6 months ago

  • Target version changed from Release 31 - bugfixes to Release 32 - Bugfixes

Philip, vi når ikke at få den med i næste release, men du må meget gerne fikse den så snart som muligt.

#20 Updated by Rasmus Høymann Laursen 6 months ago

  • Assignee changed from Philip Birk-Jensen to Steen Holten-Andersen

#21 Updated by Steen Holten-Andersen 5 months ago

  • Status changed from Reviewed - Needs info/rework to Need more info
  • Assignee changed from Steen Holten-Andersen to Christel Krabbenhøft

Vi har testet på en ny ren up-to-date installation. Vi har testet med to forskellige test-brugere, på 2 forskellige computere i 5 forskellige browsere (Safari, Chrome, FF, Edge, IE11) - og kan ikke genskabe fejlen.

Kan i give en mere detaljeret step-by-step beskrivelse af hvornår og hvordan fejlen opstår.

Vi har forsøgt at rette telefonnummer på brugerne både som XXXXXXXX, som +45XXXXXXXX, som 0045XXXXXXXX - og brugt browser autoudfyld på feltet med alle telefon-formater. Ingen af delene giver fejl, og ændringerne gemmes som de skal.

#22 Updated by Christel Krabbenhøft 5 months ago

  • Assignee changed from Christel Krabbenhøft to Gitte Barlach

Also available in: Atom PDF