docs: add Playwright screenshots to visualizer lab instructions#48
Conversation
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (7)
📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe lab documentation now includes additional screenshot embeds throughout Tasks 1–4 to visually confirm UI states and results. Task 5 is expanded with a screenshot of the SQL Console before listing example queries, providing clearer visual guidance during the lab exercises. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Pull request overview
Adds Playwright-captured screenshots to the Visualizer lab instructions so students can visually confirm they’re on the right screen and seeing the expected states during each task.
Changes:
- Inserted 7 screenshots into relevant steps in
LAB-VISUALIZER.md(Replication tab, sidebar state, results, indexing, SQL console). - Added screenshot assets under
10-databases/visualizer/screenshots/and referenced them with relative Markdown links.
Reviewed changes
Copilot reviewed 1 out of 8 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| 10-databases/visualizer/LAB-VISUALIZER.md | Embeds screenshots at key checkpoints to make the lab steps easier to follow/verify. |
| 10-databases/visualizer/screenshots/01-replication-tab.png | Screenshot asset for Replication tab checkpoint. |
| 10-databases/visualizer/screenshots/02-sidebar-state.png | Screenshot asset for sidebar state checkpoint. |
| 10-databases/visualizer/screenshots/03-replication-result.png | Screenshot asset for replicated result checkpoint. |
| 10-databases/visualizer/screenshots/04-consistency-committed.png | Screenshot asset for committed transfer checkpoint. |
| 10-databases/visualizer/screenshots/05-schema-no-index.png | Screenshot asset for “no index / full scan” checkpoint. |
| 10-databases/visualizer/screenshots/06-schema-with-index.png | Screenshot asset for “with index” checkpoint. |
| 10-databases/visualizer/screenshots/07-sql-console.png | Screenshot asset for SQL console section. |
Add 7 screenshots taken from live EC2 deployment via Playwright to LAB-VISUALIZER.md task instructions.
Summary by CodeRabbit