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

Better community treasury account derivation #412

Open
clangenb opened this issue Jan 13, 2025 · 2 comments
Open

Better community treasury account derivation #412

clangenb opened this issue Jan 13, 2025 · 2 comments

Comments

@clangenb
Copy link
Member

I suggest that we do something like: https://github.com/ajuna-network/ajuna-pallets/blob/develop/pallets/ajuna-tournament/src/account.rs

@brenzi
Copy link
Member

brenzi commented Jan 13, 2025

We use an account derivation in faucet too, where we have existing accounts already. What would be the reasoning to change it (everywhere) with possible state migrations?

@clangenb
Copy link
Member Author

The current one in the treasuries pallet seems a bit hacky to me, and the one I suggest makes it obvious (by string) if it is a treasury account, which can be nice for UIs. We could use the same and put the formatted CID in the account id string.

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

No branches or pull requests

2 participants