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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add 3 new movie urls #363

Merged
merged 2 commits into from
Apr 21, 2016
Merged

Add 3 new movie urls #363

merged 2 commits into from
Apr 21, 2016

Conversation

shoheikawano
Copy link
Contributor

Related Issue: #334

Done

I added 3 new movie urls: #334 (comment) (finally 馃檱 )

@@ -984,7 +984,7 @@
},
"language_id": "ja",
"slide_url": "https://speakerdeck.com/player/45c463463fbe47518576240a53622cdc",
"movie_url": "https://www.youtube.com/watch?v=GMAXZPndBo0",
"movie_url": "https://www.youtube.com/watch?v=lmti0rNxbmI",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'm guessing when I added movie urls for the first time I added the wrong one, so here I modified it to add the right one.

@konifar
Copy link
Owner

konifar commented Apr 20, 2016

馃憖

@konifar konifar added this to the 1.2.0 milestone Apr 20, 2016
@konifar
Copy link
Owner

konifar commented Apr 20, 2016

@shaunkawano Thanks!!
And sorry, it's CI setting matter. I'll check and fix this after work.

@shoheikawano
Copy link
Contributor Author

@konifar Thanks for the review!

Yes, I realized that Circle CI failed due to this: http://stackoverflow.com/questions/29063968/plugin-is-too-old-please-update-to-a-more-recent-version-or-set-android-daily; I'm guessing it should be fixed by updating Gradle verson. http://stackoverflow.com/a/29064084/5449688

@konifar
Copy link
Owner

konifar commented Apr 20, 2016

@shaunkawano Oh, thanks. I appreciate that you fix this if you have a time 馃檱

@konifar
Copy link
Owner

konifar commented Apr 20, 2016

Could you merge master to this branch?

@shoheikawano
Copy link
Contributor Author

馃槃 Circle CI check successful!

@konifar
Copy link
Owner

konifar commented Apr 21, 2016

Thanks so much!!

@konifar konifar merged commit 0dee149 into konifar:master Apr 21, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants