Repository navigation
Conversation
Contributor
|
Preview deployment for your docs. Learn more about Mintlify Previews.
💡 Tip: Enable Automations to automatically generate PRs for you. |
Collaborator
🟡 Heimdall Review Status
|
This branch was successfully deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What changed? Why?
docs/sdks/tokenized-stocks/overview.mdx). It states that the API returns token-level data only, with no wallet positions, cost basis, trade history, or profit and loss.balanceOfUIand the current multiplier for share-equivalent balances. It also notes that cost basis and profit and loss need trade history from every route the app wants to include.This closes Base Knowledge Base gap
gap-20261006-tokenized-stocks-there-per-wallet-portfolio: "Does Coinbase/Base offer (or plan) an API that returns a wallet's tokenized-stock positions, cost basis, and realized/unrealized P&L % across all buy/sell routes, so apps can show portfolio and P&L cards without computing them from raw onchain data?" Gaps log: https://bdocs.cbhq.net/d/base-knowledge-gaps-log. Tracking: DEVREL-859.Notes to reviewers
Primary sources:
docs/openapi/tokenized-equities.jsonon master. The paths are/v1/tokenized-stocks(page_size,page_token),/v1/tokenized-stocks/chains, and/v1/tokenized-stocks/total-supply/{contract_address}. The endpoint table indocs/sdks/tokenized-stocks/overview.mdxlists the same routes.balanceOfUIreturns raw balance × multiplier:docs/build-on-base/integrate-defi/list-tokenized-stocks.mdx(Multipliers section and ERC-8056 helper table).Owner confirmation: the Tokenized Stocks product/API owners should confirm the wording before merge. The change says nothing about the roadmap. It does not say whether a portfolio or P&L endpoint is planned, because no primary source covers that.
This PR does the same job as the tokenized-stocks paragraph in the bundled PR #2093, which has not been merged. Reviewers may want to drop that paragraph from #2093.
How has it been tested?
node scripts/lint-mdx.js --files-from=<changed files>: 1 file checked, 0 errors, 0 warnings.node scripts/validate-docs-structure.js: fails with 2 errors about the/upgrades/denim/eip-8130redirect. The same errors appear on master without this change./specifications/b20/reference/interfaces/ib20-asset/balance-of-uiexists as a file, and/build-on-base/integrate-defi/list-tokenized-stocks#multipliersmatches the### Multipliersheading.Screenshots
N/A (text-only change)
Generated with Toshi