Skip to content

Comments

refactor(http): expose withXhr to prepare for fetch by default#67231

Merged
thePunderWoman merged 1 commit intoangular:mainfrom
JeanMeche:withxhr
Feb 24, 2026
Merged

refactor(http): expose withXhr to prepare for fetch by default#67231
thePunderWoman merged 1 commit intoangular:mainfrom
JeanMeche:withxhr

Conversation

@JeanMeche
Copy link
Member

This commit sets up the necessary changes that would allow us to safely migrate G3 before switch to the FetchBackend by default.

For now the HttpXhrBackend is still the default backend for the HttpClient.

This commit sets up the necessary changes that would allow us to safely migrate G3 before switch to the `FetchBackend` by default.

For now the `HttpXhrBackend` is still the default backend for the `HttpClient`.
@angular-robot angular-robot bot added the area: common/http Issues related to HTTP and HTTP Client label Feb 23, 2026
@ngbot ngbot bot added this to the Backlog milestone Feb 23, 2026
@JeanMeche JeanMeche marked this pull request as ready for review February 23, 2026 21:27
@pullapprove pullapprove bot requested a review from thePunderWoman February 23, 2026 21:27
@JeanMeche JeanMeche added the target: major This PR is targeted for the next major release label Feb 23, 2026
Copy link
Contributor

@thePunderWoman thePunderWoman left a comment

Choose a reason for hiding this comment

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

LGTM

reviewed-for: fw-general, public-api

@thePunderWoman thePunderWoman removed the request for review from AndrewKushnir February 23, 2026 21:44
@JeanMeche JeanMeche added the action: merge The PR is ready for merge by the caretaker label Feb 24, 2026
@thePunderWoman thePunderWoman merged commit f30ed6b into angular:main Feb 24, 2026
24 checks passed
@thePunderWoman
Copy link
Contributor

This PR was merged into the repository. The changes were merged into the following branches:

JeanMeche added a commit to JeanMeche/angular that referenced this pull request Feb 24, 2026
This is a follow-up of angular#67231, in preparation of making the `FetchBackend` the default
JeanMeche added a commit to JeanMeche/angular that referenced this pull request Feb 24, 2026
This is a follow-up of angular#67231, in preparation of making the `FetchBackend` the default
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: merge The PR is ready for merge by the caretaker area: common/http Issues related to HTTP and HTTP Client target: major This PR is targeted for the next major release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants