-
Notifications
You must be signed in to change notification settings - Fork 31
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
docs: update outdated events in specs #602
Changes from 15 commits
6d27b27
21a9ce4
3136423
59ca12b
d6d6997
6125095
e2953cc
2d474dc
dbb0094
e86e40c
441b19a
6daea7b
717dd82
de0498f
fcc30b9
bd1e0b0
51ccddb
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,18 @@ | ||
|
||
# Events of LBM-SDK | ||
|
||
Note: the content of this document is based on the actual implementation in version 0.45.6 of the cosmos-sdk. As we do not intend to introduce a new specification, it's a bug of the documentation if you find any differences between this document and the implementation. | ||
|
||
Note: these specifications are subject to change. | ||
|
||
## Common events | ||
|
||
In its process, every message emits an event of type `message` which has an attribute of a key `action`. This attribute has a value of `TypeURL` of the message. `TypeURL` is just a message name of the message, prefixed by `/`. For example, `TypeURL` of a message `MsgSend` in x/bank would be `/cosmos.bank.v1beta1.MsgSend`. | ||
|
||
This event MUST proceed to any other events in its process of the message. | ||
0Tech marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
||
## Ordering | ||
|
||
The ordering of the events MUST be preserved in `Events` of `TxResponse`. | ||
|
||
Note: the field `Logs` is non-deterministic as cosmos-sdk says. |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -11,22 +11,21 @@ The bank module emits the following events: | |
### MsgSend | ||
|
||
| Type | Attribute Key | Attribute Value | | ||
| -------- | ------------- | ------------------ | | ||
|----------|---------------|--------------------| | ||
| transfer | recipient | {recipientAddress} | | ||
| transfer | sender | {senderAddress} | | ||
| transfer | amount | {amount} | | ||
| message | module | bank | | ||
| message | action | send | | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Why There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Some modules do not have the event of type `message' in its document. So, I decided to put the event into the common document, and remove it from the individual module's. |
||
| message | sender | {senderAddress} | | ||
| message | module | bank | | ||
|
||
### MsgMultiSend | ||
|
||
| Type | Attribute Key | Attribute Value | | ||
| -------- | ------------- | ------------------ | | ||
|----------|---------------|--------------------| | ||
| message | sender | {senderAddress} | | ||
| transfer | recipient | {recipientAddress} | | ||
| transfer | amount | {amount} | | ||
| message | module | bank | | ||
| message | action | multisend | | ||
| message | sender | {senderAddress} | | ||
|
||
## Keeper events | ||
|
||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,7 @@ | ||
<!-- | ||
order: 3 | ||
--> | ||
|
||
# Events | ||
|
||
The collection module emits proto events defined in [the Protobuf reference](../../../docs/core/proto-docs.md#lbm/collection/v1/event.proto). |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,7 @@ | ||
<!-- | ||
order: 3 | ||
--> | ||
|
||
# Events | ||
|
||
The token module emits proto events defined in [the Protobuf reference](../../../docs/core/proto-docs.md#lbm/token/v1/event.proto). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Current lbm-sdk is based on cosmos-sdk v0.45.1. But we need to bump up
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cosmos-sdk version up issue was added in #575 . Please check it later.