Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Drop Game Link to download and view. #8

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tarriel
Copy link
Contributor

@tarriel tarriel commented Nov 5, 2017

Gives the ability to drop a game link from the halite website, and it
will download and load the game, the game will be saved in /games
folder.

Probably not the best javascript ever as i only started learning yesterday, also
the url to download the game includes a player id, This is tsadmirals,
but any seem to work, no login required.

I'm not sure if people would want to download the game, but that was how I got it to work... also I don't check to see if the game is already there.

Gives the ability to drop a game link from the halite website, and it
will download and load the game, the game will be saved in /games
folder.

Probably not the best javascript ever as i only learnt yesteday,  also
the url to download the game includes a player id,  This is tsadmirals
but any seem to work, no login required.
@rooklift
Copy link
Owner

rooklift commented Nov 5, 2017

It seems to break for me the 2nd time you do it. How about you?

@tarriel
Copy link
Contributor Author

tarriel commented Nov 6, 2017

I just cloned a fresh copy from my repository and it seems to be working here, I can watch the game 3-4 times no problem, the same game several times in a row, or alternating between a couple of games, I even tried loading local games in between and it seems to be solid. I am running Linux, and electron is running locally installed, not sure if that would make a difference.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants