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

Improve stock not found handling in mentions with multiple stocks #12

Open
luisgc93 opened this issue Jan 17, 2021 · 0 comments
Open

Improve stock not found handling in mentions with multiple stocks #12

luisgc93 opened this issue Jan 17, 2021 · 0 comments
Labels
bug Something isn't working

Comments

@luisgc93
Copy link
Owner

luisgc93 commented Jan 17, 2021

When a mention contains multiple stocks and one of them is not found (e.g. due to a typo or the stock not trading in the NASDAQ), the bot should reply saying that it will remind the user of those stocks that have been found but that one (or more ) of them couldn't be found.

Instead, what happens currently is that the bot responds with the stock not found reply:

bot_mention

This is despite succesfully creating reminders for the other stocks ($BABA and $BTC in the above example. $TCENY is a typo and should be $TCEHY instead, hence why the stock wasn't found).

@luisgc93 luisgc93 added the bug Something isn't working label Jan 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant