Skip to content

Conversation

@rafaelcr
Copy link
Collaborator

@rafaelcr rafaelcr commented Mar 14, 2023

This PR makes the FT response be backwards-compatible to the current Stacks API FT response, so we can set up a simple redirect/rewrite from there to this new API immediately.

Example old Stacks API FT response:

{
  "token_uri": "https://heystack.xyz/token-metadata.json",
  "name": "Heystack",
  "description": "Heystack is a SIP-010-compliant fungible token on the Stacks Blockchain, used on the Heystack app",
  "image_uri": "https://heystack.xyz/assets/Stacks128w.png",
  "image_canonical_uri": "https://heystack.xyz/assets/Stacks128w.png",
  "tx_id": "0xef2ac1126e16f46843228b1dk4830e19eb7599129e4jf392cab9e65ae83a45c0",
  "sender_address": "ST399W7Z9WS0GMSNQGJGME5JAENKN56D65VGMGKGA",
  "symbol": "HEY",
  "decimals": 5
}

New FT response: https://token-metadata-csgxi67fv-blockstack.vercel.app/#tag/Tokens/operation/getFtMetadata

@rafaelcr rafaelcr requested a review from zone117x March 14, 2023 15:36
@rafaelcr rafaelcr temporarily deployed to Preview March 14, 2023 15:36 — with GitHub Actions Inactive
@github-actions
Copy link

Vercel deployment URL: https://token-metadata-csgxi67fv-blockstack.vercel.app 🚀

Copy link
Contributor

@zone117x zone117x left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@rafaelcr rafaelcr merged commit 3ca57a0 into master Mar 14, 2023
@rafaelcr rafaelcr deleted the fix/legacy-ft-response branch March 14, 2023 15:54
blockstack-devops pushed a commit that referenced this pull request Mar 14, 2023
## [0.1.1](v0.1.0...v0.1.1) (2023-03-14)

### Bug Fixes

* return FT metadata in a backwards compatible way ([#130](#130)) ([3ca57a0](3ca57a0))
@blockstack-devops
Copy link
Contributor

🎉 This PR is included in version 0.1.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants