Allow uploading of Images only

[grvsachdeva]

Although, the goal of Mapknitter is clear that images should be uploaded to make the map but for to take care of our database and spam, we should limit uploading of only images.

I tried uploading mp4 file and to surprise no error was thrown. So, we need to fix the accept type of file.

Thanks!

[IshaGupta18]

I think changing this line:

mapknitter/app/views/images/_new.html.erb

Line 19 in de15628

<i class="fa fa-plus-circle"></i> Select images <input id="warpable_uploaded_data" name="warpable[uploaded_data]" type="file" multiple />

to this line is doing the trick:

<i class="fa fa-plus-circle"></i> Select images <input id="warpable_uploaded_data" name="warpable[uploaded_data]" type="file" accept="image/*" />

Only image file options will be displayed from the user's file system:

Should I make a PR for this @gauravano @jywarren ?

[grvsachdeva]

Hi @IshaGupta18, you have surely solved the issue. You can raise a PR or open a first-timer-issue too, however you like. Thanks 😃

[IshaGupta18]

I would love to open an FTO! Thanks a lot @gauravano !

[IshaGupta18]

Hey @gauravano I have created FTO #338 however I don't have the privileges I haave in plots2, like adding labels. Could you please grant me those privileges here as well or add fto label to it? Thanks a lot!

[grvsachdeva]

Thank you so much @IshaGupta18! I checked but I am not privileged to give you write permission required to add labels to the repo. But, let's ask @jywarren. For now, I am labeling the issue. Thanks!

[grvsachdeva]

Closing this one as FTO opened at #338. Thanks!