-
Notifications
You must be signed in to change notification settings - Fork 514
Version Packages #6728
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
Version Packages #6728
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
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. |
size-limit report 📦
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #6728 +/- ##
=======================================
Coverage 55.16% 55.16%
=======================================
Files 896 896
Lines 56910 56910
Branches 3940 3938 -2
=======================================
Hits 31396 31396
Misses 25419 25419
Partials 95 95
🚀 New features to boost your workflow:
|
2f5bec1
to
cd1d23c
Compare
cd1d23c
to
41bfa08
Compare
41bfa08
to
1e74b5d
Compare
87f870d
to
55ec12b
Compare
55ec12b
to
b797db0
Compare
b797db0
to
0405f8c
Compare
0405f8c
to
7a71f49
Compare
|
||
- [#6741](https://github.com/thirdweb-dev/js/pull/6741) [`8d4d991`](https://github.com/thirdweb-dev/js/commit/8d4d991e5710cbed668a52522686cc2dce4e4790) Thanks [@joaquim-verges](https://github.com/joaquim-verges)! - Use insight for erc821/getNFT, erc721/getNFTs and erc721/getOwnedNFTs | ||
|
||
Standard ERC721 getNFT, getNFTs and getOwnedNFTs now use insight, our in house indexer by default. If indexer is not availbale, will fallback to RPC. |
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's a typo in the documentation: "availbale" should be corrected to "available" in the sentence describing the fallback behavior. This appears in the changelog entry for the insight
integration with ERC721 methods.
Standard ERC721 getNFT, getNFTs and getOwnedNFTs now use insight, our in house indexer by default. If indexer is not availbale, will fallback to RPC. | |
Standard ERC721 getNFT, getNFTs and getOwnedNFTs now use insight, our in house indexer by default. If indexer is not available, will fallback to RPC. |
Spotted by Diamond
Is this helpful? React 👍 or 👎 to let us know.
This PR was opened by the Changesets release GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to main, this PR will be updated.
Releases
@thirdweb-dev/insight@1.0.0
Major Changes
#6706
185d2f3
Thanks @joaquim-verges! - Initial release of dedicated insight TS sdkThis package is a thin openAPI wrapper for insight, our in-house indexer.
Configuration
Example Usage
thirdweb@5.95.0
Minor Changes
#6706
185d2f3
Thanks @joaquim-verges! - Expose getOwnedTokens, getOwnedNFTs and getTransaction functionsYou can now use Insight, our in-house indexer directly from the SDK with a simple API:
Get Owned ERC20 tokens
Get Owned NFTs (ERC721 and ERC1155)
Get Transactions for a given wallet address
All functions come with extra query filters for more granular queries, refer to the documentation for more details.
Patch Changes
#6760
7ecfcb9
Thanks @joaquim-verges! - Add thirdweb branding to PayEmbed#6753
4cf15a2
Thanks @joaquim-verges! - Use insight for 1155 getNFTs, getOwnedNFTs and getNFT#6752
23d3757
Thanks @joaquim-verges! - Simplify RPC request handling#6741
8d4d991
Thanks @joaquim-verges! - Use insight for erc821/getNFT, erc721/getNFTs and erc721/getOwnedNFTsStandard ERC721 getNFT, getNFTs and getOwnedNFTs now use insight, our in house indexer by default. If indexer is not availbale, will fallback to RPC.
You can also use the indexer directly using the Insight API:
for an entire collection
or for a single NFT
#6683
a3e7300
Thanks @kumaryash90! - Get indexed events fromgetContractEvents
You can now automatically query indexed events on supported chains when calling getContractEvents
This method falls back to RPC eth_getLogs if the indexer is not available.
You can also use the dedicated indexer function via the Insight export
#6732
a45c558
Thanks @gregfromstl! - Improve pay error messages#6758
62ce05e
Thanks @joaquim-verges! - Better error message for getUser#6734
2a7df33
Thanks @joaquim-verges! - Better error messages for failed requestsUpdated dependencies [
185d2f3
]:@thirdweb-dev/service-utils@0.9.2
Patch Changes
ec099a7
Thanks @arcoraven! - Update rate limit to sliding window@thirdweb-dev/wagmi-adapter@0.2.56
PR-Codex overview
This PR focuses on updating package versions, adding new features, and improving error handling across several
@thirdweb-dev
packages. It also includes the removal of outdated changelogs and introduces the@thirdweb-dev/insight
package with its initial release.Detailed summary
.changeset
files.@thirdweb-dev/service-utils
version from0.9.1
to0.9.2
.@thirdweb-dev/wagmi-adapter
version from0.2.55
to0.2.56
.@thirdweb-dev/insight
version from0.0.1
to1.0.0
, introducing its initial release.@thirdweb-dev/insight
package.thirdweb
version from5.94.2
to5.95.0
, exposing new functions and improving error messages.