Skip to content

docs: add references to SSR guide for caching options#64347

Closed
SkyZeroZx wants to merge 1 commit intoangular:mainfrom
SkyZeroZx:docs/update-caching-data-when-using-httpclient
Closed

docs: add references to SSR guide for caching options#64347
SkyZeroZx wants to merge 1 commit intoangular:mainfrom
SkyZeroZx:docs/update-caching-data-when-using-httpclient

Conversation

@SkyZeroZx
Copy link
Contributor

@SkyZeroZx SkyZeroZx commented Oct 10, 2025 •
edited
Loading

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Documentation content changes
  • angular.dev application / infrastructure changes
  • Other... Please describe:

What is the current behavior?

Issue Number: N/A

What is the new behavior?

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

@pullapprove pullapprove bot requested a review from mmalerba October 10, 2025 17:48
@angular-robot angular-robot bot added the area: docs Related to the documentation label Oct 10, 2025
@ngbot ngbot bot added this to the Backlog milestone Oct 10, 2025
JeanMeche
JeanMeche reviewed Oct 10, 2025
View reviewed changes
Copy link
Member

@JeanMeche JeanMeche left a comment

Choose a reason for hiding this comment

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

More generally, this change goes toward a API documentation and we a losing the general message that the guide should give, ie give guidance (show the possibility of global vs local cache configuration etc).

We already have the API reference for all those details, so I would revisit some of the changes here

@SkyZeroZx SkyZeroZx force-pushed the docs/update-caching-data-when-using-httpclient branch from 6651d78 to 03887e8 Compare October 10, 2025 20:47
JeanMeche
JeanMeche reviewed Oct 17, 2025
View reviewed changes
JeanMeche
JeanMeche reviewed Oct 17, 2025
View reviewed changes
@SkyZeroZx SkyZeroZx force-pushed the docs/update-caching-data-when-using-httpclient branch from 03887e8 to 8866c8d Compare October 17, 2025 23:25
@github-actions
Copy link

github-actions bot commented Oct 20, 2025

Deployed adev-preview for 8866c8d to: https://ng-dev-previews-fw--pr-angular-angular-64347-adev-prev-qcu0wa00.web.app

Note: As new commits are pushed to this pull request, this link is updated after the preview is rebuilt.

@JeanMeche JeanMeche removed the request for review from mmalerba October 20, 2025 22:30
@JeanMeche JeanMeche added action: merge The PR is ready for merge by the caretaker target: patch This PR is targeted for the next patch release labels Oct 20, 2025
thePunderWoman pushed a commit that referenced this pull request Oct 21, 2025
@SkyZeroZx @thePunderWoman
docs: add references to SSR guide for caching options (#64347)
ed99bc9 
PR Close #64347
@thePunderWoman
Copy link
Contributor

thePunderWoman commented Oct 21, 2025

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

@angular-automatic-lock-bot
Copy link

angular-automatic-lock-bot bot commented Nov 21, 2025

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 Nov 21, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@JeanMeche JeanMeche JeanMeche approved these changes

Assignees

No one assigned

Labels

action: merge The PR is ready for merge by the caretaker adev: preview area: docs Related to the documentation target: patch This PR is targeted for the next patch release

Projects

None yet

Milestone

Backlog

Development

Successfully merging this pull request may close these issues.

3 participants

@SkyZeroZx @thePunderWoman @JeanMeche