smogon / pokemon-showdown

Pokémon battle simulator.
https://pokemonshowdown.com
MIT License
4.72k stars 2.76k forks source link

[Feature Suggestion] The ability to save teams to some cloud (like dropbox) #815

Open benbot opened 10 years ago

benbot commented 10 years ago

I think that would be a pretty cool feature. I'd be glad to put some work into something like that if you guys would want that kind of functionality.

benbot commented 10 years ago

Accidentally clicked close. Sorry :3

kotarou3 commented 10 years ago

Wouldn't it be better just to save the teams to the server?

benbot commented 10 years ago

I don't know how much server space they have or if they want to use their own resources to hold the teams. If you store the data on another service then the responsibility of keeping that data secure and uncorrupted falls on dropbox... making the maintainer's lives easier.

Zarel commented 10 years ago

Well, the advantage of storing on Dropbox is that users would be able to manipulate the teams as normal files.

The advantage of storing on PS is that it's associated with the PS account with no need to link to an external service, and PS deduplication etc can be tuned to its own service.

I'll probably just store on PS, to make it seamless. But first we need to fix server load.

Protonk commented 10 years ago

Why not just symlink the teams folder into your Dropbox folder?

benbot commented 10 years ago

That's what i ended up doing. I was just saying for the betterment of showdown.... integrated cloud storage.

Zarel commented 10 years ago

I actually do plan to add a feature to store teams on accounts.

benbot commented 10 years ago

If you need any help with it just shoot me an email. I'll be more than glad to lend a hand.

Zarel commented 10 years ago

Um, how skilled at programming are you? I've been planning a refactor to save teams in the new packed team format instead of JSON; if you could do that for me that'd be pretty nice.

benbot commented 10 years ago

I've been programming for about 8 years (mostly C and C++ but I know some JS). I'm in college atm. What's the new format?

Zarel commented 10 years ago

Time and skill are different. :P

https://github.com/Zarel/Pokemon-Showdown-Client/blob/101d67ea23d7a87c117338e341312c68bddd4aa0/js/storage.js#L64

I mean use this to pack the teams.

benbot commented 10 years ago

That looks pretty straight forward. But why use this instead of JSON?

Zarel commented 10 years ago

It's a bit less than 1/3 the size of JSON, which probably already uses even less RAM than the actual object representation. Storing teams that aren't open in this format should save a huge amount of memory.

benbot commented 10 years ago

Yeah that's quite a difference. So What exactly do you need done?

Zarel commented 10 years ago

Well, teams are currently saved in localStorage in JSON. The first step would be to change that over to the Storage.packTeam format. Be sure to make it backwards-compatible!

benbot commented 10 years ago

So converting JSON to Sotrage.packTeam? Seems pretty straight forward.

Zarel commented 10 years ago

Yep. It's probably a good first task for you to do; get you used to coding for PS. The repository is:

https://github.com/Zarel/Pokemon-Showdown-Client

And you test with testclient.html.

benbot commented 10 years ago

I'll get working on that tomorrow then.

scpinion commented 9 years ago

I've been lurking in the dev room for a while now and was looking to start contributing. I know this is an old issue, but is it something you guys are still wanting to do? If so, I'd be willing to give it a go.

Zarel commented 9 years ago

Sure, assuming you mean the Storage.packTeam implementation.

scpinion commented 9 years ago

Yes, implement the Stoarage.packTeam in the current teambuilder with backwards compatibility for JSON. At the moment, the tier and team name are not included in the packed format. Was this by design?

I do not have the knowledge/experience to implement something that would allow saving teams to the server, but I'm willing to learn.

Irraquated commented 9 years ago

I have a bot capable of saving teams to its files and are able to send you the link to them after storing with a command. maybe this could relate to how this could be done?

itsjavi commented 8 years ago

Hi @Zarel what's the status of this? I still get different teams depending on the browser session I use, even if I am logged in with my regular user.

The thing is that it's not very straightforward to import the team I have in other browser (in my laptop at home) when I am using Showdown in my mobile, because most of the time I don't have access to that browser session where I have my teams.

I ended up having an online file where I manually save my teams every now and then... but It's awful, would be nicer if they could be saved server-side, or integrate Dropbox for automatic synching (for that a timestamp-based logic could be used, in order to decide if there is new data or not) if you want to avoid the server load / storage possible problems.

Thanks!

Zarel commented 8 years ago

This is planned, but unlikely to happen particularly soon

prayansh commented 8 years ago

A suggestion: You could store the file on the dropbox account and on the PS server just store the link to this file. Probably wont take much server space. And then sync it regularly with the file. Also adding a functionality to change the file url could be an added bonus.

Zarel commented 8 years ago

wouldn't take much server space, but would take a lot of programming and I don't think it's the best solution here...

itsjavi commented 8 years ago

It would be enough to be able to connect to Dropbox/Google Drive for backing up and restore all your teams in a plain text file (PS format), maybe a dump of what you have in the localStorage. It could be a first iteration of the feature.

@Zarel I could help with that with a Pull Request. Do you have any recommendations, like where to look at first in the code or what needs to be done first?. I will follow the readme, but maybe if you could throw some light on how to get started with this particular thing it could help me to find things easier. Thanks

Zarel commented 8 years ago

It should be in the client repository: Zarel/Pokemon-Showdown-Client

I still don't like this idea because external storage adds so much complication and another server I don't control that can go down etc etc

ghost commented 7 years ago

This would be a nice "Premium feature". I'd be happy to pay a fee for this additional service. You could make it one time fee, yearly or monthly.

Zarel commented 7 years ago

We don't charge for features ever, so that's not really an option.

As a note for anyone reading the earlier conversation: Teams are now already stored packed.

DeathlyPlays commented 6 years ago

When I thought of this I considered like being able to download teams from the teambuilder to the user's computer, perhaps as a text file

Edit: Oh so did jpinto lol

NicholasEther commented 6 years ago

I know this is an old topic that's harder to kill than a Wish Blissey's teammate, but it's still not solved so thought I'd add my own suggestions.

Is it possible to make it so the storage location can be chosen by the user? Then we could just choose our dropbox/onedrive/etc folder on our computers and they will sync as usual. Then I just choose that folder on each computer, so it will automatically sync and load.

I can see how that could cause issues though, obviously it's easier to use LocalStorage, but it does save you from needing to make any other changes.

Another option is to allow us to download and upload our teams directly as a file instead of copy, and pasting text. Especially if you can remember our download location preferences. Then we can click the download button after every team edit. It syncs with our cloud of choice, then on the other computer we click the upload button and choose the file. This would overwrite the localstorage. Allowing you to save that feature.

The main issue is that clicking team back up, copying teams, opening text file, pasting teams, saving; is cumbersome, especially when you're making small tweaks to the team during and after every match.

Importing isn't that much of an issue since you'd only need to do it at the beginning of the session, but being able to backup with minimal effort or preferably automatically, would be very helpful.

Zarel commented 6 years ago

You can already download and upload teams by dragging and dropping them between the teambuilder and your computer (on Chrome and the desktop client).

benedictb commented 6 years ago

Is there any reason why PS doesn't use a database to store teams? I'd be happy to contribute to making this happen.

Zarel commented 6 years ago

@benedictb Your question is way too vague. Any system that stores data can be called a "database", so in a way, PS is already storing teams in a database.

If you're asking why PS does not currently store teams in the cloud, that's for a variety of reasons, but mostly does boil down to time needed. A lot of client stuff is slated to be significantly rewritten, so this is blocked until that happens.

benedictb commented 6 years ago

I just meant in a simple relational database on the server, as it seems like it would be pretty easy jump from the packed format to a table.

Zarel commented 6 years ago

Yeah, like I said, it boils down to time needed, and also a lot of client stuff is slated to be significantly rewritten.

scheibo commented 5 years ago

@Zarel

There are currently no major blockers, although it would help if someone could figure out the relevant Google Drive API for me (the API docs are really inscrutable). It would be nice to be able to store teams there, instead of on PS servers directly. It would also make it easier to share teams between accounts.

Can you elaborate on what your vision is here? Users would link their Google accounts with PS, and then when teams are saved/loaded instead of syncing with local storage they are instead saved to Drive? (or, perhaps you still use local storage as a cache, etc).

Zarel commented 5 years ago

@scheibo yes.

timotree3 commented 5 years ago

I've been looking into how to implement this using the Google Drive API. Using this documentation, I've learned a few things.

This situation necessitates one of two solutions:

The first one has the downside that it requires building some automation around the Google developer administrative API and I'm not sure if it is within the ToS to automate Project creation like that.

The second one has the downside that it would be high overhead on our server, requiring a copy of the updated team to be uploaded and temporarily stored as the server forwards the request to Google's servers.

Once we figure out these question about API authentication, it seems like it will be pretty easy to write the code to use Drive for team storage, we already use localStorage as basically a flat file for teams, and Drive allows Google Cloud Projects to store hidden files on their users Drives.

Zarel commented 5 years ago

It looks like the correct answer is to have only one Google Cloud Project. The reason I wanted Google Drive was so the teams could be files users could move around as they wanted.

timotree3 commented 5 years ago

Is the idea that the user would, on a team-by-team basis, use a file browser to choose what text file on their drive that team is synced to?

Zarel commented 5 years ago

No, there would just be one folder "Pokémon Showdown Teams", which would be kept in sync with PS.

timotree3 commented 5 years ago

Actually, based on this article about authentication, it seems like there is an intermediary authentication token that is specific to the permissions related to that user. It seems like that token can be sent to the client which would mean that we can have the best of both worlds.

Zarel commented 5 years ago

It looks pretty hard to control a folder in Google Drive. :/

timotree3 commented 5 years ago

Based on the article on MIME-types, folders are files in Drive. (See application/vnd.google-apps.folder

Zarel commented 5 years ago

Yeah, but folder access doesn't seem to give you access to all its contents recursively.

I think no matter how we did it, it wouldn't be any better usability than the current system. We might as well just store teams on our own database, or in AppData.

benedictb commented 5 years ago

Why don't we use some cloud nosql database instead of Google drive? You wouldn't have to worry about authentication at all in that case, just let the client handle it.

If users want to manipulate their teams like files they can export them and store them themselves.

Zarel commented 5 years ago

Yeah, my original idea for using Google Drive would be the ability to access the teams directly as files. But it looks like Google doesn't make it very convenient, in which case, yes, a traditional database is probably good enough for our purposes.

mfrancis95 commented 5 years ago

What about using object storage like S3?