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.

Sign up for GitHub

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

Jump to bottom

YouTube Video Rankings & Results #31

Open
drewroberts opened this issue Feb 26, 2021 · 2 comments
Open

YouTube Video Rankings & Results #31

drewroberts opened this issue Feb 26, 2021 · 2 comments
Projects
SEO Tracking

Comments

@drewroberts
Copy link
Member

Results (#25) should be extended to include Video Listings. I'm trying to decide if these should be a new model that has a nullable relationship with Videos (drewroberts/media#2) or if the Webpages (#19) model can be updated to accommodate it.
drewroberts added a commit that referenced this issue Feb 26, 2021
@drewroberts
Require media package for #13 #14 #15 #31
a2f4613
drewroberts added a commit that referenced this issue Feb 27, 2021
@drewroberts
Add webpage reference to company video #19 #31
589402e
drewroberts added a commit that referenced this issue Feb 27, 2021
@drewroberts
Merge pull request #33 from tipoff/drewroberts/feature/31-youtube-videos
ad3c912 
Company Video Rankings #31
@devjk1
Copy link
Contributor

devjk1 commented Mar 5, 2021 •
edited
Loading

If you do decide to create a Video Model, we can use the a polymorphic relationship between Video hasMany Result. I did this with Webpage/Place hasMany Result #41

@drewroberts
Copy link
Member Author

The Video model is in drewroberts/media but I'm still thinking about if I want to use it. I think I prefer using Webpages

@alliroberts alliroberts added this to In progress in SEO Tracking Mar 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
SEO Tracking
  
In progress
Milestone
No milestone
Development

No branches or pull requests
2 participants
@drewroberts @devjk1