add dynamic commission acc seq fix

[nghuyenthevinh2000]

Summary of changes

Report of required housekeeping

• Github issue OR spec proposal link
• Wrote tests
• Updated API documentation (client/lcd/swagger-ui/swagger.yaml)
• Added a relevant changelog entry: clog add [section] [stanza] [message]

---

(FOR ADMIN) Before merging

• Added appropriate labels to PR
• Squashed all commits, uses message "Merge pull request #XYZ: [title]" (coding standards)
• Confirm added tests are consistent with the intended behavior of changes
• Ensure all tests pass

[basitalikhoso110]

fix-dyncomm

[fragwuerdig]

I think that looks good from my side.

My suggestion would be to introduce a "legacy" dyncomm ante decorator that only runs the dyncomm ante handler for "<" specified block height. The normal one would run for ">=" specified block height.

This way the chain could be forked without explicitly halting the chain.

[StrathCole]

I think that looks good from my side.

My suggestion would be to introduce a "legacy" dyncomm ante decorator that only runs the dyncomm ante handler for "<" specified block height. The normal one would run for ">=" specified block height.

This way the chain could be forked without explicitly halting the chain.

I think the validators complained about not having a proposal for notifying them about the update, so maybe a proposal with chain halt is the best way to make sure they all see it.