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

FEATURE: [xalign] add notification when order quote is over alert amount #1791

Merged
merged 1 commit into from
Nov 12, 2024

Conversation

kbearXD
Copy link
Collaborator

@kbearXD kbearXD commented Oct 24, 2024

No description provided.

@bbgokarma-bot
Copy link

Welcome back! @kbearXD, This pull request may get 225 BBG.

config/xalign.yaml Outdated Show resolved Hide resolved
config/xalign.yaml Outdated Show resolved Hide resolved
@kbearXD kbearXD force-pushed the kbearXD/xalign/notify-if-large-amount branch from 52ec12a to 302b78f Compare November 1, 2024 06:42
@bbgokarma-bot
Copy link

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

@kbearXD kbearXD force-pushed the kbearXD/xalign/notify-if-large-amount branch from 302b78f to 1813f4c Compare November 5, 2024 06:21
@bbgokarma-bot
Copy link

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

@kbearXD kbearXD requested a review from c9s November 5, 2024 06:22
@kbearXD kbearXD force-pushed the kbearXD/xalign/notify-if-large-amount branch from 1813f4c to 57e42ad Compare November 5, 2024 07:49
@bbgokarma-bot
Copy link

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

@kbearXD kbearXD force-pushed the kbearXD/xalign/notify-if-large-amount branch from 57e42ad to caf5d69 Compare November 5, 2024 07:54
@bbgokarma-bot
Copy link

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

@@ -50,6 +100,7 @@ type Strategy struct {
BalanceToleranceRange fixedpoint.Value `json:"balanceToleranceRange"`
Duration types.Duration `json:"for"`
MaxAmounts map[string]fixedpoint.Value `json:"maxAmounts"`
MaxAmountAlert *MaxAmountAlertConfig `json:"maxAmountAlert"`
Copy link
Owner

Choose a reason for hiding this comment

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

call it LargeAmountAlert?

func (m *MaxAmountAlert) SlackAttachment() slack.Attachment {
return slack.Attachment{
Color: "red",
Title: fmt.Sprintf("xalign amount alert - try to align %s with quote %s amount %f > %f",
Copy link
Owner

Choose a reason for hiding this comment

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

Large Amount Alert

@@ -50,6 +100,7 @@ type Strategy struct {
BalanceToleranceRange fixedpoint.Value `json:"balanceToleranceRange"`
Duration types.Duration `json:"for"`
MaxAmounts map[string]fixedpoint.Value `json:"maxAmounts"`
Copy link
Owner

Choose a reason for hiding this comment

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

This field can be removed?

@kbearXD kbearXD force-pushed the kbearXD/xalign/notify-if-large-amount branch from caf5d69 to b47fc08 Compare November 11, 2024 09:28
@bbgokarma-bot
Copy link

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

@kbearXD kbearXD merged commit b57c54d into main Nov 12, 2024
3 checks passed
@kbearXD kbearXD deleted the kbearXD/xalign/notify-if-large-amount branch November 12, 2024 08:15
@bbgokarma-bot
Copy link

Hi @kbearXD,

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

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

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