Image-sequencer: Restrict upload of files other than images(jpeg,png etc)

Created on 1 Oct 2018  路  8Comments  路  Source: publiclab/image-sequencer

In image-sequencer only image type files types must be uploaded,all other file types like pdf, JS, .md files must be ignored/rejected.
while uploading a gif the uploaded gif is broken. please refer to the screenshots/gif attached.

screen shot 2018-10-01 at 7 46 32 pm
20181001-074034
screen shot 2018-10-01 at 7 44 56 pm

What to be done : -
while uploading restrict all the files other than image format files.This is the best fix for this issue.

bug help wanted

All 8 comments

GitMate.io thinks the contributor most likely able to help you is @jywarren.

Possibly related issues are https://github.com/publiclab/image-sequencer/issues/356 ('No file chosen' wrongly placed in Import Image module), https://github.com/publiclab/image-sequencer/issues/31 (Issue with non-JPEG images also affecting alpha channel), and https://github.com/publiclab/image-sequencer/issues/5 (image cropping module).

GitMate.io thinks the contributor most likely able to help you is @jywarren.

Possibly related issues are https://github.com/publiclab/image-sequencer/issues/356 ('No file chosen' wrongly placed in Import Image module), https://github.com/publiclab/image-sequencer/issues/31 (Issue with non-JPEG images also affecting alpha channel), and https://github.com/publiclab/image-sequencer/issues/5 (image cropping module).

@vsk4 great catch

@avsingh999 can I be assigned this? I would like to work on this

@avsingh999 please review.

Hey @jywarren can I start working on this if this issue is not resolved?

I think it's complete, @aashna27 can you confirm? Thanks!!!

Yes it is, the pr might not have referenced this issue. But it has been done. Sorry for the confusion.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

Divy123 picture Divy123  路  5Comments

harshkhandeparkar picture harshkhandeparkar  路  4Comments

harshkhandeparkar picture harshkhandeparkar  路  4Comments

jywarren picture jywarren  路  4Comments

harshkhandeparkar picture harshkhandeparkar  路  5Comments