-
Notifications
You must be signed in to change notification settings - Fork 691
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
Fixes .com// in public api url and updates query_hash's for user_following_gql_chunk and user_followers_gql_chunk #1666
Conversation
Removed .lstrip() to fix KeyError and added model_config to fix Pydantic
@jarrodnorwell big thanks! |
user_following_gql still does not return the list of usershorts unfortunately
|
Are you using the GitHub or PyPI version of the project @s-alad? I did testing myself with the fix and it was working |
I'm using PyPi, I just ran pip3 install instagrapi Would this fix not be included in the version I pulled? |
@s-alad right now the fix is only on GitHub afaik, you'll need to install it through Git |
Thank you will try the github version and update |
@jarrodnorwell I just tried
and the same with |
@s-alad Could you send the error that comes up? I'll take another look today and see what's happening |
@jarrodnorwell There's no error the function just doesn't return anything when the account I'm testing on has a following list. Returns an empty list
returns an empty list when it should be populatede |
public.py#L208-L210 has been modified to replace any unwanted
.com//.*
strings with.com/.*
user.py#L509 has been modified to use an up-to-date
query_hash
foruser_following_gql_chunk
user.py#L664 has been modified to use an up-to-date
query_hash
foruser_followers_gql_chunk
Fixes #1648, #1638, #1580, #1060, #864
Possible fix for #1050