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

Adding video statistics functionality #50

Open
wants to merge 63 commits into
base: master
Choose a base branch
from

Conversation

Colum31
Copy link

@Colum31 Colum31 commented Apr 12, 2022

I have added a function to get statistics for a video with it's id.
It works very similar to its channel statistics counterpart.

I also added an example sketch and Arduino IDE keywords.
I plan to extend the library further.

Regards
Daniel

-> able to fetch video statistics by id
-> new datatype to hold information
-> adding example
-> adding keywords
-> adding struct
-> adding functions to fill structs
-> also added some error checking and documentation
-> although not final, library now able to fetch important informations
about videos (title, description, publish date etc.)
-> getSnippet would exit, when language informations are not set
-> it is now easier to implement more functionality
-> api calls are handled by a different calling and parsing function
-> new example prints all information about a video, that the library is
able to fetch
-> setting default baud rate to 115200 in platformio config file
-> adding development files to gitignore
-> example code did not print correct data
-> adjusted buffersizes in order to load long descriptions and content
details more efficient
-> adding struct and functions to fetch video.list:status
-> adding status information to example
-> altering doc to show which attributes are not implemented yet
-> adjusting keywords
-> adding new features to README
-> adjusting variable names to avoid conflicts in future
@Colum31
Copy link
Author

Colum31 commented Apr 22, 2022

I have added more functionality, the library is now able to fetch almost all video.list informations.
I have also rewritten the code, to make it more expandable.

The "flow" in the code is as follows:

public get function -> private getRequestedType -> private parsing function

This way, one must only write a parsing function, in order to expand the functionality further. (see commit 154894e)
I am sorry for the size of the PR, @witnessmenow .

Regards Daniel

-> liked and uploaded videos as playlist link
splitting channel related functionality into seperate class
-> a new test case checks url generation
-> refactoring url generation
-> adding documentation
-> untested and not yet implemented
-> can fetch first page of playlistItems
adding support for Playlists
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.

None yet

1 participant