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.

Sign up for GitHub

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

Catch tweetnews exceptions #4603

Merged
merged 4 commits into from
Mar 28, 2023
Merged

Catch tweetnews exceptions #4603

Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
9ec1614
catch tweetnews exceptions
montezdesousa Mar 28, 2023
5325ad4
catch all exceptions
montezdesousa Mar 28, 2023
2f8b7d6
pylint
montezdesousa Mar 28, 2023
4f0a220
Merge branch 'develop' into hotfix/tweetnews
montezdesousa Mar 28, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
15 changes: 8 additions & 7 deletions openbb_terminal/helper_funcs.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@
from openbb_terminal.core.plots.plotly_ta.ta_class import PlotlyTA

# IMPORTS INTERNAL
from openbb_terminal.core.session.current_user import get_current_user
from openbb_terminal.core.session.current_user import get_current_user, set_preference
from openbb_terminal.rich_config import console

try:
Expand All @@ -68,9 +68,10 @@
# A test to ensure that the Twitter API key is correct,
# otherwise we disable the Toolbar with Tweet News
twitter_api.get_user(screen_name="openbb_finance")
except tweepy.errors.Unauthorized:
except Exception as exc:
# Set toolbar tweet news to False because the Twitter API is not set up correctly
get_current_user().preferences.TOOLBAR_TWEET_NEWS = False
set_preference("TOOLBAR_TWEET_NEWS", False)
console.print(f"Error enabling tweet news: {exc}")


logger = logging.getLogger(__name__)
Expand Down Expand Up @@ -1856,8 +1857,8 @@ def screenshot() -> None:
else:
console.print("No plots found.\n")

except Exception as e:
console.print(f"Cannot reach window - {e}\n")
except Exception as err:
console.print(f"Cannot reach window - {err}\n")


def screenshot_to_canvas(shot, plot_exists: bool = False):
Expand Down Expand Up @@ -2085,8 +2086,8 @@ def update_news_from_tweet_to_be_displayed() -> str:
url = f"https://twitter.com/x/status/{last_tweet.id_str}"

# In case the handle provided doesn't exist, we skip it
except (tweepy.errors.NotFound, tweepy.errors.Unauthorized):
pass
except Exception as e:
console.print(f"Error enabling tweet news: {handle} - {e}\n")

if last_tweet_dt and news_tweet_to_use:
tweet_hr = f"{last_tweet_dt.hour}"
Expand Down