RedditVideoMakerBot icon indicating copy to clipboard operation
RedditVideoMakerBot copied to clipboard

Subscribe overlay to end of video

Open VirtualRabb opened this issue 2 years ago • 8 comments

Add subscribe overlay

Description

I've created a subscribe overlay for the end of the videos. It adds a overlay with a subscribe and hit the bell animation to the end of the video which can be seen in the video below

UPDATED LINK: https://youtube.com/shorts/6f6iomWNpaw?feature=share

New setting in config for enabling and disabling the overlay If it's disabled from the start it will not download or create the folders required to use the overlay

-- Choose between multiple overlays I've added a new setting within the config file, where you can choose between youtube, tiktok and instagram overlays!

-- Moved the add overlay method into the proper class I've the thing that composites the overlay onto the video from the final_video.py to video.py

It uses google drive to store the file for download

Checklist:

  • [X] I am pushing changes to the develop branch
  • [X] I am using the recommended development environment
  • [X] I have performed a self-review of my own code
  • [X] I have commented my code, particularly in hard-to-understand areas
  • [X] I have formatted and linted my code using python-black and pylint
  • [X] I have cleaned up unnecessary files
  • [X] My changes generate no new warnings
  • [X] My changes follow the existing code-style
  • [X] My changes are relevant to the project

Any other information (e.g how to test the changes)

Enable the sub_overlay in config or on startup.

VirtualRabb avatar Aug 17 '22 19:08 VirtualRabb

Awesome! But can you make that the user can select other clips easier? Because the videos can also be uploaded to TikTok and Instagram reels

OpenSourceSimon avatar Aug 17 '22 19:08 OpenSourceSimon

Awesome! But can you make that the user can select other clips easier? Because the videos can also be uploaded to TikTok and Instagram reels

Sounds like an great idea! I'll see if I can get that implemented! Thank you the suggestion!

VirtualRabb avatar Aug 17 '22 20:08 VirtualRabb

Awesome! But can you make that the user can select other clips easier? Because the videos can also be uploaded to TikTok and Instagram reels

Sounds like an great idea! I'll see if I can get that implemented! Thank you the suggestion!

I will review once that is implemented

JasonLovesDoggo avatar Aug 17 '22 20:08 JasonLovesDoggo

Perfect! One little idea, maybe upload the files to this repository. Because the Google Drive links can be edited or removed. This is just a suggestion, you don't have to do it.

OpenSourceSimon avatar Aug 18 '22 07:08 OpenSourceSimon

I already have a version of this implemented, and the best thing I recommend is to make it simpler because my version only needs a change to final_video.py plus the .env. It feels like a lot to not just download every gif on startup since they aren't massive files.

PixelNinja132 avatar Aug 18 '22 11:08 PixelNinja132

Perfect! One little idea, maybe upload the files to this repository. Because the Google Drive links can be edited or removed. This is just a suggestion, you don't have to do it.

@VirtualRabb I would 100% say to do this

JasonLovesDoggo avatar Aug 18 '22 17:08 JasonLovesDoggo

Perfect! One little idea, maybe upload the files to this repository. Because the Google Drive links can be edited or removed. This is just a suggestion, you don't have to do it.

@VirtualRabb I would 100% say to do this

It says the files are to big.-.

VirtualRabb avatar Aug 18 '22 18:08 VirtualRabb

It says the files are to big.-.

Did you upload with GitHub web or git? Git supports larger files

OpenSourceSimon avatar Aug 18 '22 18:08 OpenSourceSimon

It says the files are to big.-.

Did you upload with GitHub web or git? Git supports larger files

Sorry for being so slow, had work this entire weekend. I've added them in through the github desktop. So now they should be in :d

VirtualRabb avatar Aug 22 '22 12:08 VirtualRabb

Sorry for being so slow, had work this entire weekend. I've added them in through the github desktop. So now they should be in :d

Awesome! But can you check why there are merge conflicts?

OpenSourceSimon avatar Aug 22 '22 12:08 OpenSourceSimon

Sorry for being so slow, had work this entire weekend. I've added them in through the github desktop. So now they should be in :d

Awesome! But can you check why there are merge conflicts?

If i could figure that out i would, I've tried everything I know, kinda just wanna give up on that now can't find any conflicts so I dunno what to do now-.-

VirtualRabb avatar Aug 22 '22 13:08 VirtualRabb

If i could figure that out i would, I've tried everything I know, kinda just wanna give up on that now can't find any conflicts so I dunno what to do now-.-

Don't give up, it's a good pull request! Do you have git? Then you can run the command git status and see what is the problem.

OpenSourceSimon avatar Aug 22 '22 14:08 OpenSourceSimon

If it's too hard to fix it (which I understand), you can always make a new branch and a new pull request.

OpenSourceSimon avatar Aug 22 '22 14:08 OpenSourceSimon

If it's too hard to fix it (which I understand), you can always make a new branch and a new pull request.

Yea I'll do that :d

VirtualRabb avatar Aug 22 '22 14:08 VirtualRabb