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 Tx-boundary module #224

Merged
merged 14 commits into from
Aug 28, 2023
Merged

Add Tx-boundary module #224

merged 14 commits into from
Aug 28, 2023

Conversation

expertdicer
Copy link
Contributor

@expertdicer expertdicer commented Aug 18, 2023

This PR close #215 first task
This module limit delegate and redelegate tx

@expertdicer expertdicer marked this pull request as ready for review August 23, 2023 07:06
@expertdicer expertdicer requested review from GNaD13 and vuong177 August 23, 2023 07:06
x/tx-boundary/keeper/keeper.go Outdated Show resolved Hide resolved
x/tx-boundary/keeper/keeper.go Show resolved Hide resolved
x/tx-boundary/keeper/keeper.go Show resolved Hide resolved
app/ante/custom_ante.go Outdated Show resolved Hide resolved
x/tx-boundary/keeper/grpc_query.go Outdated Show resolved Hide resolved
x/tx-boundary/keeper/grpc_query.go Outdated Show resolved Hide resolved
app/ante/custom_ante.go Outdated Show resolved Hide resolved
app/ante/custom_ante.go Outdated Show resolved Hide resolved
@blasrodri
Copy link

Why not go with https://docs.evmos.org/protocol/modules/epochs

@expertdicer expertdicer requested a review from vuong177 August 26, 2023 02:20
@vuong177
Copy link
Contributor

vuong177 commented Aug 28, 2023

@blasrodri now the period of limit is based on block. (default 500block ~ 3000s). Because the block time is stable (6s/block), we don't need epoch module here.

@vuong177 vuong177 merged commit cd71497 into main Aug 28, 2023
tungleanh0902 pushed a commit to notional-labs/composable-cosmos that referenced this pull request Mar 7, 2024
@hoank101 hoank101 deleted the nguyen/tx-boundary-module branch April 8, 2024 03: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.

v5 scoping issue
3 participants