Skip to content

fix: clean up visualizer SVG, remove wrong screenshots and instructor guide#47

Merged
gamaware merged 1 commit into
mainfrom
fix/lab10-cleanup
Apr 14, 2026
Merged

fix: clean up visualizer SVG, remove wrong screenshots and instructor guide#47
gamaware merged 1 commit into
mainfrom
fix/lab10-cleanup

Conversation

@gamaware
Copy link
Copy Markdown
Owner

@gamaware gamaware commented Apr 14, 2026

Fix replication line, remove lab09 screenshots showing wrong stack name, remove instructor guide.

Summary by CodeRabbit

  • Documentation

    • Removed instructor guide for Lab 09 distributed file systems.
    • Removed embedded setup screenshots from Lab 10 database visualizer documentation while retaining instructional steps.
  • Style

    • Updated database visualizer replication connection display with directional arrow indicator for improved clarity.

… 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
Copilot AI review requested due to automatic review settings April 14, 2026 04:25
@gamaware gamaware merged commit b8a6e53 into main Apr 14, 2026
11 of 15 checks passed
@gamaware gamaware deleted the fix/lab10-cleanup branch April 14, 2026 04:25
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 14, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 3212c359-31cf-433d-934b-19eb2f7b30fa

📥 Commits

Reviewing files that changed from the base of the PR and between 4c64e3f and 55b2a9d.

⛔ Files ignored due to path filters (7)
  • 10-databases/visualizer/screenshots/02-aws-academy-modules.png is excluded by !**/*.png, !**/screenshots/**, !**/*.png
  • 10-databases/visualizer/screenshots/03-learner-lab-toolbar.png is excluded by !**/*.png, !**/screenshots/**, !**/*.png
  • 10-databases/visualizer/screenshots/04-aws-details-panel.png is excluded by !**/*.png, !**/screenshots/**, !**/*.png
  • 10-databases/visualizer/screenshots/06-cfn-create-stack.png is excluded by !**/*.png, !**/screenshots/**, !**/*.png
  • 10-databases/visualizer/screenshots/08-cfn-stack-name.png is excluded by !**/*.png, !**/screenshots/**, !**/*.png
  • 10-databases/visualizer/screenshots/10-cfn-creating.png is excluded by !**/*.png, !**/screenshots/**, !**/*.png
  • 10-databases/visualizer/screenshots/11-cfn-outputs.png is excluded by !**/*.png, !**/screenshots/**, !**/*.png
📒 Files selected for processing (3)
  • 09-distributed-file-systems/INSTRUCTOR-GUIDE.md
  • 10-databases/visualizer/LAB-VISUALIZER.md
  • 10-databases/visualizer/index.html

📝 Walkthrough

Walkthrough

The 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

Cohort / File(s) Summary
Documentation Cleanup
09-distributed-file-systems/INSTRUCTOR-GUIDE.md
Removed entire instructor guide containing lab flow, timing, delivery guidance, demos, and Q&A content.
Visualizer Documentation
10-databases/visualizer/LAB-VISUALIZER.md
Removed embedded screenshot image references from AWS Academy setup instructions; retained instructional text and tightened formatting.
Visualization Enhancement
10-databases/visualizer/index.html
Replaced static dashed replication line with styled arrow; adjusted stroke properties, positioning, and added marker-end attribute for arrowhead rendering.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Poem

🐰 ✨ Out with old guides, in with the clean,
Screenshots fade from the visualization scene,
Arrows now dance where dashes once played,
A sleeker design, both bold and well-made!

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/lab10-cleanup

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

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.

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