Skip to content

Conversation

@alan-agius4
Copy link
Contributor

This commits adds makeStateKey, StateKey and TransferState methods in @angular/core as public API and deprecated the same exported symbols in @angular/platform-browser.

DEPRECATED: makeStateKey, StateKey and TransferState exports have been moved from @angular/platform-browser to @angular/core. Please update the imports.

- import {makeStateKey, StateKey, TransferState} from '@angular/platform-browser';
+ import {makeStateKey, StateKey, TransferState} from '@angular/core';

@alan-agius4 alan-agius4 force-pushed the deprecate-transfer-utils-from-platform-browser branch from 119aaae to e3abd03 Compare March 23, 2023 11:32
@angular-robot angular-robot bot added detected: deprecation PR contains a commit with a deprecation detected: feature PR contains a feature commit labels Mar 23, 2023
@alan-agius4 alan-agius4 added the area: core Issues related to the framework runtime label Mar 23, 2023
@ngbot ngbot bot added this to the Backlog milestone Mar 23, 2023
@alan-agius4 alan-agius4 added the target: major This PR is targeted for the next major release label Mar 23, 2023
@alan-agius4 alan-agius4 marked this pull request as ready for review March 23, 2023 11:38
@alan-agius4 alan-agius4 added the action: review The PR is still awaiting reviews from at least one requested reviewer label Mar 23, 2023
@pullapprove pullapprove bot requested review from dylhunn and jelbourn March 23, 2023 11:38
@alan-agius4 alan-agius4 force-pushed the deprecate-transfer-utils-from-platform-browser branch 6 times, most recently from e1707ad to 9d25ef6 Compare March 23, 2023 15:38
@JeanMeche
Copy link
Member

Would you be interested in a PR providing a migration for that ?

@alan-agius4
Copy link
Contributor Author

@JeanMeche, sure why not. If you want to take a look at a similar migration you can check https://github.com/angular/angular/tree/12.3.x/packages/core/schematics/migrations/xhr-factory

@pullapprove pullapprove bot requested review from jessicajaniuk March 27, 2023 22:39
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

@jessicajaniuk jessicajaniuk 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: fw-core, fw-platform-server, integration-tests, public-api

@jessicajaniuk jessicajaniuk removed their request for review March 27, 2023 22:46
@alan-agius4 alan-agius4 force-pushed the deprecate-transfer-utils-from-platform-browser branch 2 times, most recently from 269c4a9 to 6fd7e1a Compare March 28, 2023 07:50
@alan-agius4 alan-agius4 added the merge: caretaker note Alert the caretaker performing the merge to check the PR for an out of normal action needed or note label Mar 28, 2023
@alan-agius4 alan-agius4 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 merge: caretaker note Alert the caretaker performing the merge to check the PR for an out of normal action needed or note labels Mar 28, 2023
@alan-agius4 alan-agius4 force-pushed the deprecate-transfer-utils-from-platform-browser branch from 6fd7e1a to 0a891bb Compare March 29, 2023 08:41
@atscott atscott force-pushed the deprecate-transfer-utils-from-platform-browser branch from 0a891bb to b4a06ec Compare March 29, 2023 16:25
alan-agius4 and others added 3 commits March 29, 2023 16:49
This commits adds `makeStateKey`, `StateKey` and  `TransferState` methods in `@angular/core` as public API and deprecated the same exported symbols in `@angular/platform-browser`.

DEPRECATED:  `makeStateKey`, `StateKey` and  `TransferState` exports have been moved from `@angular/platform-browser` to `@angular/core`. Please update the imports.

```diff
- import {makeStateKey, StateKey, TransferState} from '@angular/platform-browser';
+ import {makeStateKey, StateKey, TransferState} from '@angular/core';
```
Co-authored-by: Andrew Kushnir <43554145+AndrewKushnir@users.noreply.github.com>
@alan-agius4 alan-agius4 force-pushed the deprecate-transfer-utils-from-platform-browser branch from b4a06ec to afc4d33 Compare March 29, 2023 16:57
@atscott
Copy link
Contributor

atscott commented Mar 29, 2023

This PR was merged into the repository by commit c024574.

@atscott atscott closed this in c024574 Mar 29, 2023
@alan-agius4 alan-agius4 deleted the deprecate-transfer-utils-from-platform-browser branch March 29, 2023 17:25
@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 Apr 29, 2023
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: core Issues related to the framework runtime detected: deprecation PR contains a commit with a deprecation detected: feature PR contains a feature commit target: major This PR is targeted for the next major release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants