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

[TODO] - Make more API calls, load less pages #21

Open
Russell-Newton opened this issue Jan 21, 2023 · 1 comment
Open

[TODO] - Make more API calls, load less pages #21

Russell-Newton opened this issue Jan 21, 2023 · 1 comment
Labels
enhancement New feature or request

Comments

@Russell-Newton
Copy link
Owner

At the moment, TikTokPy loads TikTok's website to grab data. This circumvents the need for a verify_fp cookie, but it increases runtime and hinders scalability.

Changes could be made such that API calls are made directly instead of loading pages. A page should only be loaded completely when the API call doesn't work. This could reload the cookies necessary to make future API calls.

Steps towards accomplishing goal:

  1. Identify TikTok API calls necessary to get the information currently expected by TikTokPy
  2. Migrate TikTokPy calls to using API calls unless overridden manually
  3. Create automatic override determined by cookie staleness
@Russell-Newton Russell-Newton added the enhancement New feature or request label Jan 21, 2023
@Russell-Newton Russell-Newton mentioned this issue Feb 17, 2023
Closed
@Russell-Newton
Copy link
Owner Author

Likely to present an additional solution to #38 and #40.

May solve #23 and #30.

@Russell-Newton Russell-Newton mentioned this issue May 31, 2023
Closed
Russell-Newton added a commit that referenced this issue Jun 15, 2023
@Russell-Newton
#21 - Initial API calls setup
417859d 
* Create functions for executing API calls of 4 kinds:
    * comment/list/ - video comments
    * post/item_list/ - user posts
    * challenge/item_list/ - popular videos tagged with a challenge
    * related/item_list/ - videos related to this one

* Opens up potential future resolutions for #35, #38, #40, #43, and #44
@Russell-Newton Russell-Newton mentioned this issue Jul 5, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@Russell-Newton