Skip to content

Conversation

@iuwqyir
Copy link
Contributor

@iuwqyir iuwqyir commented Apr 10, 2025

[Dashboard] Feature: Add HTTP methods to blueprint paths and reorganize blueprint data

Notes for the reviewer

This PR adds HTTP method information to each API path in the insight blueprints and reorganizes the blueprint data structure. The main changes include:

  1. Added a method field to each path in the MinimalBlueprintSpec type
  2. Updated all blueprint paths with their corresponding HTTP methods
  3. Simplified the blueprint fetching logic in utils.ts
  4. Reorganized the order of blueprints in insightBlueprints.ts
  5. Fixed the name of the decode endpoint from "Unknown" to "Decode logs and transactions​"

How to test

Verify that the blueprints display correctly in the playground with the proper HTTP methods shown for each endpoint.


PR-Codex overview

This PR primarily focuses on enhancing the BlueprintSpec type and modifying the fetchAllBlueprints function to streamline the fetching and mapping of blueprint data, including the addition of new description and openapi properties.

Detailed summary

  • Added description, openapiJson, openapiUrl, and paths to BlueprintSpec.
  • Updated fetchAllBlueprints to directly use fetchBlueprintList and map paths.
  • Modified insightBlueprints to include new blueprints and paths.
  • Removed outdated paths for events and reorganized entries.

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

@vercel vercel bot temporarily deployed to Preview – wallet-ui April 10, 2025 19:22 Inactive
@vercel vercel bot temporarily deployed to Preview – login April 10, 2025 19:22 Inactive
@vercel
Copy link

vercel bot commented Apr 10, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
thirdweb_playground ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 10, 2025 7:34pm
4 Skipped Deployments
Name Status Preview Comments Updated (UTC)
docs-v2 ⬜️ Skipped (Inspect) Apr 10, 2025 7:34pm
login ⬜️ Skipped (Inspect) Apr 10, 2025 7:34pm
thirdweb-www ⬜️ Skipped (Inspect) Apr 10, 2025 7:34pm
wallet-ui ⬜️ Skipped (Inspect) Apr 10, 2025 7:34pm

@vercel vercel bot temporarily deployed to Preview – thirdweb-www April 10, 2025 19:22 Inactive
@vercel vercel bot temporarily deployed to Preview – docs-v2 April 10, 2025 19:22 Inactive
@changeset-bot
Copy link

changeset-bot bot commented Apr 10, 2025

⚠️ No Changeset found

Latest commit: 58eee97

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

@github-actions github-actions bot added the Playground Changes involving the Playground codebase. label Apr 10, 2025
Copy link
Contributor Author

iuwqyir commented Apr 10, 2025


How to use the Graphite Merge Queue

Add either label to this PR to merge it via the merge queue:

  • merge-queue - adds this PR to the back of the merge queue
  • hotfix - for urgent hot fixes, skip the queue and merge this PR next

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

@github-actions
Copy link
Contributor

github-actions bot commented Apr 10, 2025

size-limit report 📦

Path Size Loading time (3g) Running time (snapdragon) Total time
thirdweb (esm) 47.66 KB (0%) 954 ms (0%) 161 ms (+184.25% 🔺) 1.2 s
thirdweb (cjs) 129.94 KB (0%) 2.6 s (0%) 218 ms (+52.47% 🔺) 2.9 s
thirdweb (minimal + tree-shaking) 5.62 KB (0%) 113 ms (0%) 88 ms (+1934.69% 🔺) 200 ms
thirdweb/chains (tree-shaking) 514 B (0%) 11 ms (0%) 42 ms (+2463.94% 🔺) 52 ms
thirdweb/react (minimal + tree-shaking) 19.36 KB (0%) 388 ms (0%) 104 ms (+638.13% 🔺) 491 ms

@iuwqyir iuwqyir force-pushed the 04-10-update_insight_plaground_routes branch from 352977f to 58eee97 Compare April 10, 2025 19:32
@vercel vercel bot temporarily deployed to Preview – thirdweb-www April 10, 2025 19:33 Inactive
@vercel vercel bot temporarily deployed to Preview – docs-v2 April 10, 2025 19:33 Inactive
@vercel vercel bot temporarily deployed to Preview – wallet-ui April 10, 2025 19:33 Inactive
@vercel vercel bot temporarily deployed to Preview – login April 10, 2025 19:33 Inactive
@joaquim-verges joaquim-verges merged commit e9db59e into main Apr 10, 2025
29 of 30 checks passed
@joaquim-verges joaquim-verges deleted the 04-10-update_insight_plaground_routes branch April 10, 2025 19:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Playground Changes involving the Playground codebase.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants