Skip to content

Conversation

narumiruna
Copy link
Contributor

No description provided.

@bbgokarma-bot
Copy link

Welcome back! @narumiruna, This pull request may get 219 BBG.

@bbgokarma-bot
Copy link

Re-estimated karma: this pull request may get 229 BBG

@bbgokarma-bot
Copy link

Re-estimated karma: this pull request may get 246 BBG

return nil
}

func (s *Strategy) Defaults() error {
if s.OrderType == "" {
Copy link
Owner

Choose a reason for hiding this comment

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

order type needs to be aligned with the price type

if price type = taker, the default limit maker won’t work

Copy link
Contributor Author

Choose a reason for hiding this comment

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

so add a warning here?

Copy link
Owner

Choose a reason for hiding this comment

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

how about if priceType = taker { then use normal limit order instead }
else { then use limit maker order as a default }

Copy link
Owner

Choose a reason for hiding this comment

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

the default order type should be "limit", because it would work for all the cases

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I will set the default order type to limit

@narumiruna narumiruna force-pushed the narumi/autobuy-boll branch from 15196ea to 1b06fcc Compare August 22, 2024 06:36
@bbgokarma-bot
Copy link

Re-estimated karma: this pull request may get 301 BBG

@c9s c9s merged commit b2f1f7d into main Aug 23, 2024
3 checks passed
@c9s c9s deleted the narumi/autobuy-boll branch August 23, 2024 05:05
@bbgokarma-bot
Copy link

Hi @narumiruna,

Well done! 306 BBG has been sent to your polygon wallet. Please check the following tx:

https://polygonscan.com/tx/0xdf9bc7b800e9dfb97597607eaa5df4b85b795c48eb139a56882c364b0b842536

Thank you for your contribution!

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.

3 participants