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

embed video in task description/ instruction #738

Closed
wants to merge 5 commits into from

Conversation

yiqingj
Copy link

@yiqingj yiqingj commented Jan 26, 2016

for issue #721
this is to enable embedding video/image into task description/instruction by adding an oEmbed extension to python markdown library.

this solution is coming from https://github.com/rennat/python-markdown-oembed with a few bug fixes

a sample markdown for Youtube:

![video](http://www.youtube.com/watch?v=StTqXEQ2l-Y)

===== update======
also support the preview tab by updating showdown version and adding a youtube extension.

@pyrog I just noticed that PR #527 also try to update showdown version and make all markdown client rendered. if that's the case we could just add the js youtube extension in my PR and remove the python markdown extension. Do a cherrypick or just let me know after you merge #572 so I could update my PR accordingly.

@dalekunce
Copy link
Contributor

Brilliant!!

@pgiraud
Copy link
Collaborator

pgiraud commented Jan 27, 2016

Thanks for this nice contribution!
I'll review this as soon as possible. Don't hesitate to hunt if I forget.

@pgiraud
Copy link
Collaborator

pgiraud commented Jan 28, 2016

Your code looks nice. Can you please squash your commits into one?

@yiqingj
Copy link
Author

yiqingj commented Jan 29, 2016

created PR #739 with one commit for all changes as requested by @pgiraud. closing this one.

@yiqingj yiqingj closed this Jan 29, 2016
@pgiraud
Copy link
Collaborator

pgiraud commented Jan 30, 2016

You didn't have to create a new pull request for that. You can squash/fixup commits and push force again your branch to the remote. It's worth to learn this nice feature. You'll find dedicated tutorials.

@aawiseman
Copy link

Hey guys, is this live yet? I tried it on task 1460 but it didn't work, I just got a missing image type box. Thanks.

@pgiraud
Copy link
Collaborator

pgiraud commented Feb 3, 2016

Hi @aawiseman. Sorry, it's not live. I expect to do a release soon and get it on production.

@aawiseman
Copy link

@pgiraud ah thanks. Do you know when roughly? Just curious. Thanks!

@aawiseman
Copy link

@pgiraud hi Pierre, it still looks like it's not working, I just see a little box with a question mark. I'm looking at http://tasks.hotosm.org/project/1460 under the Instructions tab. Is the new version not live yet? Thanks!

@pgiraud
Copy link
Collaborator

pgiraud commented Feb 9, 2016

@aawiseman sorry it's still not live. The release process takes a bit of time. Hopefully, we should be able to get this working live in the next days.
However, you can already test it on the dev instance.

@aawiseman
Copy link

@pgiraud Got it, thanks!

@pgiraud
Copy link
Collaborator

pgiraud commented Feb 24, 2016

@aawiseman We finally managed to deploy the 2.12 release! It's now live.

@aawiseman
Copy link

@pgiraud woohoo! Thank you, it looks great!

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