Skip to content

Conversation

@sindrenm
Copy link
Contributor

No description provided.

@jgorset
Copy link
Contributor

jgorset commented Aug 21, 2014

Nice. I don't actually know how we survived without this.

@jgorset
Copy link
Contributor

jgorset commented Aug 21, 2014

We should add it to the view for videos, too, though.

@sindrenm
Copy link
Contributor Author

Haha, yeah. 😃

@sindrenm
Copy link
Contributor Author

Although the original reason I added this was to easily find it in an admin. Before you'd have to remember the video ID, which we usually don't.

@jgorset
Copy link
Contributor

jgorset commented Aug 21, 2014

You don't remember dQw4w9WgXcQ on YouTube?

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling a720203 on feature/add-title-to-videos into d161b65 on master.

@sindrenm
Copy link
Contributor Author

Yeah, I didn't add any specs for this …

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 3fc7295 on feature/add-title-to-videos into d161b65 on master.

@sindrenm
Copy link
Contributor Author

And @jgorset, is that the one with the cat?

jgorset added a commit that referenced this pull request Aug 22, 2014
@jgorset jgorset merged commit 94ca77a into master Aug 22, 2014
@jgorset jgorset deleted the feature/add-title-to-videos branch August 22, 2014 06:35
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.

4 participants