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

Each card should use its owner's cemetery #62

Closed
allentiak opened this issue Apr 26, 2020 · 6 comments
Closed

Each card should use its owner's cemetery #62

allentiak opened this issue Apr 26, 2020 · 6 comments
Labels
bug Something isn't working

Comments

@allentiak
Copy link
Member

Currently, all cards go into only one of the two cemeteries...

@antonioalonzi
Copy link
Member

Again... I've seen it happening once but I'm not able to reproduce it.

Let's leave it open for a while.

@antonioalonzi
Copy link
Member

See #48

@allentiak
Copy link
Member Author

Maybe adding detailed logs to the game might help to reproduce this cases?

@antonioalonzi
Copy link
Member

Yep, I have few issues here.

#52 might help.
And reporting a gameId (might help even right now)

But ultimately if I want to get users I'll have to start paying for my virtual machine and have the logs persisted. (I assume now the log stays for very short time).

I don't know if you have ever seen the server logs... they provide the state at each action meaning that you should be able to reproduce the problems. But I didn't spend much time on them right now as I'm mostly using the app locally.

Soon they will be critical.

@antonioalonzi antonioalonzi changed the title Game: Each card should use its owner's cemetery Each card should use its owner's cemetery Apr 28, 2020
@antonioalonzi antonioalonzi added the bug Something isn't working label May 1, 2020
@antonioalonzi
Copy link
Member

I managed to reproduce it and check the logs:

        "owner": "Antonio85-2",
        "controller": "Antonio85-1",

Why the owner and controller are different?
I must have some bug when creating the deck (retrievingPlayerInfo).

It's more likely to happen for Guest... as I have just changed the username for USERs to not have that -N appended.

@antonioalonzi
Copy link
Member

Fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants