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

cannot create TransientStoreService #20262

Closed
beer-1 opened this issue May 3, 2024 · 0 comments · Fixed by #20261
Closed

cannot create TransientStoreService #20262

beer-1 opened this issue May 3, 2024 · 0 comments · Fixed by #20261
Labels
needs-triage Issue that needs to be triaged

Comments

@beer-1
Copy link
Contributor

beer-1 commented May 3, 2024

in #20261

@tac0turtle
> Hey we don't support the transient store as a service. If a module uses it, it will need to be passed in directly. In store v2 the transient store also is no longer present.

then can you give some example how to pass transient store service to the below keeper? seems v0.50 still has transient store service and only depinject can use this module without sdk changes.

https://github.com/noble-assets/forwarding/blob/0884f39cf30c95fc662b72a17137b7240fafc34e/x/forwarding/keeper/keeper.go#L45

Originally posted by @beer-1 in #20261 (comment)

@github-actions github-actions bot added the needs-triage Issue that needs to be triaged label May 3, 2024
@github-project-automation github-project-automation bot moved this to 📋 Backlog in Cosmos-SDK May 3, 2024
@github-project-automation github-project-automation bot moved this from 📋 Backlog to 🥳 Done in Cosmos-SDK May 9, 2024
@tac0turtle tac0turtle removed this from Cosmos-SDK Jul 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-triage Issue that needs to be triaged
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant