Prevent AIA files from being uploaded to projects as media assets about appinventor-sources HOT 16 CLOSED

NIce point. People also upload PDFs and DOC files. We ought to trap for
these. Any interest in writing the code for this?

== Hal Abelson
[email protected]
Prof. of Comp. Sci. and Eng.
MIT Dept. of Elec. Eng. and Comp. Sci.
MIT Media Lab, Second Class Faculty

On Mon, Dec 8, 2014 at 5:28 PM, Mark Sherman [email protected]
wrote:

It is possible for beginners who are trying to load a template AIA source
file to upload them into an existing project as media assets. Beginners may
be susceptible to this mistake if they had recently learned how to upload
media assets, which is commonly a 1st or 2nd session event.

This mistake was observed during a teacher training.

I propose a feature that checks media uploads. If the user attempts to
upload the AIA, refuse, and guide the user towards a web page describing
how to use AIA files.

—
Reply to this email directly or view it on GitHub
#173.

from appinventor-sources.

It actually might be useful to upload a PDF in some circumstances. When I found DOC and PDF files in the system, it was usually the result of someone unziping a project file into a directory with other stuff and then re-zipping it up and the other stuff gets included :-)

-Jeff

from appinventor-sources.

P.S. I can easily put in a check for files ending in ".aia" and generate an appropriate error.

from appinventor-sources.

If Jeff makes the error, I'll write the explanation page to go with it.
On Mon, Dec 8, 2014 at 10:20 PM Jeffrey Schiller [email protected]
wrote:

P.S. I can easily put in a check for files ending in ".aia" and generate
an appropriate error.

—
Reply to this email directly or view it on GitHub
https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_mit-2Dcml_appinventor-2Dsources_issues_173-23issuecomment-2D66229087&d=AAMCaQ&c=TqceLeU8_c9CVUat-eLTFu19-lyGiWSvB_skZXlxt8Q&r=k-tN9RiDS5X8a15wxR_FMzWjUtK1yw-seWE6tKapqes&m=xdDXwuCszhbvBj_-rlKeVt0D0-eZmbIVejWqXJS0qYg&s=R1A6bEKFXP1tb04FOAlFLWVZjBZK8Nj1pHe1JLuw9xs&e=
.

from appinventor-sources.

@jisqyv
I can put this check in, but I don't know where to put it. Somewhere in FileUploadWizard??

from appinventor-sources.

@jisqyv have just removed you from this one; marking as help needed for easy fixes to get started with the sources.

from appinventor-sources.

Hi, my name is Varun! I'm a new UROP student with the App Inventor team.
@bxie assigned this to me as a starter project
I'll prevent AIA files from being uploaded as media assets.

from appinventor-sources.

I have added an appropriate condition to check if the file being uploaded is an aia file in the FileUploadWizard class. I have also added an appropriate error message in OdeMessages class and will soon sent a pull request, thanks.

from appinventor-sources.

I'll check it out this week, @wadleo (though I have no pull power, it seems right that I give it a test!)

from appinventor-sources.

thanks @marksherman, waiting for your reply.

from appinventor-sources.

@wadleo : this may be redundant work.
@varunm22 has been working on this over the past month and his work is almost ready to be merged (currently working on some UI fixes).

@varunm22 please confirm

from appinventor-sources.

Hello @bxie,
sorry i didn't realise he is still working on this looking at the dates above and please can you provide a link to his work, thanks for the info.

from appinventor-sources.

Hi @wadleo my fixes are on the bytesize branch of my fork at https://github.com/varunm22/appinventor-sources/tree/bytesize. In addition to adding the .aia fix, I've replaced the window.alerts that show up during file upload errors with dialog boxes for greater consistency.

from appinventor-sources.

Hi @varunm22, thanks for the feedback.

from appinventor-sources.

Hi @wadleo, sorry for the confusion, and please let me know if you have any more questions!

from appinventor-sources.

Closed by #747

from appinventor-sources.