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

API Docs for the new provideHttpClient #47901

Closed
wants to merge 2 commits into from
Closed

Conversation

alxhub
Copy link
Member

@alxhub alxhub commented Oct 28, 2022

No description provided.

This commit adds documentation and API tagging for `provideHttpClient` and
the `with*` feature functions.
This rename reflects what the function actually does. Although the intention
is still not to have two different interceptor mechanisms, that is now
communicated in the docs for the function instead of in its name.

Fixes angular#47764
@alxhub alxhub added the target: rc This PR is targeted for the next release-candidate label Oct 28, 2022
@ngbot ngbot bot added this to the Backlog milestone Oct 28, 2022
Copy link
Contributor

@AndrewKushnir AndrewKushnir left a comment

Choose a reason for hiding this comment

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

👍

Comment on lines +56 to +57
* `withInterceptors(...)` feature.
*
Copy link
Contributor

Choose a reason for hiding this comment

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

nit (not a blocker for this PR): it might be helpful to have a small code snippet to show the structure of the provideHttpClient call (how you would use it with bootstrapApplication call). Since other with* functions refer to this one - it'd be a good location to have an example.

@pullapprove pullapprove bot requested review from atscott October 29, 2022 00:52
Copy link
Contributor

@AndrewKushnir AndrewKushnir left a comment

Choose a reason for hiding this comment

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

Reviewed-for: public-api

Copy link
Contributor

@atscott atscott left a comment

Choose a reason for hiding this comment

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

reviewed-for: public-api

Copy link
Contributor

@dylhunn dylhunn left a comment

Choose a reason for hiding this comment

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

reviewed-for: public-api, fw-http

@jessicajaniuk jessicajaniuk added the action: merge The PR is ready for merge by the caretaker label Nov 3, 2022
@AndrewKushnir
Copy link
Contributor

This PR was merged into the repository by commit febf29d.

AndrewKushnir pushed a commit that referenced this pull request Nov 4, 2022
#47901)

This rename reflects what the function actually does. Although the intention
is still not to have two different interceptor mechanisms, that is now
communicated in the docs for the function instead of in its name.

Fixes #47764

PR Close #47901
AndrewKushnir pushed a commit that referenced this pull request Nov 4, 2022
…7901)

This commit adds documentation and API tagging for `provideHttpClient` and
the `with*` feature functions.

PR Close #47901
AndrewKushnir pushed a commit that referenced this pull request Nov 4, 2022
#47901)

This rename reflects what the function actually does. Although the intention
is still not to have two different interceptor mechanisms, that is now
communicated in the docs for the function instead of in its name.

Fixes #47764

PR Close #47901
nouraellm pushed a commit to nouraellm/angular that referenced this pull request Nov 6, 2022
…gular#47901)

This commit adds documentation and API tagging for `provideHttpClient` and
the `with*` feature functions.

PR Close angular#47901
nouraellm pushed a commit to nouraellm/angular that referenced this pull request Nov 6, 2022
angular#47901)

This rename reflects what the function actually does. Although the intention
is still not to have two different interceptor mechanisms, that is now
communicated in the docs for the function instead of in its name.

Fixes angular#47764

PR Close angular#47901
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Dec 5, 2022
trekladyone pushed a commit to trekladyone/angular that referenced this pull request Feb 1, 2023
…gular#47901)

This commit adds documentation and API tagging for `provideHttpClient` and
the `with*` feature functions.

PR Close angular#47901
trekladyone pushed a commit to trekladyone/angular that referenced this pull request Feb 1, 2023
angular#47901)

This rename reflects what the function actually does. Although the intention
is still not to have two different interceptor mechanisms, that is now
communicated in the docs for the function instead of in its name.

Fixes angular#47764

PR Close angular#47901
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
action: merge The PR is ready for merge by the caretaker target: rc This PR is targeted for the next release-candidate
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants