Repository navigation
docs(core): remove reference to ServerTransferStateModule from TransferState - #71052
Merged
kirjs merged 1 commit intoOct 1, 2026
Merged
Conversation
angular-robot
Bot
added
area: docs
JeanMeche
reviewed
Sep 29, 2026
Comment on lines
+61
to
+63
| * On the client, just inject this token using DI and use it, it will be lazily initialized. | ||
| * On the server it's already included if `renderApplication` function is used. Otherwise, import | ||
| * the `ServerTransferStateModule` module to make the `TransferState` available. | ||
| * On the server it's set up by `provideServerRendering()` (or `ServerModule` in NgModule-based | ||
| * applications), so no additional import is needed. |
Member
There was a problem hiding this comment.
The token is provided in root. Am I missing something we don't even need to mention that at all ?
Contributor
Author
There was a problem hiding this comment.
Good point, it's providedIn: 'root', so there's nothing to set up on either side. Updated to drop the server note entirely.
…erState ServerTransferStateModule was removed in v18. TransferState is provided in root, so no setup is needed on the server either; drop the outdated server note.
GabeSilvaDev
force-pushed
the
docs/transfer-state-removed-module
branch
from
September 30, 2026 04:35
c492880 to
f34025f
Compare
JeanMeche
approved these changes
Sep 30, 2026
JeanMeche
added
action: merge
Contributor
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
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
PR Checklist
Please check if your PR fulfills the following requirements:
PR Type
What kind of change does this PR introduce?
What is the current behavior?
Issue Number: N/A
The
TransferStateAPI docs still tell users to importServerTransferStateModulewhen using the server withoutrenderApplication.That module was removed in v18 in #54874, and the update guide already tells users to remove the import because it is no longer needed.
What is the new behavior?
The outdated server note is removed from the JSDoc.
TransferStateisprovidedIn: 'root', so no setup is needed on the server either.Does this PR introduce a breaking change?
Other information
This is a JSDoc-only change in
packages/core/src/transfer_state.ts.