Skip to content

FEE-823 Make BRAT repository URLs copyable on installation docs page#1122

Merged
mdroidian merged 2 commits into
mainfrom
update-brat-url-copyable
Jul 22, 2026
Merged

FEE-823 Make BRAT repository URLs copyable on installation docs page#1122
mdroidian merged 2 commits into
mainfrom
update-brat-url-copyable

Conversation

@jsmorabito

@jsmorabito jsmorabito commented Jun 10, 2026 •
edited by devin-ai-integration Bot
Loading

Copy link
Copy Markdown
Collaborator

Summary

  • Wraps both BRAT repository URLs in fenced code blocks so Nextra renders a copy button on hover
  • Enables defaultShowCopyCode: true in the Nextra config (applies site-wide to all docs code blocks)
  • Updates step text to "Enter the repository URL below and choose 'Latest version'"

Test plan

  • Visit /docs/obsidian/welcome/installation and hover over both repository URL code blocks — copy button should appear
  • Verify steps 5 and 6 still render correctly after each code block
  • Spot-check other docs pages to confirm copy buttons appear on existing code blocks

🤖 Generated with Claude Code

Open in Devin Review

@vercel

vercel Bot commented Jun 10, 2026

Copy link
Copy Markdown

Deployment failed with the following error:

You don't have permission to create a Preview Deployment for this Vercel project: discourse-graph.

View Documentation: https://vercel.com/docs/accounts/team-members-and-roles

@supabase

supabase Bot commented Jun 10, 2026

Copy link
Copy Markdown

This pull request has been ignored for the connected project zytfjzqyijgagqxrzbmz because there are no changes detected in packages/database/supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.

Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

@jsmorabito jsmorabito changed the title Make BRAT repository URLs copyable on installation docs page FEE-823 Make BRAT repository URLs copyable on installation docs page Jun 10, 2026
@linear-code

linear-code Bot commented Jun 10, 2026

Copy link
Copy Markdown

FEE-823

devin-ai-integration[bot]
devin-ai-integration Bot reviewed Jun 10, 2026
View reviewed changes

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Devin Review found 2 potential issues.

View 1 additional finding in Devin Review.

Open in Devin Review

Comment thread apps/website/app/(home)/page.tsx Outdated
Comment thread apps/website/content/obsidian/welcome/installation.md
@vercel

vercel Bot commented Jul 21, 2026 •
edited
Loading

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
discourse-graph Ready Ready Preview, Comment Jul 22, 2026 2:34pm

Request Review

@jsmorabito
jsmorabito requested a review from mdroidian July 21, 2026 16:02
@jsmorabito

jsmorabito commented Jul 21, 2026

Copy link
Copy Markdown
Collaborator Author

@mdroidian sorry to bother you with a small docs thing, but could you take a look at this PR and let me know whats left for me to do before we can merge it? also asking from a meta/process standpoint so I can learn how to do this with enough confidence (from myself and you) to be able to merge a docs change on my own without needing to tag you (getting review help from Karola as needed for content)

@mdroidian
mdroidian force-pushed the update-brat-url-copyable branch from d606b44 to 1b62986 Compare July 22, 2026 14:32
jsmorabito and others added 2 commits July 22, 2026 08:32
@jsmorabito @claude @mdroidian
Make BRAT repository URLs copyable on installation docs page
634cd59 
- Wrap both repo URLs in fenced code blocks so Nextra renders a copy button
- Enable defaultShowCopyCode site-wide in nextra config
- Update step text to "Enter the repository URL below and choose Latest version"

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@jsmorabito @claude @mdroidian
Fix list items collapsed by prettier after code block insertion
1b62986 
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@mdroidian

mdroidian commented Jul 22, 2026

Copy link
Copy Markdown
Member

@mdroidian sorry to bother you with a small docs thing, but could you take a look at this PR and let me know whats left for me to do before we can merge it? also asking from a meta/process standpoint so I can learn how to do this with enough confidence (from myself and you) to be able to merge a docs change on my own without needing to tag you (getting review help from Karola as needed for content)

Looks like it was on an older version of main, so there were some additional unwanted changes and a merge conflict. I rebased off main and will double check after vercel builds the website again

mdroidian
mdroidian approved these changes Jul 22, 2026
View reviewed changes

@mdroidian mdroidian left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

👍

@mdroidian
mdroidian merged commit d73cada into main Jul 22, 2026
9 checks passed
@mdroidian
mdroidian deleted the update-brat-url-copyable branch July 22, 2026 14:36
@jsmorabito Linear Code

jsmorabito commented Jul 22, 2026

Copy link
Copy Markdown
Collaborator Author

sweet, thank you Michael!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@devin-ai-integration devin-ai-integration[bot] devin-ai-integration[bot] left review comments
@mdroidian mdroidian mdroidian approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jsmorabito @mdroidian