Skip to content
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.

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

Clarify SSR Docs #185

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Clarify SSR Docs #185

wants to merge 3 commits into from

Conversation

ofekd
Copy link

@ofekd ofekd commented Sep 23, 2024

While reading the Lustre guide, I mistook the Server Side Rendering page as the only way to do SSR, and then didn't continue to read Server Components.

This PR remedies this by changing the three main places in which a potential user is introduced to SSR: the features section in the readme, and the two relevant guides.

For the readme, I've changed the features more for the new phrasing to flow better with the rest of them, feel free to change yourself or ask for changes.

For the guides, I've clearly separated the two modes of SSR, but the naming is still a bit confusing. I would at the very least change "Server Side Rendering" to more specific title, although I could not come up with one.

Ofek Doitch added 3 commits September 23, 2024 18:20
Clairfying this feature has caused the feature list flow to read
differently, so other minor adjustments were made
@metruzanca
Copy link

I like this, this is a good tweak IMO.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants