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

FIX: [telegram] prevent sending error to telegram for the case of no opened position #1521

Merged

Conversation

michaljirman
Copy link
Contributor

Executing the Telegram command /position to request information about open positions results in an error message when there are no open positions. The state of having no open positions should not be treated as an error.

@bbgokarma-bot
Copy link

Hi @michaljirman,

This is KarmaBot, and we reward your contributions with tokens sent directly to your wallet to support development.

This pull request may get 205 BBG.

To receive BBG tokens, please provide your Polygon (can be Ethereum) address as an issue comment in this pull request, following this format:

polygon:0xAb5801a7D398351b8bE11C439e05C5B3259aeC9B

Once this pull request is merged, your BBG tokens will be transferred to your wallet.

--
If you're interested in our project, feel free to join our Telegram group https://t.me/bbgo_intl

Best,
KarmaBot

@CLAassistant
Copy link

CLAassistant commented Feb 2, 2024

CLA assistant check
All committers have signed the CLA.

@michaljirman michaljirman changed the title telegram: prevent sending error in case of no opened position FIX: prevent sending error to telegram for case of no opened position Feb 2, 2024
@michaljirman michaljirman changed the title FIX: prevent sending error to telegram for case of no opened position FIX: [telegram] prevent sending error to telegram for the case of no opened position Feb 2, 2024
@c9s c9s enabled auto-merge February 2, 2024 16:25
@c9s c9s merged commit 80ed46e into c9s:main Feb 2, 2024
3 checks passed
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.

4 participants