-
Notifications
You must be signed in to change notification settings - Fork 542
Add Insight Usage Docs for LLMs & Agents #5880
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
You must have Developer access to commit code to thirdweb on Vercel. If you contact an administrator and receive Developer access, commit again to see your changes. Learn more: https://vercel.com/docs/accounts/team-members-and-roles/access-roles#team-level-roles |
How to use the Graphite Merge QueueAdd either label to this PR to merge it via the merge queue:
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. |
icon: "thirdweb", | ||
}, | ||
title: "thirdweb Pay - Onramp Providers | thirdweb", | ||
description: ".", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The metadata description
field contains only a period. Consider replacing it with a descriptive summary that explains the purpose of this page, such as: "Learn about the different onramp providers available in thirdweb Pay and their supported countries"
Spotted by Graphite Reviewer
Is this helpful? React 👍 or 👎 to let us know.
1. **Get All Events** | ||
|
||
```typescript | ||
GET / v1 / events; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There appears to be unnecessary whitespace in the API endpoint specification. The correct format should be GET /v1/events
without spaces between the path segments.
Spotted by Graphite Reviewer
Is this helpful? React 👍 or 👎 to let us know.
size-limit report 📦
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #5880 +/- ##
=======================================
Coverage 55.19% 55.19%
=======================================
Files 1123 1123
Lines 59622 59622
Branches 5029 5029
=======================================
Hits 32906 32906
Misses 25996 25996
Partials 720 720
*This pull request uses carry forward flags. Click here to find out more. |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like this is causing the build to fail
This PR has been inactive for 7 days. It is now marked as stale and will be closed in 2 days if no further activity occurs. |
title: "[SDK/Dashboard/Portal] Feature/Fix: Concise title for the changes"
If you did not copy the branch name from Linear, paste the issue tag here (format is TEAM-0000):
Notes for the reviewer
Anything important to call out? Be sure to also clarify these in your comments.
How to test
Unit tests, playground, etc.
PR-Codex overview
This PR enhances the documentation and functionality of the
thirdweb Pay
andthirdweb Insight
components by adding new sections for onramp providers, preferred providers, and improving API usage examples for agents and LLMs.Detailed summary
Onramp Providers
section inpage.mdx
with provider details and links.Preferred Provider
option inPayEmbed
andConnectButton
examples.Get Started
guide ininsight
with improved API usage examples.For AI Agents & LLMs
documentation with API usage instructions.