Skip to content

feat: Add dynamicfee module#48

Open
giuliostramondo wants to merge 6 commits intorelease/v0.50.xfrom
giuliostramondo/chore/dynamicfee-sdk
Open

feat: Add dynamicfee module#48
giuliostramondo wants to merge 6 commits intorelease/v0.50.xfrom
giuliostramondo/chore/dynamicfee-sdk

Conversation

@giuliostramondo
Copy link
Collaborator

No description provided.

@github-actions
Copy link

@giuliostramondo your pull request is missing a changelog!

@giuliostramondo giuliostramondo changed the title feat: Add dynamicfee module to sdk feat: Add dynamicfee module Dec 19, 2025
@@ -0,0 +1,18 @@
syntax = "proto3";

package cosmos.dynamicfee.module.v1;
Copy link
Collaborator

Choose a reason for hiding this comment

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

So if we change the proto path, which sounds legit, we'll break again the API.
Fortunately it should not be widely used, but there's a pending PR on Hermes that will need to be updated (even though it will never be merged, let's be honest...)

Copy link
Collaborator

@tbruyelle tbruyelle left a comment

Choose a reason for hiding this comment

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

LGTM

@giuliostramondo giuliostramondo force-pushed the giuliostramondo/chore/dynamicfee-sdk branch from 352f5ba to 2a6fb4a Compare January 26, 2026 17:01
@giuliostramondo giuliostramondo force-pushed the giuliostramondo/chore/dynamicfee-sdk branch from 2a6fb4a to dc057b0 Compare January 27, 2026 13:27
Co-authored-by: Giuseppe Natale <12249307+giunatale@users.noreply.github.com>
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