Skip to content

Add Creditcoin3 support to Generic Ledger App #1985

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

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

salina-dev
Copy link

@salina-dev salina-dev commented Apr 30, 2025

Add Creditcoin3 support to Generic Ledger App

@salina-dev salina-dev marked this pull request as draft April 30, 2025 10:53
@valentinfernandez1 valentinfernandez1 self-requested a review May 2, 2025 16:17
@valentinfernandez1
Copy link
Contributor

Are there any updated about this Draft PR?

@salina-dev salina-dev marked this pull request as ready for review May 7, 2025 11:04
@DylanVerstraete
Copy link

@valentinfernandez1 please check 👍 😄

Copy link
Member

@TarikGul TarikGul left a comment

Choose a reason for hiding this comment

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

🚀

Copy link
Member

@TarikGul TarikGul left a comment

Choose a reason for hiding this comment

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

Apologies for the early approval - believe inside of hw-ledger/src/defaults.ts creditcoin3 should be under genericLedgerApps instead

@TarikGul
Copy link
Member

Just needs a yarn lint

@beqaabu
Copy link

beqaabu commented Jul 17, 2025

@TarikGul, we dont want to enable ledger support on mainnet before testing it on our testnet first, but the note in genesis.ts says:

// IMPORTANT: Apart from the test relays, this list is limited to live parachains
// and live production networks. It does not and should not contain any testnets,
// either stand-alone or connected to test relays such as Westend/Rococo

see the link: https://github.com/gluwa/common/blob/ab1d3c0b07bd79cf8aab937ad22e5df4b68e78ba/packages/networks/src/defaults/genesis.ts#L11-L13
We have made necessary changes on the protocol level to support ledger, but without the changes in this pr our ledger support page looks like this
image
instead of this
image
As you can see, there is no option to attach ledger, which is expected. Could you please suggest how we could achieve what we want without violating the note given above?

@beqaabu
Copy link

beqaabu commented Jul 21, 2025

Hi @TarikGul @valentinfernandez1, could you please take a look?

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.

5 participants