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

fix(core): BrowserAnimationsModule is optional dependency now #1377 #2054

Merged
merged 1 commit into from
Apr 2, 2022

Conversation

satanTime
Copy link
Member

closes #1377

Copy link

@codeclimate codeclimate bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR diff size of 23843 lines exceeds the maximum allowed for the inline comments feature.

@zuzusik
Copy link

zuzusik commented Apr 1, 2022

According to test results It seems like this actually perfectly reproduces original issue brought up in #1377 😄

@satanTime satanTime force-pushed the issues/1377 branch 2 times, most recently from ff983fb to 5666df4 Compare April 2, 2022 08:53
@satanTime satanTime changed the title fix(core): optional animations and forms #1377 fix(core): BrowserAnimationsModule is optional dependency now #1377 Apr 2, 2022
@satanTime satanTime force-pushed the issues/1377 branch 8 times, most recently from 9124bae to fd67209 Compare April 2, 2022 10:48
…-mom#1377

Please add `ngMocks.globalReplace(BrowserAnimationsModule, NoopAnimationsModule);` if you need the old behavior.
@satanTime satanTime merged commit 1cc5a0b into help-me-mom:master Apr 2, 2022
@satanTime satanTime deleted the issues/1377 branch April 2, 2022 11:01
@satanTime
Copy link
Member Author

v13.4.0 has been released and contains a fix for the issue. Feel free to reopen the issue or to submit a new one if you meet any problems.

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

Successfully merging this pull request may close these issues.

Library tests fail for projects which don't include @angular/animations module
2 participants