Skip to content

fix(router): Should not freeze original object used for route data - #53635

Closed
atscott wants to merge 1 commit into
angular:mainfrom
atscott:datafreeze
Closed

fix(router): Should not freeze original object used for route data#53635
atscott wants to merge 1 commit into
angular:mainfrom
atscott:datafreeze

Conversation

@atscott

@atscott atscott commented Dec 19, 2023

Copy link
Copy Markdown
Contributor

This was broken in 3278966 where the new code fails to copy the data object when not inheriting data.

fixes #53632

This was broken in 3278966 where the
new code fails to copy the data object when not inheriting data.

fixes angular#53632
@atscott atscott added action: review The PR is still awaiting reviews from at least one requested reviewer area: router target: patch This PR is targeted for the next patch release labels Dec 19, 2023
@ngbot ngbot Bot modified the milestone: Backlog Dec 19, 2023
@atscott atscott added action: merge The PR is ready for merge by the caretaker and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Dec 19, 2023
@atscott
atscott removed the request for review from AndrewKushnir December 19, 2023 17:19
@atscott

atscott commented Dec 19, 2023

Copy link
Copy Markdown
Contributor Author

This PR was merged into the repository by commit 502f300.

@atscott atscott closed this in 502f300 Dec 19, 2023
atscott added a commit that referenced this pull request Dec 19, 2023
…53635)

This was broken in 3278966 where the
new code fails to copy the data object when not inheriting data.

fixes #53632

PR Close #53635
@angular-automatic-lock-bot

Copy link
Copy Markdown

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 Jan 19, 2024
ChellappanRajan pushed a commit to ChellappanRajan/angular that referenced this pull request Jan 23, 2024
…ngular#53635)

This was broken in 3278966 where the
new code fails to copy the data object when not inheriting data.

fixes angular#53632

PR Close angular#53635
rlmestre pushed a commit to rlmestre/angular that referenced this pull request Jan 26, 2024
…ngular#53635)

This was broken in 3278966 where the
new code fails to copy the data object when not inheriting data.

fixes angular#53632

PR Close angular#53635
amilamen pushed a commit to amilamen/angular that referenced this pull request Jan 26, 2024
…ngular#53635)

This was broken in 3278966 where the
new code fails to copy the data object when not inheriting data.

fixes angular#53632

PR Close angular#53635
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 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.

Object set as readonly when passed as a route data

2 participants