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

Create a new provider-xion package. #135

Merged

Conversation

adairrr
Copy link
Contributor

@adairrr adairrr commented Oct 29, 2024

  • create a new xion provider
  • Update CI version thing to include xion

Copy link

changeset-bot bot commented Oct 29, 2024

🦋 Changeset detected

Latest commit: b5e465e

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@abstract-money/provider-xion Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@adairrr adairrr changed the title adair/ajs 46 create a new xion provider in abstractjs Create a new provider-xion package. Oct 29, 2024
@adairrr
Copy link
Contributor Author

adairrr commented Oct 29, 2024

Hm @Pessina do you think we should make it depend on the graz provider? Would mean that we wouldn't have to deal with the weirdness of the current cosmwasm client querier.... though it would also mean that devs would have to take a dep on the GrazProvider...

@Pessina
Copy link
Contributor

Pessina commented Oct 30, 2024

Hm, @adairrr, I think if we will only depend on graz to get the cosmwas client, we should not do it. The current code it's not very complicated and including graz will increase the bundle size.

@adairrr adairrr merged commit 1dfb7c3 into mainline Oct 30, 2024
2 of 4 checks passed
@github-actions github-actions bot mentioned this pull request Oct 30, 2024
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