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

Vk-youtube-connect #132

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

Vk-youtube-connect #132

wants to merge 6 commits into from

Conversation

vikram-kher
Copy link
Contributor

Created Route:

Route Details

  • POST request, takes in name, email, and URI of video

When route is hit, oauth2 authorization occurs. If this is successful, then the video located at the URI is uploaded the the gladeoapp youtube channel.

Notes:

  • I reached the daily api quota limit for uploading youtube videos, so I couldn't fully test the software. I was able to successfully upload videos to youtube though. I'd thought it would be better to upload sooner though so you have an opporuntity to review the code.

  • Since I have a windows computer, I tested the program using a random mov file that I placed in the main directory of the program.

  • Inorder for the authentication to work, you need to have the following file in your main directory "client_secret.json". I have uploaded this file to the api docs.

@vikram-kher vikram-kher requested a review from apetranik May 15, 2020 00:58
Copy link
Member

@apetranik apetranik left a comment

Choose a reason for hiding this comment

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

good work. one small thing about the email password.

user: 'INSERT SENDER EMAIL HERE',
pass: 'INSERT SENDER EMAIL PASSWORD HERE'
user: '[email protected]',
pass: 'Abq12Yx34z!8'
Copy link
Member

Choose a reason for hiding this comment

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

make this a .env variable

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