Commit Graph

5 Commits

Author SHA1 Message Date
Adi Sai
4fe6cf44a3 Feature: Ultima Insights NewsMonitor Integration (#4685)
* fix typing issue that prevents running

* news monitor integration for news

* UX improvements with news monitor integration

* italicize riskExtDesc

* minor cosmetic changes

* minor cosmetic changes

* minor cosmetic changes

* Update ultima_newsmonitor_view.py

* minor fixes to launch properly

* setting Ultima API key

* Ultima NewsMonitor integration into terminal done

* integration of the commands is done

* add Ultima to SDK trailmap, but SDK generation failed on M1 Mac

* add tests

* add unit tests

* Linting and sdk

* display message if not supported by Ultima News Monitor

* Update api-keys.md

* fix stock controller problem + remove source filtering from Ultima

* remove source filtering from Ultima

* add report issue message

* add some error handling in the view to ensure unsupported companies don't get fetched

* remove powered by Ultima Insights from console output

* ran black formatter to fix linting issues

* fix ruff issues

* fix pylint issues

* lint fix

* fixed linting issue and changed keys in the Feedparser model

* get all tests to pass

* proper testing with --record-mode=rewrite

* re-run tests for updated VCRs

* redid black formatting

* re-run test_print_help in stocks controller

* make breaking news output consistent and re-run tests

* remove unused import time

* reran tests with --rewrite-expected

---------

Co-authored-by: ovidysus0 <111447781+ovidysus0@users.noreply.github.com>
Co-authored-by: James Maslek <jmaslek11@gmail.com>
Co-authored-by: andrewkenreich <andrew.kenreich@gmail.com>
2023-04-05 05:33:48 +00:00
montezdesousa
12a41ca63a Fix stocks/news command (#4653)
* fix news dataframe

* fix feedparser

* rewrite expected
2023-04-02 23:26:46 +00:00
DidierRLopes
58fb4cd190 fix news sources (#3118)
* fix news sources

* lint

* ordering

* fix tests

Co-authored-by: James Maslek <jmaslek11@gmail.com>
2022-10-27 14:51:59 +01:00
Colin Delahunty
87fac96b7c Improved sorting capabilities in the news function (#2682)
* Added sorting to news command

* Separate PRs

* Added fixes

* Added fixes

* Forgot to remove item from api

* Test fixes
2022-10-03 11:44:06 +02:00
James Simmons
5f5f07c295 Fix bugs for news (SSL, sources) and fred (SSL) (#2585)
* potential solution number 1

* Making sure certifi is referenced for news and fred

* Fix default sources not being applied

* Add unit and integration tests

Co-authored-by: James Maslek <jmaslek11@gmail.com>
Co-authored-by: Theodore Aptekarev <aptekarev@gmail.com>
2022-09-22 19:32:27 +03:00