Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

KAS-4933: Aanpassen CSS rond ember-flatpickr (mbt package update) #2276

Open
wants to merge 4 commits into
base: development
Choose a base branch
from

Conversation

brenner-company
Copy link
Contributor

Tickets binnen Jira:

⚠️ Moet op zich niet echt gereviewed worden, maar alle tests moeten wel volledig ok zijn om te weten dat er geen functionele impact is.

@brenner-company brenner-company self-assigned this Dec 17, 2024
@ValenberghsSven
Copy link
Contributor

Enkele testen lijken te falen op datepicker. precies of dat de "popup" datepicker er niet is terwijl die verwacht wordt er te zijn.

@brenner-company
Copy link
Contributor Author

brenner-company commented Jan 4, 2025

Enkele testen lijken te falen op datepicker. precies of dat de "popup" datepicker er niet is terwijl die verwacht wordt er te zijn.

@ValenberghsSven Ik zie niet direct waar de issue kan liggen 🤔 Als ik manueel even die testflow doorloop lijkt alles goed te werken. Zou er eventueel een reden zijn dat er een delay zou moeten toegevoegd worden? (Misschien dat de .open class er nu met de update er iets trager op komt?)

@brenner-company
Copy link
Contributor Author

brenner-company commented Jan 5, 2025

@ValenberghsSven Zie het verschil bij test #13 bij de 1ste en 2de run (die wel slaagt).

@ValenberghsSven
Copy link
Contributor

Ik zal deze testen eens bekijken en zien of ik het lokaal ook stuk krijg deze week

…nd-kaleidos into DES/KAS-4933-css-adjustments-regarding-flatpickr-update
@ValenberghsSven
Copy link
Contributor

ValenberghsSven commented Jan 9, 2025

Update voor deze, het lijkt te zijn dat er een moment is tijdens het laden van de pagina waar de datepicker element gewoon niet opengaat bij het klikken in het inputveld.
Manueel reproduceerbaar enkel bij traag netwerk.
Maar jenkins loopt daar dus tegen. Fix voor de jenkins testen zal dus zijn om een mini timeout toe te voegen ofzo.
Er is geen duidelijke indicatie (zoals eens loader) waar op we kunnen wachten

@ValenberghsSven
Copy link
Contributor

kleine timeout toevoegen lijkt te hebben geholpen (100 milliseconden op een beperkt aantal plaatsen).
build 4 is ok mits 1 flaky maar ik ga nog enkele extra runs laten doen.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants