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

Create a script that adds meetup videos to the website #183

Closed
wants to merge 1 commit into from

Conversation

egiurleo
Copy link
Contributor

@egiurleo egiurleo commented Aug 7, 2023

Historically, the process of adding meetup videos to our website has not been documented anywhere. In writing this script, my goal is to automate the process so anybody can do it without having to know all of our database models beforehand. This could one day be adapted into some admin forms eventually, but I want the process to be codified before I go on parental leave.

How to run:

On Heroku:

heroku run rails runner scripts/upload_meetup_talk.rb -a wnb-rb

Locally:

rails runner scripts/upload_meetup_talk.rb

Historically, the process of adding meetup videos to our website has not
been documented anywhere. In writing this script, my goal is to automate
the process so anybody can do it without having to know all of our
database models beforehand. This could one day be adapted into some
admin forms.
@luciagirasoles
Copy link
Collaborator

Hello, @egiurleo On this [PR] (#182), I've been working that adds the speaker model, controller and views. It will undoubtedly conflict in some manner when it updates our Speaker model. So, is there a way we could add some tests for this script?, or even create an issue to handle the tests so we could remember it.

@egiurleo
Copy link
Contributor Author

@luciagirasoles Oh wow 😍 You rock. Yes, I can write tests this weekend!

@luciagirasoles
Copy link
Collaborator

Hi, @egiurleo! Please let us know whether this PR is still in process, if the present admin site is sufficient, or if there is something lacking from this script.

@egiurleo
Copy link
Contributor Author

Yeah I'm just going to close this. Thanks for checking @luciagirasoles!

@egiurleo egiurleo closed this Feb 13, 2024
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