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

V5.0.0 #803

Merged
merged 18 commits into from
Feb 14, 2022
Merged

V5.0.0 #803

merged 18 commits into from
Feb 14, 2022

Conversation

davidteather
Copy link
Owner

@davidteather davidteather commented Jan 24, 2022 •
edited
Loading

IMPORTANT CHANGES

  • Please read the upgrading from V4 to V5 section on the readme
    • Higher level approach using objects
    • Auto-parsing of attributes and responses back into objects
    • Should be easier to work with (especially for new-comers not as familiar with this package)
  • dropped support for selenium (sorry but it's been too annoying to main multiple browsers)

TODO

  • None

DONE

  • New YouTube video tutorial for v5
  • refactor kwargs stuff so it's not as bad as it is right now
  • make sure everything is typed and valid
  • write a section in the api modules about what different keys mean and their types
  • split hashtag, video, sound, and user into their own classes
    • creates a higher level feel of the package
  • switch to generators for all the methods
  • update tests
  • improve logging
  • Write an upgrading to v5 section in the readme
  • Add more doc strings to the methods everywhere
  • update readme to reflect new changes
  • update issue templates
  • change process_kwargs
  • try to minimize amount of instance variables on TikTokApi for cleanliness or at least use private instance variables

V5 Example Usage

from TikTokApi import TikTokApi

verify_fp = "verify_xxx"
api = TikTokApi(custom_verify_fp=verify_fp)

tag = api.hashtag(name="funny")
print(tag.info())

for video in tag.videos():
    print(video.id)
    for u_vid in video.author.videos():
        print(u_vid.id)

davidteather and others added 18 commits January 21, 2022 00:27
@davidteather @Daan-Grashoff
Initial changes for v5 of the project.
2076ac7 
Initial changes for v5 of the project.

Thanks @Daan-Grashoff for some of the helpers.extract_tag_contents code from a comment on #787

Co-Authored-By: Daan Grashoff <9222025+Daan-Grashoff@users.noreply.github.com>
@davidteather
finish separating classes
a505538
@davidteather
Finish higher level modeling of objects
7905b35
@davidteather
update tests & auto call object
427fca6
@davidteather
Fix package-test
3a13796
@davidteather
Bump version
ca2eabf
@davidteather
simplify info_full
a2274a4
@davidteather @zokalo
Logging Improvements
5bb0a64 
Logging changes from #804 by @zokalo

Co-Authored-By: Dmitriy <16061619+zokalo@users.noreply.github.com>
@davidteather
I forgot to save one file on the previous commit
c168a78
@davidteather
Improve docs
ad52e51
@davidteather
Fix some mypy typing issues
a5787ae
@davidteather
Fix typing
85783ac
@davidteather
Investigated discover_type, still doing work on it though
df4eb23
@davidteather
Some TODOS
16b870c
@davidteather
Merge branch 'v5.0.0' of https://github.com/davidteather/TikTok-Api i…
5a0cf57 
…nto v5.0.0
@davidteather
Move old search functions to trending & fix process_kwargs
a6c1dd9
@davidteather
Private self variables
b2ce6d2
@davidteather
Yeahhhh
46c964f
@davidteather davidteather changed the title [WIP] V5.0.0 V5.0.0 Feb 14, 2022
@davidteather davidteather merged commit 3973da6 into master Feb 14, 2022
dwoffinden added a commit to dwoffinden/tiktok-rss-flat that referenced this pull request Feb 14, 2022
@dwoffinden
Update for v5 of TikTokApi
bc47116 
davidteather/TikTok-Api#803
dwoffinden added a commit to dwoffinden/tiktok-rss-flat that referenced this pull request Feb 14, 2022
@dwoffinden
Update for v5 of TikTokApi
0de2fa2 
davidteather/TikTok-Api#803
dwoffinden added a commit to dwoffinden/tiktok-rss-flat that referenced this pull request Feb 14, 2022
@dwoffinden
Update for v5 of TikTokApi
383d36f 
davidteather/TikTok-Api#803
@dwoffinden dwoffinden mentioned this pull request Feb 14, 2022
Merged
dwoffinden added a commit to dwoffinden/tiktok-rss-flat that referenced this pull request Feb 14, 2022
@dwoffinden
Update for v5 of TikTokApi
8810b6a 
See davidteather/TikTok-Api#803

Fixes conoro#14
@estatistics
Copy link

how to download a user's videos; can you provide an example?

3garbilx
3garbilx reviewed Mar 12, 2023
View reviewed changes
Copy link

@3garbilx 3garbilx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

README.md

3garbilx
3garbilx reviewed Mar 12, 2023
View reviewed changes
tests/test_user.py
@@ -1,52 +1,41 @@
from TikTokApi import TikTokApi
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1

tests/test_user.py
@@ -1,52 +1,41 @@
from TikTokApi import TikTokApi
import os
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

2

3garbilx
3garbilx reviewed Mar 12, 2023
View reviewed changes
Copy link

@3garbilx 3garbilx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

72

3garbilx
3garbilx reviewed Mar 12, 2023
View reviewed changes
Copy link

@3garbilx 3garbilx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

gh pr checkout 803

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@3garbilx 3garbilx 3garbilx left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@davidteather @estatistics @3garbilx