Galapagos icon indicating copy to clipboard operation
Galapagos copied to clipboard

Support saving files to the server

Open TheBizzle opened this issue 11 years ago • 8 comments

I believe that this will also require user accounts and permissioning (since many people seem hesitant about making their work-in-progress models available to the public). Or maybe the initial draft of this should just upload all files into one central pool...?

We should also think about restrictions for what can and can't be uploaded. We don't really want people uploading any old thing they feel like putting on the server. What should the size limit be? Do we need to check that the file is a valid '.nlogo' file? Should we verify that the model passes the compiler before accepting it? (Probably not.)

TheBizzle avatar Feb 04 '15 18:02 TheBizzle

Would it be acceptable to use localStorage instead? That would allow users to keep the file on their computer so that they didn't have to reopen it every time without making us liable for storage (and backup and security and authorization) of user data. We could even make something that shows users all the Tortoise files they've used up to this point, making switching between them really easy.

mrerrormessage avatar Feb 05 '15 15:02 mrerrormessage

I think localStorage plus "upload" and "download" buttons (that really just puts the file in the browser/takes it out of) to allow transferring files between computers would be sufficient, especially in the short term. localStorage is super easy to work with.

qiemem avatar Feb 05 '15 16:02 qiemem

That works for me.

TheBizzle avatar Feb 05 '15 16:02 TheBizzle

Sounds great. Are you thinking about adding versioning support (undo, revert to point X, etc) at some point, or do you see this as something separate from the tool?

hirokiterashima avatar Feb 05 '15 16:02 hirokiterashima

It would definitely be good to have user file versioning at some point, but I think that can wait. And, fortunately, the Code tab that currently exists for Tortoise authoring already supports "undo" operations.

TheBizzle avatar Feb 05 '15 16:02 TheBizzle

how about working with @reuven to keep the models on the Modeling Commons?

SethTisue avatar Feb 05 '15 18:02 SethTisue

Good idea @SethTisue.

I think we should have the localStorage thing in place no matter what, just as a convenience. It lets you do "starts up where left off" and such things easily.

qiemem avatar Feb 05 '15 18:02 qiemem

There are a lot of good long-term ideas here. I think that we would be wise to keep them in mind going forward, but, for now, what we need the most is a basic solution that allows people to save and open code that they wrote in Tortoise. Consequently, I've split the local storage stuff off into a separate ticket (#193) that should be given priority.

TheBizzle avatar Feb 05 '15 19:02 TheBizzle