Hotfix/reddit spacc

[Ltwo3five]

Description

Summary of the change / bug fix.

Using finviz.get_stock(t_ticker[0]) to validate the tickers in a for loop throws error 429 which causes some tickers that were mentioned in the reddit posts on r/spacc to not appear in the result
I used screener_view from finvizfinance to return a list of all tickers to compare against instead.
Additionally the function print_rich_table has logic which was converting any column entry that is not a time stamp into a numeric value. praw.models.reddit.subreddit.Subreddit class could not be converted into int which throws a value error, and if it is not converted into the correct type when the dataframe is converted into JSON, a recursion error will be thrown.
My solution was to convert all column entries into strings in the except block

- Link # issue, if applicable.

#4929

- Screenshot of the feature or the bug before/after fix, if applicable.

tables are now printing when command is run

- List any dependencies that are required for this change.

  finvizfinance

How has this been tested?

End to End tests and running existing tests

Please describe the tests that you ran to verify your changes.

Ran all tests to see if changes cause any error

Provide instructions so we can reproduce.

Command /stocks/ba/spacc

Please also list any relevant details for your test configuration.

• Make sure affected commands still run in terminal
• Ensure the SDK still works
• Check any related reports

Checklist:

• I have adhered to the GitFlow naming convention and my branch name is in the format of feature/feature-name or hotfix/hotfix-name.
• Update our documentation following these guidelines. Update any user guides that are affected by the changes.
• Update our tests following these guidelines.
• Make sure you are following our CONTRIBUTING guidelines.
• If a feature was added make sure to add it to the corresponding integration test script.

Others

• I have performed a self-review of my own code.
• I have commented my code, particularly in hard-to-understand areas.

[jmaslek]

Thanks for fixing up the extra files!

something I asked in a previous open PR on this was if you can combine the outputs into a single df so that it doesnt open multiple table windows?

[Ltwo3five]

Ok I will do that

[Ltwo3five]

I have pushed the changes

[jmaslek]

Works as advertised! Thanks for the fix!

[jmaslek]

I just piggybacked and added a quick patch for the wsb prints

[Ltwo3five]

Works as advertised! Thanks for the fix!

You're welcome!