[bugfix][protocol] fix settled funding event emission (#1051)

* [bugfix][protocol] fix settled funding event emission * add TODO
dydxprotocol · Feb 6, 2024 · ab6a65c · ab6a65c
1 parent e6398f3
commit ab6a65c
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/protocol/x/subaccounts/types/events.go b/protocol/x/subaccounts/types/events.go
@@ -19,6 +19,7 @@ const (
 
 // NewCreateSettledFundingEvent constructs a new funding sdk.Event. Note that `fundingPaid` is positive
 // if the subaccount paid funding, negative if the subaccount received funding.
+// TODO(CT-245): Add tests that this event is emitted.
 func NewCreateSettledFundingEvent(
 	subaccount SubaccountId,
 	perpetualId uint32,
@@ -27,7 +28,7 @@ func NewCreateSettledFundingEvent(
 	return sdk.NewEvent(
 		EventTypeSettledFunding,
 		sdk.NewAttribute(AttributeKeySubaccount, subaccount.Owner),
-		sdk.NewAttribute(AttributeKeySubaccount, fmt.Sprint(subaccount.Number)),
+		sdk.NewAttribute(AttributeKeySubaccountNumber, fmt.Sprint(subaccount.Number)),
 		sdk.NewAttribute(AttributeKeyPerpetualId, fmt.Sprint(perpetualId)),
 		sdk.NewAttribute(AttributeKeyFundingPaid, fundingPaid.String()),
 	)