Dustin/engine playground #5507
Dustin/engine playground #5507
Conversation
DustinTurska
requested review from
gregfromstl,
jnsdls and
joaquim-verges
as code owners
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
Your org has enabled the Graphite merge queue for merging into mainAdd the label “merge-queue” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge. Or use the label “hotfix” to add to the merge queue as a hot fix. You must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link. |
github-actions
bot
added
the
Playground
size-limit report 📦
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #5507 +/- ##
=======================================
Coverage 46.32% 46.32%
=======================================
Files 1082 1082
Lines 58672 58672
Branches 3950 3950
=======================================
Hits 27178 27178
Misses 30811 30811
Partials 683 683
*This pull request uses carry forward flags. Click here to find out more. |
|
New and removed dependencies detected. Learn more about Socket for GitHub ↗︎
|
| const CHAIN_ID = "84532"; | ||
| const BACKEND_WALLET_ADDRESS = process.env.ENGINE_BACKEND_WALLET as string; | ||
|
|
||
| console.log("Environment Variables:"); |
There was a problem hiding this comment.
Removed this file entirely, it is unused, removed logs from the rest as well
| return NextResponse.json({ error: "Empty data array" }, { status: 400 }); | ||
| } | ||
|
|
||
| console.log(`Attempting to mint batch to ${data.length} receivers`); |
| name: "Webhooks", | ||
| href: "/engine/webhooks", | ||
| }, | ||
| // { |
There was a problem hiding this comment.
Remove these if we don't have them yet
| <div className="flex w-full flex-col gap-8 md:flex-row"> | ||
| {/* Form Section */} | ||
| <div className="flex flex-col items-center justify-center"> | ||
| <ConnectButton client={THIRDWEB_CLIENT} chain={baseSepolia} /> |
There was a problem hiding this comment.
Formatting on this button looks weird when no connected
| // Keeping OP here for consistency. Will be adding a component for Optimism soon. | ||
| const getExplorerUrl = () => { | ||
| switch (result.network) { | ||
| case "Base Sep": |
There was a problem hiding this comment.
Nit: probably enough room to spell out "Sepolia"
There was a problem hiding this comment.
Fixed, good call out
DustinTurska
added
the
merge-queue
Merge activity
You must have a Graphite account in order to use the merge queue. Create an account and try again using this link
|
graphite-app
bot
removed
the
merge-queue
DustinTurska
added
the
merge-queue
joaquim-verges
left a comment
joaquim-verges
left a comment
There was a problem hiding this comment.
What's missing from these is the "how" of these engine demos. How do i build this for myself? was expecting to see some code snippets, or at least some high level endpoints that get called in those demos
gregfromstl
force-pushed
the
dustin/engine_playground
branch
from
9051de5 to
9edbe1c
Compare
## Problem solved FIxes [DASH-495](https://linear.app/thirdweb/issue/DASH-495/engine-playground) Short description of the bug fixed or feature added Added Engine Section to playground -Airdrop -Mint -Webhooks Updated AppSidebar `expanded: false` to show `otherLinks` section - previously this was unseen by users and there was no scroll option Create EngineAPIHeader for Engine CTA and utm link updates Slack [thread](https://thirdwebdev.slack.com/archives/C07U1LM6S2W/p1731687004602369) for initiative <!-- start pr-codex --> --- ## PR-Codex overview This PR introduces new features and updates to the `playground-web` application, including new API routes for minting and airdropping tokens, enhancements to the UI components, and the addition of new environment variables for build processes. ### Detailed summary - Added environment variables for `ENGINE_ACCESS_TOKEN`, `ENGINE_BACKEND_WALLET`, and `ENGINE_URL`. - Introduced new dependencies: `@thirdweb-dev/engine` and `timeago.js`. - Updated `navLinks.ts` to collapse sections and added a new `Engine` section. - Created new pages for `Webhooks`, `Airdrop`, and `Minting` with respective UI components. - Implemented API routes for transaction status, minting, and airdropping. - Enhanced the `EngineAPIHeader` component to display dynamic content. - Added polling functionality for transaction status updates in the `Sponsorship` and `AirdropERC20` components. - Introduced pagination for transaction results in `ClaimTransactionResults` components. > The following files were skipped due to too many changes: `pnpm-lock.yaml` > ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}` <!-- end pr-codex -->
DustinTurska
force-pushed
the
dustin/engine_playground
branch
from
9edbe1c to
149ee05
Compare
Problem solved
FIxes DASH-495
Short description of the bug fixed or feature added
Added Engine Section to playground
-Airdrop
-Mint
-Webhooks
Updated AppSidebar
expanded: falseto showotherLinkssection - previously this was unseen by users and there was no scroll optionCreate EngineAPIHeader for Engine CTA and utm link updates
Slack thread for initiative
PR-Codex overview
This PR introduces several enhancements and new features to the
playground-webapplication, including new API routes for transaction handling, updates to the navigation structure, and the addition of several UI components for airdrop and minting functionalities.Detailed summary
turbo.json.package.jsonwith new dependencies:@thirdweb-dev/engineandtimeago.js.navLinks.tsto collapse certain sections and added an "Engine" section.webhooks,airdrop, andminting.Webhook,Sponsorship,AirdropERC20, andERC1155MintTo.