EarnIntro/EarnAmount/EarnConfirmEstimation #1310
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
editaahn
force-pushed
the
editaahn/keplr-753,keplr-763
branch
from
09280c2 to
ed3125e
Compare
editaahn
changed the title
| return; | ||
| } | ||
|
|
||
| if (currency.coinMinimalDenom === "uusdc") { |
There was a problem hiding this comment.
당장은 문제가 안될 것 같은데 USDC.axl도 coinMinimalDenom이 uusdc 라서 좀 더 엄격한 로직이면 좋을 것 같아요.
| {viewToken.token.currency.coinDenom === "USDC" && | ||
| !viewToken.chainInfo.chainId.includes("eip155:") ? ( |
There was a problem hiding this comment.
나중에 혹시라도 USDC 라는 토큰이 생길수도 있으니까 좀 더 확실한 방법으로 가면 좋을 것 같아요.
익스텐션에서는 ibc 토큰이 IBCCurrency 라는 타입의 형태를 띄게 되는데 그 타입에 originChainId 라는 필드가 있어요. 그 필드를 활용해서 noble 에서 온 usdc 토큰이면 뜨게하도록 로직을 변경해주면 좋을 것 같아요.
ex) viewToken.token.currency.coinMinimalDenom === "uusdc" && (viewToken.token.currency as IBCCurrency)?.originChainId === "noble-1"
There was a problem hiding this comment.
이 코드는 임시로 진입점 만드려고 마지막에 테스트용 커밋 넣어둔거긴 해요! 그러고보니 정환님 델리반님도 개발하면서 테스트하기 용이하게끔 그냥 커밋 머지하고 디자인 픽스 후에 추가 업데이트하면 좋겠군요. coinMinimalDenom은 ibc일 수 있어서 아래 조건으로 반영하면 될 것 같아요
(viewToken.token.currency.coinMinimalDenom === "uusdc" && viewToken.chainInfo.chainId === "noble-1")
|| (viewToken.token.currency as IBCCurrency)?.originChainId === "noble-1"
|
@delivan |
No description provided.