-
-
Notifications
You must be signed in to change notification settings - Fork 32.3k
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
[useScrollTrigger] Add test to fix useScrollTrigger on first load #20680
[useScrollTrigger] Add test to fix useScrollTrigger on first load #20680
Conversation
@oliviertassinari we need to merge before run checks? |
Details of bundle changes.Comparing: 48134ab...bac13f1 Details of page changes
|
@marcosvega91 Perfect, #20678 does fix this test. I think that we could merge #20678 then #20680. |
026b610
to
bac13f1
Compare
@marcosvega91 We have a background task running, a migration from enzyme to testing-library. We don't have a dedicated migration effort, we don't do it just for the sake of it, however, we try to migrate as we go. Here, given had to wrap our head around what's going on in this component, it was a good opportunity to migrate the tests. Side note, your GitHub email isn't configured correctly (we don't see your avatar nor slug): |
For the user I changed it now for the project, I have another one globally. What do you mean with "given had to wrap our head around what's going on in this component" ? |
You can add multiple emails.
Load the context in our head. |
Yes I saw the code and understand :)
Yes I know, thanks |
test to fix #20676