Skip to content

Conversation

tomuli
Copy link
Contributor

@tomuli tomuli commented Aug 27, 2025

Kuntalaisen aloitussivuksi halutaan kalenterinäkymä (mikäli tämä on mahdollinen, eli ainakin yhdellä lapsella on sijoitus) sekä vahvalla että heikolla kirjautumisella.

Muutos vaati päivityksiä testeihin. Moni testi alkaa sisäänkirjautumisella, jossa oletettu sivu on ollut useimmiten edellinen "Hakemukset". Poistettu ylimääräiset navigoinnit kalenterisivulle, mikäli uudella tavalla kirjautuessa sinne ollaan jo saavuttu.

Kalenterisivu lataa sisältöä lisää perussivun latauduttua. Tämä aiheuttaa ongelman niissä testeissä, joissa navigoidaan yläosan pudotusvalikoista. Mikäli kalenterin sisältö latautuu valmiiksi sen jälkeen kun pudotusvalikko on testin toimesta avattu, mutta ennen kuin valikkoa on klikattu, kalenterisivun renderöinti "sulkee" pudotusvalikon ennen kuin testi on ehtinyt painaa navigointia. Tämä pyritään estämään asettamalla useimmissa paikoissa käytettyyn enduserLoginiin tarkistus että kalenterin sisältö on latautunut (mikäli urlin mukaan ollaan ko. sivulla).

@tomuli tomuli added the enhancement Uusi toiminnallisuus tai parannus label Aug 27, 2025
@tomuli tomuli force-pushed the calendar-startpage-after-placement branch 3 times, most recently from bb54dda to 15eb32c Compare August 29, 2025 13:33
@tomuli tomuli marked this pull request as ready for review August 29, 2025 13:56
e2e test login to wait for calendar data before next interaction

e2e-test fix related to initial calendar page
@tomuli tomuli force-pushed the calendar-startpage-after-placement branch from 15eb32c to a671592 Compare September 2, 2025 09:10
@tomuli tomuli merged commit 415f777 into master Sep 2, 2025
28 checks passed
@tomuli tomuli deleted the calendar-startpage-after-placement branch September 2, 2025 10:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Uusi toiminnallisuus tai parannus
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants