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

Add CosmosKit integration #50

Merged
merged 23 commits into from
Dec 19, 2023
Merged

Add CosmosKit integration #50

merged 23 commits into from
Dec 19, 2023

Conversation

Redm4x
Copy link
Contributor

@Redm4x Redm4x commented Nov 10, 2023

  • Switch to CosmosKit for wallet management
  • Added support for Cosmostation + Cosmostation Mobile wallets
  • Added support for Keplr Mobile wallet
  • Added bundle-analyzer to track bundle size

@Redm4x Redm4x requested a review from baktun14 November 10, 2023 19:35
@Redm4x Redm4x linked an issue Nov 13, 2023 that may be closed by this pull request
9 tasks
@Redm4x Redm4x force-pushed the features/cosmoskit-integration branch from 88c8ff8 to a6823a5 Compare November 21, 2023 15:30
@schnetzlerjoe
Copy link

schnetzlerjoe commented Dec 13, 2023

In this commit (https://github.com/schnetzlerjoe/cloudmos/tree/features/cosmoskit-integration) I add one line of code to this PR to add Metamask support. We have been working the past week on adding Metamask to Cloudmos using the old wallet context. This adds it to the new context provider as well whenever you guys merge this!

@baktun14
Copy link
Contributor

@schnetzlerjoe This is great thanks! cc @anilmurty

@Redm4x Redm4x marked this pull request as draft December 15, 2023 19:02
Comment on lines 175 to 184
//const simulation = await client.simulate(walletAddress, msgs, "");
// const estimatedFees = {
// amount: [
// {
// amount: "0.025",
// denom: uAktDenom
// }
// ],
// gas: Math.ceil(simulation * 1.25).toString()
// };
Copy link
Contributor

@baktun14 baktun14 Dec 16, 2023

Choose a reason for hiding this comment

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

✂️

Copy link
Contributor

Choose a reason for hiding this comment

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

Ah it's draft.

@Redm4x Redm4x marked this pull request as ready for review December 19, 2023 17:34
@Redm4x
Copy link
Contributor Author

Redm4x commented Dec 19, 2023

Metamask support will be added in a second step, progress will be tracked in #62

@Redm4x Redm4x merged commit f2c5c0e into main Dec 19, 2023
5 checks passed
@Redm4x Redm4x deleted the features/cosmoskit-integration branch December 19, 2023 18:00
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.

Add Cosmokit Integration
3 participants