FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Revert "fix(router): limit protocol-relative URL handling to serialization" by amishne · Pull Request #70245 · angular/angular · GitHub

Revert "fix(router): limit protocol-relative URL handling to serialization" - #70245

Merged
thePunderWoman merged 1 commit into
angular:mainfrom
amishne:revert-url-tree-protocol-relative
Aug 18, 2026
Merged

Revert "fix(router): limit protocol-relative URL handling to serialization"#70245
thePunderWoman merged 1 commit into
angular:mainfrom
amishne:revert-url-tree-protocol-relative

Conversation

amishne commented Aug 17, 2026
edited
Loading

Copy link
Copy Markdown
Contributor

This reverts commit 435f8b2.

ngbot Bot added this to the Backlog milestone Aug 17, 2026
amishne added the target: patch This PR is targeted for the next patch release label Aug 17, 2026
amishne requested a review from JeanMeche August 17, 2026 22:44
amishne marked this pull request as ready for review August 17, 2026 22:46
amishne force-pushed the revert-url-tree-protocol-relative branch from 7587ece to 8dc0ba4 Compare August 17, 2026 23:58
pullapprove Bot requested a review from kirjs August 17, 2026 23:58
amishne changed the title fix(router): remove protocol-relative URL check during serialization Revert "fix(router): limit protocol-relative URL handling to serialization" Aug 18, 2026
JeanMeche added the action: merge The PR is ready for merge by the caretaker label Aug 18, 2026
pullapprove Bot requested a review from crisbeto August 18, 2026 15:01
JeanMeche removed request for crisbeto and kirjs August 18, 2026 15:01
pullapprove Bot requested review from crisbeto and kirjs August 18, 2026 15:01
pullapprove Bot requested a review from crisbeto August 18, 2026 15:04

thePunderWoman left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

reviewed-for: public-api

thePunderWoman merged commit 292991e into angular:main Aug 18, 2026
25 checks passed

Copy link
Copy Markdown
Contributor

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

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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: router target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants


Back | FazBrowse Home | New Git URL