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 misc logging #77

Open
wants to merge 3 commits into
base: upcoming
Choose a base branch
from
Open

Add misc logging #77

wants to merge 3 commits into from

Conversation

DaSichuan
Copy link

No description provided.

@DaSichuan
Copy link
Author

Gonna redo this to try to use procedural macros, and output as json

@DaSichuan DaSichuan force-pushed the WM_logging branch 7 times, most recently from 3da84ac to a62dead Compare February 26, 2022 04:40
@DaSichuan DaSichuan changed the title Log successful price updates Log price updates and oracle errors Feb 26, 2022

#[derive(Serialize)]
pub struct PythOraclePriceUpdate {
pub event_type: LogEventType,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can't this be a macro?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Which part?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the event type being on every struct

@DaSichuan DaSichuan force-pushed the WM_logging branch 19 times, most recently from c4b635a to 3c2b626 Compare March 8, 2022 02:32
@nope-finance nope-finance changed the base branch from mainnet to upcoming March 8, 2022 06:10
@DaSichuan DaSichuan force-pushed the WM_logging branch 5 times, most recently from 671bc48 to a307171 Compare March 16, 2022 00:45
@DaSichuan DaSichuan changed the title Log price updates and oracle errors Add misc logging Mar 16, 2022
@DaSichuan DaSichuan force-pushed the WM_logging branch 5 times, most recently from b8410c2 to 059d4a3 Compare March 18, 2022 23:49
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.

None yet

2 participants