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

feat: Remove ToastInjector use Injector.create #947

Merged
merged 2 commits into from Sep 28, 2022

Conversation

AndrewKushnir
Copy link
Contributor

In this PR the custom ToastInjector (that provides a single token) is removed in favor of creating a new injector instance using the Injector.create call.

The goal of this change is to make the library forward-compatible with an upcoming change to the Injector interface, see angular/angular#46761.

…njector.create

This commit removes a custom ToastInjector (that provided a single token) in favor of creating a new injector instance using the `Injector.create` call.

The goal of this change is to make the library forward-compatible with an upcoming change to the Injector interface, see angular/angular#46761.
Since the Toastinjector was removed in the previous commit, the file name becomes inaccurate. This commit renames the toast-injector.ts to toast-ref.ts to better reflect the content of the file.
@codecov
Copy link

codecov bot commented Sep 26, 2022

Codecov Report

Base: 55.57% // Head: 55.42% // Decreases project coverage by -0.15% ⚠️

Coverage data is based on head (d17fce1) compared to base (8a0a9bf).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #947      +/-   ##
==========================================
- Coverage   55.57%   55.42%   -0.16%     
==========================================
  Files          19       19              
  Lines         511      507       -4     
  Branches       88       87       -1     
==========================================
- Hits          284      281       -3     
  Misses        216      216              
+ Partials       11       10       -1     
Impacted Files Coverage Δ
src/lib/toastr/toast-ref.ts 75.86% <ø> (ø)
src/lib/toastr/toastr-config.ts 94.73% <ø> (ø)
src/lib/toastr/toastr.service.ts 51.04% <100.00%> (+2.62%) ⬆️
src/app/home/home.component.ts 58.57% <0.00%> (-1.43%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@scttcper scttcper changed the title refactor: remove ToastInjector in favor of creating an instance via Injector.create feat: remove ToastInjector in favor of creating an instance via Injector.create Sep 28, 2022
@scttcper scttcper changed the title feat: remove ToastInjector in favor of creating an instance via Injector.create feat: Remove ToastInjector use Injector.create Sep 28, 2022
@scttcper scttcper merged commit b765833 into scttcper:master Sep 28, 2022
@scttcper
Copy link
Owner

🎉 This PR is included in version 15.1.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@AndrewKushnir
Copy link
Contributor Author

@scttcper thank you! 👍

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

Successfully merging this pull request may close these issues.

None yet

2 participants