From 779266eedded156d346568935a0705cf0cee7467 Mon Sep 17 00:00:00 2001 From: RafilxTenfen Date: Mon, 7 Oct 2024 10:53:11 -0300 Subject: [PATCH 1/5] fix: removed btc status unbonding from error return string --- x/btcstaking/types/btc_delegation.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/x/btcstaking/types/btc_delegation.go b/x/btcstaking/types/btc_delegation.go index 1c3cd4ff6..38850c4a9 100644 --- a/x/btcstaking/types/btc_delegation.go +++ b/x/btcstaking/types/btc_delegation.go @@ -30,7 +30,7 @@ func NewBTCDelegationStatusFromString(statusStr string) (BTCDelegationStatus, er case "any": return BTCDelegationStatus_ANY, nil default: - return -1, fmt.Errorf("invalid status string; should be one of {pending, active, unbonding, unbonded, any}") + return -1, fmt.Errorf("invalid status string; should be one of {pending, active, unbonded, any}") } } From 8f74c213ddec32bf5f98ad1ba9aa459cdb5b40bc Mon Sep 17 00:00:00 2001 From: RafilxTenfen Date: Mon, 7 Oct 2024 10:54:35 -0300 Subject: [PATCH 2/5] chore: add #139 to changelog --- CHANGELOG.md | 2 ++ 1 file changed, 2 insertions(+) diff --git a/CHANGELOG.md b/CHANGELOG.md index 79e0d916f..39f24b3a0 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -50,6 +50,8 @@ messages inside `authz.MsgExec` * [#113](https://github.com/babylonlabs-io/babylon/pull/113) Add multibuild binary for upgrade handler `testnet` and `mainnet`. +* [#139](https://github.com/babylonlabs-io/babylon/pull/139) Removed `unbonding` +delegation status in query. ## v0.11.0 From bed5a48ecd70a3b86d95def7dfcce3230cd135e2 Mon Sep 17 00:00:00 2001 From: RafilxTenfen Date: Mon, 7 Oct 2024 10:55:54 -0300 Subject: [PATCH 3/5] fix: update changelog PR to 140 instead of 139 --- CHANGELOG.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 39f24b3a0..ecd7ce15b 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -50,7 +50,7 @@ messages inside `authz.MsgExec` * [#113](https://github.com/babylonlabs-io/babylon/pull/113) Add multibuild binary for upgrade handler `testnet` and `mainnet`. -* [#139](https://github.com/babylonlabs-io/babylon/pull/139) Removed `unbonding` +* [#140](https://github.com/babylonlabs-io/babylon/pull/140) Removed `unbonding` delegation status in query. ## v0.11.0 From 5b9cc82e2b433e6ccf0297bb01ec1613bfe397fc Mon Sep 17 00:00:00 2001 From: RafilxTenfen Date: Mon, 7 Oct 2024 12:28:17 -0300 Subject: [PATCH 4/5] chore: add verified to NewBTCDelegationStatusFromString --- x/btcstaking/types/btc_delegation.go | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/x/btcstaking/types/btc_delegation.go b/x/btcstaking/types/btc_delegation.go index 38850c4a9..9af4fe325 100644 --- a/x/btcstaking/types/btc_delegation.go +++ b/x/btcstaking/types/btc_delegation.go @@ -23,6 +23,8 @@ func NewBTCDelegationStatusFromString(statusStr string) (BTCDelegationStatus, er switch statusStr { case "pending": return BTCDelegationStatus_PENDING, nil + case "verified": + return BTCDelegationStatus_VERIFIED, nil case "active": return BTCDelegationStatus_ACTIVE, nil case "unbonded": @@ -30,7 +32,7 @@ func NewBTCDelegationStatusFromString(statusStr string) (BTCDelegationStatus, er case "any": return BTCDelegationStatus_ANY, nil default: - return -1, fmt.Errorf("invalid status string; should be one of {pending, active, unbonded, any}") + return -1, fmt.Errorf("invalid status string; should be one of {pending, verified, active, unbonded, any}") } } From d851cb1f0b5acaccea7182a9acf68f942568376b Mon Sep 17 00:00:00 2001 From: RafilxTenfen Date: Mon, 7 Oct 2024 12:29:41 -0300 Subject: [PATCH 5/5] fix: changelog #140 as bugfix --- CHANGELOG.md | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index ecd7ce15b..9684cb676 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -37,6 +37,11 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/) ## Unreleased +### Bug Fixes + +* [#140](https://github.com/babylonlabs-io/babylon/pull/140) Removed `unbonding` +and add `verified` to delegation status parse `NewBTCDelegationStatusFromString`. + ### State Machine Breaking * [#130](https://github.com/babylonlabs-io/babylon/pull/130) Fix bugs in the @@ -50,8 +55,6 @@ messages inside `authz.MsgExec` * [#113](https://github.com/babylonlabs-io/babylon/pull/113) Add multibuild binary for upgrade handler `testnet` and `mainnet`. -* [#140](https://github.com/babylonlabs-io/babylon/pull/140) Removed `unbonding` -delegation status in query. ## v0.11.0