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

Extend server / client communication #259

Merged
merged 2 commits into from Jun 14, 2019

Conversation

Projects
None yet
2 participants
@befeleme
Copy link
Contributor

commented Jun 9, 2019

Par pokusu, ktere prikladam jako draft - rada bych, abychom mely neco, o cem muzeme diskutovat.
Reseni jsou docela "zbastlena", ale funkcni.

  • okenko stavajiciho stavu hry, ktery vykresluje client_receiver je male a neda se zvetsit/zmensit. Chtela jsem zaridit, aby se vykreslovalo podobne jako u interface. Problem je, ze intreface nepotrebuje stav k vykresleni okna, zatimco hraci plocha ano. Vysledkem je vytvoreni a vykresleni okna posunute az na moment obdrzeni informace ze servru. Jde to udelat jinak, lepe?

  • server tvori dva seznamy pripojenych klientu: interface a receiver. Kazdemu posila podle route, na kterem jsou propojeni, jine informace. Klienti si to nejak ctou podle zacatku zpravy a zobrazi, co obdrzeli.

  • protoze server umi posilat nejen stringy, ale i jsony, zkusila jsem poslat a nacist stav hry poslany metodou send_json, funguje to.

@befeleme befeleme referenced this pull request Jun 9, 2019

Open

Udělat server #191

@ivet1987

This comment has been minimized.

Copy link
Contributor

commented Jun 10, 2019

Je to mazec, koukám a nechápu, hezké, tak to zítra proberem :)

Show resolved Hide resolved server.py

@befeleme befeleme marked this pull request as ready for review Jun 12, 2019

@ivet1987
Copy link
Contributor

left a comment

ok for now, will be updated.

@ivet1987 ivet1987 changed the title Attempt to extend server / client communication Extend server / client communication Jun 14, 2019

@ivet1987 ivet1987 merged commit 428702b into PyLadiesCZ:master Jun 14, 2019

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.