digipolisantwerp / antwerp-ui_angular

Antwerp UI is a component interface library for building user interfaces and responsive web apps.
https://digipolisantwerp.github.io/antwerp-ui_angular
MIT License
13 stars 19 forks source link

Datepicker bugs in validation #206

Open nickcuypers opened 4 years ago

nickcuypers commented 4 years ago

Niet-bestaande datums Op het scherm waar ik nieuwe lidmaatschappen kan toevoegen, vul ik het volgende in:

image

Bij het invullen of opslaan krijg ik geen foutmelding. Na het opslaan zie ik:

image

Ik had hier verwacht dat ik al rode tekst of zo zou krijgen bij het ingeven. Als ik 32/02/2020 ingeef als geboortedatum (of een andere niet bestaande datum), dan krijg ik van de datepicker component achterliggend deze datum ook terug, maar visueel toont hij een andere datum dus visueel is er wel een soort validatie en correctie maar achterliggend niet. Correctie is dan weer niet wenselijkn eerder een validatie error of indicatie.

Onvolledige datums Als je de datum manueel invult en niet via de datepicker, en dan bijv. enkel een dag invult, dan kan men opslaan, maar de datum is dan wel verkeerd. Bijv. ik vul enkel als dag 12 in. Bij het opslaan maakt het systeem er 01/12/2001 van.

image

Na opslaan staat er:

image

Als ik 12/03 invul, maar geen jaartal, dan wordt er 12/03/2001 van gemaakt. Lijkt iets te hebben met jaartal 2001 😊.

Dus achterliggend maakt hij van gedeeltelijke datums zelf maar iets terwijl ik hier een validatie error zou verwachten.

TriangleJuice commented 4 years ago

Hey @nickcuypers, kan je eens checken of jullie wel de laatste versie van ACPaaS UI gebruiken, want als ik beide bugs probeer te reproduceren op de datepicker documentatiepagina dan krijg ik telkens wél een foutboodschap.

nickcuypers commented 4 years ago

@TriangleJuice oei hier heb ik geen melding van gekregen vandaar mijn late antwoord. heb je een versienummer zodat we zeker zijn dat we over dezelfde versie spreken indien verder onderzoek vereist is?

TriangleJuice commented 4 years ago

Zeker, @nickcuypers! De huidige versie is v4.5.2 waar het probleem zich volgens mij niet voordoet.

nickcuypers commented 4 years ago

@TriangleJuice die versie gaat over de calendar zeker? https://www.npmjs.com/package/@acpaas-ui/ngx-calendar maar wij maken gebruik van de ngx-components package https://www.npmjs.com/package/@acpaas-ui/ngx-components waarbij we de genoemde issues hebben bij de datepicker

TriangleJuice commented 4 years ago

Hey @nickcuypers het klopt dat de werking van die versie met momenten niet helemaal juist was. Is het voor jullie veel moeite om te upgraden naar de nieuwe versie van ACPaaS UI? Welke versie van Angular gebruiken jullie?

nickcuypers commented 4 years ago

@TriangleJuice hangt ervan af wat de impact is, we zitten nl in de laatste fase van het onderweg project, momenteel gebruiken angular 8.2.14 met volgende packages van acpaas ui image

nickcuypers commented 4 years ago

@TriangleJuice kan je hiervoor de impact nog aangeven?

TriangleJuice commented 4 years ago

Hey @nickcuypers normaal gezien is de enige impact dat jullie de ACPaaS UI import paths moeten veranderen. Maar veel hangt ook af van de versie van Angular die jullie gebruiken. Als dat v8 of hoger is, dan zou je nagenoeg geen problemen mogen hebben. Als je niet zeker bent, stuur me dan even de repo door, dan kan ik eventueel zelf mee kijken of het een grote impact gaat hebben of niet.

nickcuypers commented 4 years ago

@TriangleJuice als dat de enige impact is, kunnen we dit wel doen lijkt me, voor de zekerheid kan je het checken in volgende repo: https://bitbucket.antwerpen.be/projects/OCMW/repos/onderweg_app_aspnetcore/browse

kan je zoiezo even verduidelijken welke package we dan moeten gebruiken, m.a.w. wat bedoel je met "upgraden naar de nieuwe versie van ACPaaS UI"?

TriangleJuice commented 4 years ago

Hey @nickcuypers Ik heb het vlug even bekeken en heb volgende zaken vastgesteld:

Sidenote: er zijn volgens mij enkele componenten in de ACPaaS UI bibliotheek die jullie nog niet gebruiken. Ik geef het maar mee om in de toekomst gemakkelijker te kunnen upgraden. De header is daar een voorbeeld van.

nickcuypers commented 3 years ago

hey @TriangleJuice , ik heb die aanpassing gedaan en alles werkt idd nog maar ik zie ook geen verschil met de problemen die ik aangaf in dit bugticket. ik krijg idd in sommige gevallen de error 'ongeldige datum' maar lang niet op alle momenten die in dit ticket zijn aangegeven. Als ik bv maar een gedeeltelijke datum ingeef geeft hij enkel een error als de dag of maand of het jaar maar gedeeltelijk zijn ingegeven, maar als ik bv 01// ingeef krijg ik geen error en na opslaan maakt hij er zelf nog steeds een andere datum van. ook gaat hij pas vanaf 13// een error geven wss omdat hij dat als maand rekent terwijl dat de plek is om een dag in te vullen dus structureel lijken er mij nog wat zaen mis te lopen en dit zijn maar enkele voorbeelden. Kan dit worden nagekeken?

TriangleJuice commented 3 years ago

Hey @nickcuypers Kan het zijn dat de datepickers waarvan je spreekt, deel uitmaken van de form renderer en dus niet 'rechtstreeks' uit ACPaaS UI komen? Kan je bevestigen dat deze foutieve data ook geen error geven in de datepicker onderaan deze pagina?

nickcuypers commented 3 years ago

@TriangleJuice die pagina lijkt nog een nieuwere versie te gebruiken want daar komen die errors idd niet voor. maar het gaat wel degelijk om de aui-datepicker en dus niet een datepicker uit een form. op dit moment hebben we volgende versie staan: image met volgende imports: image en volgende config: image

TriangleJuice commented 3 years ago

@nickcuypers Ik denk dat je toch iets over het hoofd ziet, want deze versie is maar sinds enkele minuten verhoogd en heeft exact dezelfde configuratie als die van jou.