Skip to content

Commit

Permalink
fix oembeds on assignment review tool
Browse files Browse the repository at this point in the history
  • Loading branch information
mitchelljkotler committed Jun 4, 2018
1 parent ca9c153 commit 24fb10e
Show file tree
Hide file tree
Showing 6 changed files with 43 additions and 12 deletions.
29 changes: 21 additions & 8 deletions muckrock/assets/js/crowdsource.js
Expand Up @@ -2,7 +2,6 @@
**
*/

import Embedo from 'embedo';

$(document).ready(function(){
var formBuilder = $("#build-wrap").formBuilder({
Expand Down Expand Up @@ -96,8 +95,6 @@ $(document).ready(function(){
flag = null,
search = "";

var embedo = new Embedo();

function handleUpdateResponses(data) {
var response, values, dataValues, dataUrlP, oEmbed, flagged, galleried, tags;
var responses = $("section.assignment-responses");
Expand All @@ -110,7 +107,7 @@ $(document).ready(function(){
dataUrlP = `<p>Data:
<a href="${data.results[i].data}">${data.results[i].data}</a>
</p>`;
oEmbed = `<div class="embedo" data-url="${data.results[i].data}"></div>`;
oEmbed = `<div class="embed" data-url="${data.results[i].data}"></div>`;
} else {
dataUrlP = "";
oEmbed = "";
Expand Down Expand Up @@ -173,8 +170,16 @@ $(document).ready(function(){
});
});
if ($("#data-inline").prop("checked")) {
$(".embedo").each(function(){
embedo.load(this, $(this).data('url'));
$(".embed").each(function(){
var embed = $(this);
$.ajax({
url: "/assignment/oembed/",
type: "GET",
data: {"url": $(this).data('url')},
success: function(data) {
embed.html(data);
}
});
});
}

Expand Down Expand Up @@ -233,8 +238,16 @@ $(document).ready(function(){

$("#data-inline").change(function() {
if ($(this).prop("checked")) {
$(".embedo").each(function(){
embedo.load(this, $(this).data('url'));
$(".embed").each(function(){
var embed = $(this);
$.ajax({
url: "/assignment/oembed/",
type: "GET",
data: {"url": $(this).data('url')},
success: function(data) {
embed.html(data);
}
});
});
}
});
Expand Down
1 change: 0 additions & 1 deletion muckrock/assets/js/muckrock.js
Expand Up @@ -16,7 +16,6 @@ import 'jquery-ui/widget';
import 'jquery-ui/sortable';
import 'formBuilder';
import 'tooltipster';
import 'embedo';
import '../vendor/formset';
import '../vendor/loupe';
import '../vendor/quicksearch';
Expand Down
2 changes: 1 addition & 1 deletion muckrock/assets/scss/view/_crowdsource.scss
Expand Up @@ -36,7 +36,7 @@
.textbox__section {
flex: 1 1 50%;
}
.embedo {
.embed {
flex: 1 1 50%;
}
}
5 changes: 5 additions & 0 deletions muckrock/crowdsource/urls.py
Expand Up @@ -55,4 +55,9 @@
views.CrowdsourceCreateView.as_view(),
name='crowdsource-create',
),
url(
r'^oembed/$',
views.oembed,
name='crowdsource-oembed',
),
]
17 changes: 16 additions & 1 deletion muckrock/crowdsource/views.py
Expand Up @@ -6,7 +6,12 @@
from django.contrib.auth.decorators import login_required
from django.contrib.auth.mixins import PermissionRequiredMixin
from django.core.urlresolvers import reverse
from django.http import Http404, StreamingHttpResponse
from django.http import (
Http404,
HttpResponse,
HttpResponseBadRequest,
StreamingHttpResponse,
)
from django.shortcuts import redirect
from django.utils.decorators import method_decorator
from django.utils.text import slugify
Expand Down Expand Up @@ -40,6 +45,7 @@
)
from muckrock.crowdsource.models import (
Crowdsource,
CrowdsourceData,
CrowdsourceResponse,
CrowdsourceValue,
)
Expand Down Expand Up @@ -510,3 +516,12 @@ def form_valid(self, form):
)
messages.success(self.request, msg)
return redirect(crowdsource)


def oembed(request):
"""AJAX view to get oembed data"""
if 'url' in request.GET:
data = CrowdsourceData(url=request.GET['url'])
return HttpResponse(data.embed())
else:
return HttpResponseBadRequest()
1 change: 0 additions & 1 deletion package.json
Expand Up @@ -52,7 +52,6 @@
"dependencies": {
"autonumeric": "^1.9.43",
"axios": "^0.13.1",
"embedo": "^1.7.3",
"fine-uploader": "5.14.1",
"formBuilder": "^2.9.8",
"jquery-ui": "1.10.x",
Expand Down

0 comments on commit 24fb10e

Please sign in to comment.