Skip to content

[Node] handle full bot start#3512

Merged
GuillaumeDSM merged 1 commit into
devfrom
start_full_bot
Jun 25, 2026
Merged

[Node] handle full bot start#3512
GuillaumeDSM merged 1 commit into
devfrom
start_full_bot

Conversation

@GuillaumeDSM

Copy link
Copy Markdown
Member

fixes many bugs related to process bots, now everything seens to work fine (and is tested)

@GuillaumeDSM GuillaumeDSM self-assigned this Jun 24, 2026
@GuillaumeDSM GuillaumeDSM requested a review from Herklos as a code owner June 24, 2026 20:45
@GuillaumeDSM GuillaumeDSM force-pushed the start_full_bot branch 2 times, most recently from 5737d26 to cfe15ca Compare June 24, 2026 21:06
{
configuration_type: "generic_process",
} satisfies GenericProcessConfiguration,
"USDC",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Shouldn't the ref market be a param too alongside id?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

it could but it's just a template here, we just want default values. id is a param to generate a new random id at each call but we don't have this need for ref markets

@Herklos Herklos left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

👍

@GuillaumeDSM GuillaumeDSM merged commit b5c7766 into dev Jun 25, 2026
23 checks passed
@GuillaumeDSM GuillaumeDSM deleted the start_full_bot branch June 25, 2026 07:12
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