Skip to content

changeset#8664

Merged
joaquim-verges merged 1 commit intomainfrom
joaquim/change
Feb 5, 2026
Merged

changeset#8664
joaquim-verges merged 1 commit intomainfrom
joaquim/change

Conversation

@joaquim-verges
Copy link
Member

@joaquim-verges joaquim-verges commented Feb 5, 2026


PR-Codex overview

This PR focuses on fixing the SiteLink component to properly handle hashed URL paths, ensuring that navigation works correctly when using fragment identifiers in URLs.

Detailed summary

  • Updated the SiteLink component to manage hashed URL paths effectively.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

Summary by CodeRabbit

  • Bug Fixes
    • Fixed SiteLink component not properly handling hashed URL paths.

@vercel
Copy link

vercel bot commented Feb 5, 2026

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

Project Deployment Actions Updated (UTC)
docs-v2 Ready Ready Preview, Comment Feb 5, 2026 2:08am
nebula Ready Ready Preview, Comment Feb 5, 2026 2:08am
thirdweb_playground Ready Ready Preview, Comment Feb 5, 2026 2:08am
thirdweb-www Error Error Feb 5, 2026 2:08am
wallet-ui Ready Ready Preview, Comment Feb 5, 2026 2:08am

@changeset-bot
Copy link

changeset-bot bot commented Feb 5, 2026

🦋 Changeset detected

Latest commit: 25e01b0

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 4 packages
Name Type
thirdweb Patch
@thirdweb-dev/nebula Patch
@thirdweb-dev/wagmi-adapter Patch
wagmi-inapp Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@joaquim-verges joaquim-verges merged commit b50e304 into main Feb 5, 2026
8 of 14 checks passed
@joaquim-verges joaquim-verges deleted the joaquim/change branch February 5, 2026 01:56
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 5, 2026

Caution

Review failed

The pull request is closed.

Walkthrough

A changelog entry was added to document a patch fix for the "thirdweb" package, specifically addressing an issue where SiteLink was not properly handling hashed URL paths.

Changes

Cohort / File(s) Summary
Changelog Entry
.changeset/honest-trains-behave.md
New changeset file documenting a patch-level fix for SiteLink's handling of hashed URL paths.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch joaquim/change

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

@codecov
Copy link

codecov bot commented Feb 5, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 52.73%. Comparing base (9baa251) to head (25e01b0).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #8664   +/-   ##
=======================================
  Coverage   52.73%   52.73%           
=======================================
  Files         934      934           
  Lines       62975    62975           
  Branches     4137     4137           
=======================================
  Hits        33210    33210           
  Misses      29665    29665           
  Partials      100      100           
Flag Coverage Δ
packages 52.73% <ø> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@joaquim-verges joaquim-verges mentioned this pull request Feb 5, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Feb 5, 2026

size-limit report 📦

Path Size
@thirdweb-dev/nexus (esm) 105.66 KB (0%)
@thirdweb-dev/nexus (cjs) 319.47 KB (0%)

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.

1 participant