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

updated screener #6

Merged
merged 3 commits into from
Feb 17, 2021
Merged

updated screener #6

merged 3 commits into from
Feb 17, 2021

Conversation

mzaganczyk
Copy link
Contributor

No description provided.

@daxm daxm merged commit 0b6e739 into daxm:master Feb 17, 2021
@daxm
Copy link
Owner

daxm commented Feb 17, 2021

Did I miss them when I initially build this method or were these options just added? My apologies if I just missed them.

Thanks for the PR!

@mzaganczyk
Copy link
Contributor Author

Actually I don't know, I'm using this API since yesterday and I've just found your python module, it's great! Is this PR already available in the PyPi package?

@daxm
Copy link
Owner

daxm commented Feb 17, 2021

It is now. :-) 20210217.0

@mzaganczyk
Copy link
Contributor Author

Thank you!

@daxm
Copy link
Owner

daxm commented Feb 17, 2021

I don't know how else to communicate with you so I'm putting it on this PR.

I'm a bit over-busy right now. Would you be willing to add in the "insider_trading" method? I swear I had added everything in back in December but either FMP made changes or I missed this.

@mzaganczyk
Copy link
Contributor Author

Sure! I'll work on it tomorrow!

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.

None yet

2 participants