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 ourterms of serviceand privacy statement.We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs(core): Update dependecy injection documentation #56844

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

MeddahAbdellah
Copy link

In the current documentation, nowhere is mentioned that the providers array in Route configures an Environment Injector, which might create confusion on which Injector will be looked at first, the one defined in the Route or the App root component.

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Documentation content changes
  • angular.io application / infrastructure changes
  • Other... Please describe:

What is the current behavior?

Issue Number: N/A

What is the new behavior?

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

In the current documentation, nowhere is mentioned that the providers array in Route configures an Environment Injector, which might create confusion on which Injector will be looked at first, the one defined in the Route or the App root component.
@angular-robot angular-robot bot added area: docs Related to the documentation area: core Issues related to the framework runtime labels Jul 4, 2024
@ngbot ngbot bot added this to theBacklogmilestone Jul 4, 2024
@JeanMeche JeanMeche requested review from atscott and removed request for josephperrott July 7, 2024 21:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: core Issues related to the framework runtime area: docs Related to the documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant