-
Notifications
You must be signed in to change notification settings - Fork 291
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
Port all existing event.rs
files to use proto events.
#3504
Comments
Note to self, that we should also port the staking component events to use proto-event. |
I will push this over the finish line before we release Testnet 65. I will first take care of staking component related items. |
Was this completed? We should ensure it's completed as part of V1 -- we can always add more events but changing events people are using will be difficult. |
while looking around through files named `event.rs`, per the description of #3504, i found that some of these files are empty. this commit removes these empty files.
while looking around through files named `event.rs`, per the description of #3504, i found that some of these files are empty. this commit removes these empty files.
while looking around through files named `event.rs`, per the description of #3504, i found that some of these files are empty. this commit removes these empty files.
The only place using non-protobuf event types is now the IBC code. This will require defining protobuf event types in |
Following the work in #3460 to define protobuf event handling, it would be good to convert the existing events:
In particular, porting over the DEX events will make them useful for trading bots (cf https://discord.com/channels/824484045370818580/1163684711667867678/1181748568135061514 )
The text was updated successfully, but these errors were encountered: