feat: add tying-the-masterpoint-knot marketing page #92
Conversation
Add a standalone marketing page at /tying-the-masterpoint-knot with an embedded YouTube video tutorial and step-by-step Comic Sans instructions for tying the double fisherman's knot that inspired the Masterpoint logo. Includes a lightbox effect for the instruction image. 🤖 Generated with Claude Code Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
✅ Deploy Preview for masterpoint ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
WalkthroughAdds a new Markdown content file that provides a tutorial-style post about tying the Masterpoint (double fisherman's) knot, including front matter metadata and body content with an embedded YouTube video, a lightbox image link, and a contact section. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ Finishing touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Gowiem
changed the title
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (1)
static/img/marketing/tying-masterpoint-knot-instructions.pngis excluded by!**/*.png
📒 Files selected for processing (1)
content/tying-the-masterpoint-knot.md(1 hunks)
🧰 Additional context used
📓 Path-based instructions (1)
content/**/*.md
📄 CodeRabbit inference engine (CLAUDE.md)
content/**/*.md: Place all site content in Markdown files under /content/
Use absolute URLs in content (paths should start with "/", e.g., /blog/ not blog/)
Reference images in content using the /img/ path (which maps to /static/img/)
Use front matter draft: true to hide in-progress content from production
Use provided Hugo shortcodes (button, buttonout, form, testimonials, faqs, team, services, process, supports, client-logos) in content instead of duplicating HTML
Files:
content/tying-the-masterpoint-knot.md
🪛 GitHub Check: Trunk Check
content/tying-the-masterpoint-knot.md
[failure] 1-1: prettier
Incorrect formatting, autoformat by running 'trunk fmt'
🪛 LanguageTool
content/tying-the-masterpoint-knot.md
[typographical] ~25-~25: This sign-off should end with a comma when followed by your name.
Context: ...ng the double fisherman's knot" >}} ## Thanks Thanks for checking out this funny mar...
(EN_SIGN_OFF_WITHOUT_COMMA)
[style] ~27-~27: Using many exclamation marks might seem excessive (in this case: 3 exclamation marks for a text that’s 901 characters long)
Context: ...a photo of your knot, and let's catch up!
(EN_EXCESSIVE_EXCLAMATION)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)
- GitHub Check: Redirect rules - masterpoint
- GitHub Check: Header rules - masterpoint
- GitHub Check: Pages changed - masterpoint
🔇 Additional comments (3)
content/tying-the-masterpoint-knot.md (3)
1-10: Front matter and overall structure look good.The metadata is appropriately configured for a published marketing page, with clear description, author attribution, and correct slug. The draft status is set to false, which aligns with the PR's marketing campaign purpose.
23-23: Image reference correctly implemented.The lightboximg shortcode properly references the image using an absolute
/img/path as required by the guidelines.
15-17: > Likely an incorrect or invalid review comment.
Apply automatic formatting from trunk fmt. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
There was a problem hiding this comment.
Actionable comments posted: 0
♻️ Duplicate comments (1)
content/tying-the-masterpoint-knot.md (1)
1-26: Run formatter (Trunk/Prettier) to clear the existing CI complaint.Prior Trunk Check indicates Prettier formatting issues; run
trunk fmtand re-commit the formatted Markdown. Based on learnings, Trunk is the expected workflow.
🧹 Nitpick comments (1)
content/tying-the-masterpoint-knot.md (1)
14-16: Prefer a Hugo shortcode for the YouTube embed (and consideryoutube-nocookie+ lazy-load).Guidelines ask to avoid duplicating raw HTML in content when shortcodes exist. If the repo has a YouTube shortcode, use it here; also consider privacy-enhanced embeds (
youtube-nocookie.com) andloading="lazy"(or shortcode equivalent) to reduce tracking and improve performance.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (1)
static/img/marketing/tying-masterpoint-knot-instructions.pngis excluded by!**/*.png
📒 Files selected for processing (1)
content/tying-the-masterpoint-knot.md(1 hunks)
🧰 Additional context used
📓 Path-based instructions (1)
content/**/*.md
📄 CodeRabbit inference engine (CLAUDE.md)
content/**/*.md: Place all site content in Markdown files under /content/
Use absolute URLs in content (paths should start with "/", e.g., /blog/ not blog/)
Reference images in content using the /img/ path (which maps to /static/img/)
Use front matter draft: true to hide in-progress content from production
Use provided Hugo shortcodes (button, buttonout, form, testimonials, faqs, team, services, process, supports, client-logos) in content instead of duplicating HTML
Files:
content/tying-the-masterpoint-knot.md
🧠 Learnings (1)
📚 Learning: 2025-09-19T19:21:07.260Z
Learnt from: CR
Repo: masterpointio/masterpoint.io PR: 0
File: CLAUDE.md:0-0
Timestamp: 2025-09-19T19:21:07.260Z
Learning: Run Trunk for linting and formatting (trunk check, trunk fmt)
Applied to files:
content/tying-the-masterpoint-knot.md
🪛 LanguageTool
content/tying-the-masterpoint-knot.md
[typographical] ~24-~24: This sign-off should end with a comma when followed by your name.
Context: ...ng the double fisherman's knot" >}} ## Thanks Thanks for checking out this funny mar...
(EN_SIGN_OFF_WITHOUT_COMMA)
[style] ~26-~26: Using many exclamation marks might seem excessive (in this case: 3 exclamation marks for a text that’s 900 characters long)
Context: ...a photo of your knot, and let's catch up!
(EN_EXCESSIVE_EXCLAMATION)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)
- GitHub Check: Redirect rules - masterpoint
- GitHub Check: Header rules - masterpoint
- GitHub Check: Pages changed - masterpoint
🔇 Additional comments (2)
content/tying-the-masterpoint-knot.md (2)
1-10: Confirm front-matter flags (visible) and publish intent (draft: false).
draft: falsewill publish immediately; make sure that’s intended for the 2025-12-12 campaign launch. Also please confirmvisible: trueis a supported front-matter param in this Hugo/theme setup (otherwise it becomes misleading config).
22-22: Verify the referenced image asset exists at the expected static path.
/img/...maps to/static/img/...; please confirmstatic/img/marketing/tying-masterpoint-knot-instructions.pngexists (and thatlightboximgis the correct shortcode name/args).
oycyc
left a comment
oycyc
left a comment
There was a problem hiding this comment.
This is cool! If you bring some next time I wanna try this activity out!
what
/tying-the-masterpoint-knotwith an embedded YouTube video tutorialwhy
references
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.