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

Added Video (info) edit function #53

Merged
merged 4 commits into from Oct 30, 2021

Conversation

TentacleSama4254
Copy link
Contributor

@TentacleSama4254 TentacleSama4254 commented Oct 30, 2021

update() Simmilar` to the upload function, but insted of file Path the link to video is required.
Will throw error if video isn't owned by signed in account.

import {update} from 'youtube-videos-uploader' 
//or
const {update} = require ('youtube-videos-uploader') 

TO DO

  • fix error appearing at save sometimes
  • fix publishType error (public/private/unlisted)
  • better error handling
  • fix browser going to YouTube upload page before navigating to video
  • add example to readme
  • test if everything works

This change is Reviewable

@TentacleSama4254 TentacleSama4254 changed the title Video (info) edit function Added Video (info) edit function Oct 30, 2021
@fawazahmed0
Copy link
Owner

It was little harder to review the changes, because you have also formatted the code in this PR, other than that things are working great.
Thanks for this PR

@fawazahmed0 fawazahmed0 merged commit e39b89d into fawazahmed0:main Oct 30, 2021
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

2 participants