Move the Vacation Planner introduction into the Samples section - #24
Merged
Conversation
The repository opened with the Vacation Planner app, its list of backing data services and a screenshot of its UI, which described the samples rather than the repository: it now also holds thirteen tutorials that never deploy that app at all, so the first thing a reader saw was the narrower half of the content. The introduction is now about the repository, and points at its two kinds of content. Everything specific to the Vacation Planner, the description, the data-service list and the screenshot, moved down to the top of the Samples section, where the sample table it belongs to already lives. No content was dropped. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
This PR reorganizes the README introduction to describe the repository at a higher level and relocates the Vacation Planner–specific description to the Samples section where it best contextualizes the sample table.
Changes:
- Rewrote the README opening to describe the repo’s purpose and link readers to the two content types: Samples and Tutorials.
- Moved the Vacation Planner description, backing data-service list, and screenshot to the top of the Samples section, adding a sentence explaining why the app is kept identical across samples.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
The list named the eight services the sample table below already names, with links and more detail (the in-cluster PostgreSQL row even describes its streaming replicas), so it was pure duplication once the section moved down here. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
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
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.
Motivation
The README opened with the Vacation Planner app: a paragraph about it, the list of data services the samples vary, and a screenshot of its UI. That describes the samples, not the repository. With thirteen tutorials now in
tutorials/that never deploy that app, the first thing a reader saw was the narrower half of the content, and the screenshot set the expectation that the whole repository is one web app in eight flavours.Changes
The introduction is now about the repository and points at its two kinds of content, samples and tutorials, each linking to its section.
Everything specific to the Vacation Planner moved to the top of the Samples section, immediately above the sample table it describes: the app description, the list of backing data services, and the screenshot. One sentence was added there to say why the app is deliberately identical across samples, which is what makes them comparable.
No content was removed, and no other section changed.
Testing
Documentation only. The two new in-page links (
#samples,#tutorials) resolve to the existing headings, and the image path is unchanged.