fix: clean up visualizer SVG, remove wrong screenshots and instructor guide#47
Conversation
… guide - Fix replication dashed line to connect Primary->Replica horizontally at node level instead of floating below - Remove lab09 screenshots that showed wrong stack name - Remove instructor guide from lab 09
|
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 (3)
📝 WalkthroughWalkthroughThe pull request removes the complete instructor guide for the distributed file systems lab (682 lines), trims embedded screenshots from database visualizer documentation to streamline content, and refines SVG styling for the replication line visualization by replacing a labeled dashed line with a styled arrow. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Poem
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Pull request overview
This PR updates the Lab 10 database visualizer materials by correcting the replication indicator in the SVG diagram, removing screenshots that showed an incorrect CloudFormation stack name, and removing the Lab 09 instructor guide document.
Changes:
- Adjust the replication indicator in the visualizer SVG (use a dashed replication arrow below the main connection line).
- Remove screenshot embeds from the Lab 10 visualizer instructions and delete the associated (incorrect) screenshot assets.
- Remove
09-distributed-file-systems/INSTRUCTOR-GUIDE.md.
Reviewed changes
Copilot reviewed 3 out of 10 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| 10-databases/visualizer/index.html | Updates the replication line rendering to a dashed arrow with the existing SVG marker definition. |
| 10-databases/visualizer/LAB-VISUALIZER.md | Removes outdated screenshot references from the CloudFormation setup instructions. |
| 10-databases/visualizer/screenshots/02-aws-academy-modules.png | Removed outdated screenshot asset. |
| 10-databases/visualizer/screenshots/03-learner-lab-toolbar.png | Removed outdated screenshot asset. |
| 10-databases/visualizer/screenshots/04-aws-details-panel.png | Removed outdated screenshot asset. |
| 10-databases/visualizer/screenshots/06-cfn-create-stack.png | Removed outdated screenshot asset. |
| 10-databases/visualizer/screenshots/08-cfn-stack-name.png | Removed outdated screenshot asset. |
| 10-databases/visualizer/screenshots/10-cfn-creating.png | Removed outdated screenshot asset. |
| 10-databases/visualizer/screenshots/11-cfn-outputs.png | Removed outdated screenshot asset. |
| 09-distributed-file-systems/INSTRUCTOR-GUIDE.md | Removes the instructor guide document from the repo. |
Fix replication line, remove lab09 screenshots showing wrong stack name, remove instructor guide.
Summary by CodeRabbit
Documentation
Style