Skip to content

Navigation Menu

Sign in

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings

docs: add intro to new ssr guide and refactor information architecture #61377

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 1 commit into
base: main
Choose a base branch
Loading
from
Open

docs: add intro to new ssr guide and refactor information architecture #61377

wants to merge 1 commit into from
+72 −70

Conversation

bencodezen
Copy link
Contributor

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?

  1. I added an introduction for those new to the concept of SSR and hybrid rendering so that they have a baseline to start from.
  2. I updated some of the language in the setup section to show more of the connection between SSR and hybrid rendering .
  3. I reorganized the sections to try and reflect what would be most relevant to people learning about hybrid rendering and pushed more of the advanced content further down the page.

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

All feedback is always welcome!

@angular-robot angular-robot bot added the area: docs Related to the documentation label May 15, 2025
@ngbot ngbot bot added this to the Backlog milestone May 15, 2025
@bencodezen bencodezen requested a review from alan-agius4 May 15, 2025 18:17
@AndrewKushnir

This comment was marked as resolved.

Sign in to view
@thePunderWoman thePunderWoman added action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews target: patch This PR is targeted for the next patch release labels May 16, 2025
@alan-agius4 alan-agius4 added target: rc This PR is targeted for the next release-candidate action: review The PR is still awaiting reviews from at least one requested reviewer and removed action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews target: patch This PR is targeted for the next patch release labels May 18, 2025
alan-agius4
alan-agius4 approved these changes May 18, 2025
View reviewed changes
Copy link
Contributor

@alan-agius4 alan-agius4 left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for this.

@alan-agius4 alan-agius4 added action: merge The PR is ready for merge by the caretaker action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews and removed action: review The PR is still awaiting reviews from at least one requested reviewer action: merge The PR is ready for merge by the caretaker labels May 18, 2025
@alan-agius4 alan-agius4 changed the base branch from 20.0.x to main May 18, 2025 08:07
@pullapprove pullapprove bot added the requires: TGP This PR requires a passing TGP before merging is allowed label May 18, 2025
@alan-agius4 alan-agius4 added action: merge The PR is ready for merge by the caretaker and removed action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews labels May 18, 2025
@pullapprove pullapprove bot removed the requires: TGP This PR requires a passing TGP before merging is allowed label May 18, 2025
@alan-agius4 alan-agius4 removed the request for review from jelbourn May 18, 2025 08:10
@github-actions GitHub Actions
Copy link

Deployed adev-preview for 3df3dac to: https://ng-dev-previews-fw--pr-angular-angular-61377-adev-prev-bnvjyu6t.web.app

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@alan-agius4 alan-agius4 alan-agius4 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: rc This PR is targeted for the next release-candidate
Projects
None yet
Milestone
Backlog
Development

Successfully merging this pull request may close these issues.
4 participants
@bencodezen @AndrewKushnir @alan-agius4 @thePunderWoman
Morty Proxy This is a proxified and sanitized view of the page, visit original site.