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

Update neutron ecosystem #61

Merged
merged 6 commits into from
Mar 11, 2024
Merged

Update neutron ecosystem #61

merged 6 commits into from
Mar 11, 2024

Conversation

claraexmachina
Copy link
Collaborator

뉴트런 ecosystem 페이지에 아래의 두 dApp 추가.

  • Nolus (기존에 explore 탭에도 있는 것 같아서 이렇게 추가하는 게 맞을지 확인 필요)
  • Eclipse Fi

@blacktoast
Copy link
Collaborator

기존 nolus에서는

  "appSummary": "The Cross-Chain Lease Protocol. Elevate your game with up to 3x equity. Dive into a world of minimized risks and unlock the full potential of your assets.",

인것 같은데요 해당 pr에서는 변경 된 것 같아요
혹시 뉴트론 ecosystem에서는 보여주는 appSummary가 다른걸까요??

그게 아니라면 nolus는 https://github.com/chainapsis/keplr-app-registry/blob/main/apps/nolus/app.json 에서

"appWebsiteUrlsByChainId": {
    "neutron-1": "https://app.nolus.io/"
  }

를 추가 하는게 더 좋을것 같습니다

@claraexmachina
Copy link
Collaborator Author

그럼 혹시 기존 설명을 "Reshaping Money Markets with the first DeFi Lease." 이걸로 바꾸고 밑에

"appWebsiteUrlsByChainId": {
"neutron-1": "https://app.nolus.io/"
}

를 추가해도 될까용? (왜냐면 기존 설명이 어차피 너무 길어서 짤리고 있었어요 🥹) 새로운 설명은 민트스캔의 뉴트런 페이지에 등록되어 있는대로 가져왔습니다!

@blacktoast
Copy link
Collaborator

넵 그러면 괜찮을것 같습니다!

@claraexmachina
Copy link
Collaborator Author

수정했습니다!

@blacktoast blacktoast merged commit 911773b into main Mar 11, 2024
1 check 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
Development

Successfully merging this pull request may close these issues.

2 participants