-
Notifications
You must be signed in to change notification settings - Fork 18
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
Implement support for further common file types #6
Comments
Add support for png, jpg, txt and gif #6
@aspark21 I've merged your pull request to add: png, jpg, txt and gif. Also you can take the office archive from the repo and uncompress it locally (on linux), then run the libreOffice executable locally to test the conversion, the rough commands are: Cheers, |
Hi Matt, I have been testing file types on our Moodle 3.7 test server. Regards |
Hi Matt, Thanks for taking in my commit. David - @cybernotic - has being doing the testing for the one's currently declared by you & in my additions. The full list is occcurences of file types which unoconv supports but which haven't yet been declared in librelambda - i.e. pending testing shows they work. The HTML one is an interesting case though because if someone saves a website with all the assets and submitted that to the Moodle assignment the assets would not be passed along to the converter so the page is doomed to be rendered horribly. I'm not sure of use cases but seems maybe more likely that an html document being graded would be in computer science in which case viewing the underlying code - i.e. as a text document - would make more sense than as a rendered web page. (?) David - if you could take a quick look through submissions where html was submitted and figure out why, then maybe would guide us between both options. Think the other file types would be nice to test but we're going to be tied up until August so may come back to these after our upgrade & go-live with this. Thanks |
Would be great if jpg, txt and png could be supported by this plugin.
We are now using the onedrive fileconverter as our primary but it doesn't support everything.
I've listed below the filetypes not supported by librelambda which unoconv supports and number of files in our system as a representation of proportions
jpg | 4098
txt | 775
png | 616
gif | 13
tiff | 7
xml | 3
bmp | 2
emf | 2
eps | 2
svg | 1
wps | 1
These are filetypes Onedrive supports which librelambda doesn't
csv | 96
odp | 17
pptm | 12
ppsx | 6
potx | 4
Also can take a look at https://bitbucket.org/uonmoodle/moodle-fileconverter_onedrive/issues/1/max-file-size-supported-is-4mb-should#comment-50552953 for a full picture
The text was updated successfully, but these errors were encountered: