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

main: Update ticket buyer startup logging. #2409

Closed
wants to merge 1 commit into from

Conversation

jholdstock
Copy link
Member

In all code and log messages the ticket buyer is referred to by name, however in startup/shutdown logging it was given the more vague name "auto transaction creator".

Its also helpful to log whether or not the ticket buyer is purchasing mixed tickets because that has a large impact on its behaviour and potential failure modes.

In all code and log messages the ticket buyer is referred to by name,
however in startup/shutdown logging it was given the more vague name
"auto transaction creator".

Its also helpful to log whether or not the ticket buyer is purchasing
mixed tickets because that has a large impact on its behaviour and
potential failure modes.
@jrick
Copy link
Member

jrick commented Sep 9, 2024

We started leaning (or at least, I did) towards calling it the auto transaction creator since it also is responsible for mixing accounts, and it would be the place that we did hook up any other sort of automatic transaction buying in the future if we were to add more.

@jholdstock
Copy link
Member Author

OK I see that now, might revisit this later.

@jholdstock jholdstock closed this Sep 9, 2024
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.

2 participants