Skip to content

Conversation

@MananTank
Copy link
Contributor

@MananTank MananTank commented Feb 3, 2026

Request

Before

image

After

image

Response

Before

image

After

image

PR-Codex overview

This PR focuses on enhancing the user interface and experience of the API documentation by updating styles and adding helpful content. It includes improvements to loading states, component styling, and example presentations.

Detailed summary

  • Added a note for ecosystem wallets in page.mdx.
  • Updated loading fallback in OpenApiEndpoint.tsx with a spinner.
  • Changed SelectTrigger styles in RequestExample.tsx for better aesthetics.
  • Enhanced code display styles in DynamicRequestExample.tsx.
  • Modified parameter section styles for better readability.
  • Improved the presentation of response examples in ApiEndpoint.tsx.

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

Summary by CodeRabbit

  • Documentation

    • Added guidance to include ecosystem headers when making wallet API requests.
  • Style & UI

    • Improved API docs visual consistency: card backgrounds, spacing, badges, and code block presentation.
    • Replaced text loading indicator with a centered animated spinner.
    • Refined dropdown trigger appearance and sizing for better visual hierarchy.

@MananTank MananTank requested review from a team as code owners February 3, 2026 14:56
@changeset-bot
Copy link

changeset-bot bot commented Feb 3, 2026

⚠️ No Changeset found

Latest commit: 4022e33

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

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

@vercel
Copy link

vercel bot commented Feb 3, 2026

@MananTank is attempting to deploy a commit to the thirdweb Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions github-actions bot added the Portal Involves changes to the Portal (docs) codebase. label Feb 3, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 3, 2026

Walkthrough

Styling and UI refinements were applied to API documentation components (Request/Response cards, badges, tabs, parameter displays, loading state, select triggers). A single-line guidance note was added to the Get Single User docs advising inclusion of ecosystem headers for ecosystem wallets.

Changes

Cohort / File(s) Summary
Documentation Content
apps/portal/src/app/wallets/get-users/page.mdx
Added a guidance sentence: "For ecosystem wallets, include the ecosystem headers in your requests."
APIEndpointMeta Styling
apps/portal/src/components/Document/APIEndpointMeta/ApiEndpoint.tsx, .../DynamicRequestExample.tsx, .../OpenApiEndpoint.tsx, .../RequestExample.tsx
Visual updates only: applied bg-card to containers, adjusted code block/card styling, refined badges and status indicators, spacing/padding/tabs tweaks, replaced loading text with a Spinner, and changed SelectTrigger radii/widths.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 Pre-merge checks | ✅ 1 | ❌ 2
❌ Failed checks (2 warnings)
Check name Status Explanation Resolution
Description check ⚠️ Warning Description does not follow the required template; it lacks the issue tag, Notes for the reviewer, and How to test sections. Populate the PR description template by adding the issue tag, Notes for the reviewer, and How to test sections with clear testing instructions.
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 (1 passed)
Check name Status Explanation
Title check ✅ Passed Title concisely describes UI tweaks to the OpenApiEndpoint card component, matching a core aspect of the changeset.

✏️ 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

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.

❤️ Share

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

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

Labels

Portal Involves changes to the Portal (docs) codebase.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant