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

Jump to bottom

[Bug] /stocks/ins/lins - No insider information found for WMT #5040

Closed
deeleeramone opened this issue May 17, 2023 · 5 comments
Closed

[Bug] /stocks/ins/lins - No insider information found for WMT #5040

deeleeramone opened this issue May 17, 2023 · 5 comments
Labels
bug Fix bug

Comments

@deeleeramone
Copy link
Contributor

The error is probably similar to the other recent issues from the Finviz library that have been patched already.

/stocks/load wmt/ins/lins

Screenshot 2023-05-17 at 1 15 20 PM

#5010
#4954
#3732
@deeleeramone deeleeramone added the bug Fix bug label May 17, 2023
@jmaslek
Copy link
Collaborator

jmaslek commented May 19, 2023

error is directly in the finviz library we use:

Screenshot 2023-05-19 at 3 00 02 PM

@jmaslek
Copy link
Collaborator

jmaslek commented May 19, 2023

Found the cause. Will need to update when this is fixed:

mariostoev/finviz#170

@the-praxs
Copy link
Contributor

I thought we were using finvizfinance api. This one is different.

@deeleeramone
Copy link
Contributor Author

Looks like there's two different Finviz libraries in the dependencies..

@deeleeramone
Copy link
Contributor Author

mariostoev/finviz#171

This will be a needed fix as well.

@jmaslek jmaslek closed this as completed May 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Fix bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@jmaslek @deeleeramone @the-praxs