Skip to content

fix: API quickstart key, Assets reserve stat, Addresses pagination and received amounts - #597

Merged
Macnelson9 merged 4 commits into
Octo-Protocol-org:devfrom
jhaydeeee-web:fix/jhaydeeee-web-api-assets-addresses-issues-500-503
Oct 19, 2026
Merged

Macnelson9 merged 4 commits into
Octo-Protocol-org:devfrom
jhaydeeee-web:fix/jhaydeeee-web-api-assets-addresses-issues-500-503

Conversation

@jhaydeeee-web

Copy link
Copy Markdown
Contributor

Closes #500
Closes #501
Closes #502
Closes #503

Summary

6e078e0: fix: use the API key in the API page quickstart (#500)

The quickstart in wallets/[id]/api/page.tsx showed a login token placeholder and re-read the env var in a local apiBase(). It now uses Bearer <YOUR_API_KEY>, imports the shared API_URL from @/lib/api, and renders through the docs Code component, which adds a copy button.

c6c4ae3: fix: drop the hardcoded Reserve stat from the Assets page (#501)

wallets/[id]/assets/page.tsx always showed "Reserve 1.0 XLM", which ignores the 0.5 XLM per trustline. SpendableReservedBreakdown already shows the real reserve, so the stat is removed (grid is now 3 columns) and the No. of assets expression is simplified to balances.length.

f7c2f73: fix: paginate the Addresses page and label stats per page (#502)

The page loaded with listAddresses but refreshed with listRecentAddresses, so row counts differed, and there was no way to reach older addresses. Load and refresh now share one load built on listAddressesPage, with cursor pagination and the Pagination component as on the Transactions page. The "Addresses on this page" stat is labelled to match.

e551e7d: fix: stop labelling mixed-asset received totals as XLM (#503)

received_stroops can include deposits in more than one asset, but it was summed and labelled XLM, and zero rendered as "$0". The cross-address "Total received" stat and sumStroops use are removed, rows show the plain amount ("0" for zero), and the detail modal drops the XLM unit and notes that per-asset amounts are on the Transactions page. Per-asset totals would need a backend change.

@vercel

vercel Bot commented Oct 18, 2026

Copy link
Copy Markdown

@jhaydeeee-web is attempting to deploy a commit to the paps' projects Team on Vercel.

A member of the Team first needs to authorize it.

@drips-wave

drips-wave Bot commented Oct 18, 2026

Copy link
Copy Markdown

@jhaydeeee-web Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@Macnelson9
Macnelson9 merged commit a3853ff into Octo-Protocol-org:dev Oct 19, 2026
0 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants