search

mozilla / aestimia

[Archived] Assessment tool
4 stars 10 forks source link

Allow arbitrary media types #55

Closed andrewhayward closed 11 years ago

andrewhayward commented 11 years ago

The word on the street is that learners in CSOL should be able to upload any file type for evidence. This means that Aestimia needs to be able to handle this too; it is currently restricted to image and link.

toolness commented 11 years ago

Hmm, what's wrong with just classfying any non-image as link, thereby allowing the mentor to download the evidence and peek at it? This allows any kind of file to be uploaded, since all evidence must be a URL in the first place.

The only argument against this is if we really want the evidence to appear embedded on the assessment page, as images currently are. But the only other embeddable type I can think of is video, though the video world is quite a mess. I suppose the Aestimia client can upload a HTML snippet containing embed code for the video (or anything else it wants to embed), though this will wreak a bit of havoc with Aestimia's default content security policy.

toolness commented 11 years ago

Perhaps I actually mis-named the mediaType property--it's not actually specifying the mime-type of the evidence, it's just communicating to Aestimia how the evidence will be embedded in the page, if at all (if it's not embedded, it's simply linked-out to, hence the link value). So perhaps I should have called it embeddingType instead of mediaType.

andrewhayward commented 11 years ago

Ah, right, OK; I misunderstood the point of link evidence.

Yes, this is fine - I'll send anything that isn't image/* through as a link, in that case.