Skip to content
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

[ENG-1528] Use assets endpoint instead of chain registry #15

Merged
merged 8 commits into from
Jul 28, 2023

Conversation

thal0x
Copy link
Member

@thal0x thal0x commented Jul 27, 2023

No description provided.

@thal0x thal0x self-assigned this Jul 27, 2023
@vercel
Copy link

vercel bot commented Jul 27, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
ibc-dot-fun ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 28, 2023 0:07am

@@ -0,0 +1,29 @@
import { Asset, AssetWithMetadata } from "./types";

export function assetHasMetadata(asset: Asset) {
Copy link
Member Author

Choose a reason for hiding this comment

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

Currently the API returns a lot of assets that are not necessarily "assets", like osmosis pool tokens (see: https://api.skip.money/v1/fungible/assets?chain_id=osmosis-1&native_only=true).

Do we want this? (will post in slack)

Either way for now this seems to be the best way to filter them out of the UI.

@thal0x thal0x changed the title Use assets endpoint instead of chain registry [ENG-1528] Use assets endpoint instead of chain registry Jul 27, 2023
thal0x and others added 4 commits July 27, 2023 12:28
@thal0x thal0x merged commit 1b0aaf2 into create-skip-api-client Jul 28, 2023
2 checks passed
@thal0x thal0x deleted the use-assets-endpoint branch July 28, 2023 00:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants