Skip to content

fix: extension share link#48

Merged
scanash00 merged 1 commit into
paddinglabs:mainfrom
hacdias:fix/extension-share-link
Feb 16, 2026
Merged

fix: extension share link#48
scanash00 merged 1 commit into
paddinglabs:mainfrom
hacdias:fix/extension-share-link

Conversation

@hacdias
Copy link
Copy Markdown
Contributor

@hacdias hacdias commented Feb 16, 2026

Smol fix.

Summary by CodeRabbit

  • Bug Fixes
    • Updated the share functionality endpoint to resolve issues with the shared page URL generation.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Feb 16, 2026

No actionable comments were generated in the recent review. 🎉


📝 Walkthrough

Walkthrough

The share URL endpoint in the popup component is updated from /site/{encodedUrl} to /url/{encodedUrl}. This single-line change modifies the API route called when users trigger the share functionality, with no alterations to logic or control flow.

Changes

Cohort / File(s) Summary
Share URL Endpoint Update
extension/src/components/popup/App.tsx
Modified share button endpoint path from /site/ to /url/ in the API URL construction.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A path once traveled through the site,
Now routes through URLs instead—so bright!
One little change, a hop away,
Sharing flows in a newer way! 🎉

🚥 Pre-merge checks | ✅ 3 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'fix: extension share link' directly corresponds to the changeset, which modifies the share URL endpoint in the extension popup component.
Merge Conflict Detection ✅ Passed ✅ No merge conflicts detected when merging into main

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Tip

Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord.


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

@scanash00 scanash00 merged commit ccc7151 into paddinglabs:main Feb 16, 2026
1 check passed
@hacdias hacdias deleted the fix/extension-share-link branch February 16, 2026 12:25
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