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

Fix error 403 for blocked or old user agent #49

Closed
wants to merge 17 commits into from
Closed

Fix error 403 for blocked or old user agent #49

wants to merge 17 commits into from

Conversation

Dielee
Copy link
Collaborator

@Dielee Dielee commented Jun 15, 2022

No description provided.

@Dielee Dielee mentioned this pull request Jun 15, 2022
@MaxWinterstein
Copy link
Owner

This is so cool! I love the google-play-scraper part 😄

Can you give it a quick rebase and a little squash? Would love to merge this - and keep your credits for the great work!

Dielee and others added 4 commits June 17, 2022 08:08
…google playstore

Add new requirements

Add new requirements to pyproject.toml

Catch error if old token file exists

Move info logs to info log level

Add token version check at startup

Optimize version check

[pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

Optimize version check

[pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

Optimize random UA generation

[pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

Add autoupdate UA if new tgtg version was released

[pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci
@Dielee
Copy link
Collaborator Author

Dielee commented Jun 17, 2022

So, rebased and squashed in 51267de. Ready to merge!

@MaxWinterstein
Copy link
Owner

@Dielee Still not mergeable :(

Not sure what happened here, but there are still 17 commits to be applied. Might have a second look?

@Dielee
Copy link
Collaborator Author

Dielee commented Jun 17, 2022

I will close this PR and fetch upstream source. Wait a minute.

@Dielee Dielee closed this Jun 17, 2022
@MaxWinterstein
Copy link
Owner

Continued in PR #51

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.

2 participants