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/issue 379 #380

Merged
merged 4 commits into from
Nov 7, 2023
Merged

Conversation

AntLer-24rus
Copy link
Contributor

@maximelafarie, hello!

I think I have solved the problem described in issued #379. In the test environment, at first glance, everything works correctly. The angular version was also raised to 16 and the expectations in the tests were slightly added. This library is very important to me, two large projects depend on it and I would not like to switch to something else.

If possible, I ask you to conduct a review and include changes in the production build. Thank you in advance!

close #379

Copy link
Owner

@maximelafarie maximelafarie left a comment

Choose a reason for hiding this comment

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

LGTM. Thank you very much @AntLer-24rus for your work and the kind words. You did a great job, I will push a release today!

@maximelafarie
Copy link
Owner

Also thank you @mohsen-amani for pre-approving

@maximelafarie maximelafarie merged commit 9f15d76 into maximelafarie:master Nov 7, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

modalService create<>() throws NG0902: This view is already attached to a ViewContainer!
3 participants