Respect game visibility
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 45/100
Direzione di ricerca
Inizia da GameService#markDirty, DelayedResponse, ClientService#broadcastDelayedResponses e GameResponsesTransformer#game, quindi esamina gli unit test correlati. Il lavoro è completato quando le partite private non vengono trasmesse, le partite riservate agli amici raggiungono solo gli amici, la visibilità viene omessa dalle risposte di gioco del client, GameAccess viene rimosso e i test modificati passano.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Currently, the server broadcasts all games to all players. However, games marked as PRIVATE should not be broadcasted at all and games marked as FRIENDS should only be sent to friends.
- Remove the unused
com.faforever.server.game.GameAccess - Don't serialize
visibilitywhen sending games to client (incom.faforever.server.integration.legacy.transformer.GameResponsesTransformer#game) - Implement a method similar to
com.faforever.server.game.GameService#markDirtythat allows specifying a list of players to send to - Extend
com.faforever.server.client.DelayedResponseso that a list of receivers can be specified. The list should be nullable (so make its getter return an Optional), in which case it's considered a broadcast (document this in the getter) - Refactor
com.faforever.server.client.ClientService#broadcastDelayedResponsesso that it either broadcasts the response or sends it to the list of recipients. - Adjust and add unit tests accordingly
- Lingua principale
- Java
- Stelle
- 12
- Fork
- 5
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di FAForever/faf-java-server
-
Server closes peers to ppl disconnecting from serverForse di nuovo libera @1-alex98 l’ha presa 2823 giorni fa e non c’è nessuna pull request aperta. Apertabug
FAForever/faf-java-server#112 · 1 assegnatario ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 25/100
FAForever/faf-java-server#110 · 1 commento ·
-
beginner-friendly
Difficoltà 3/5 1-2 giorni Idoneità per principianti 20/100
FAForever/faf-java-server#100 ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 48/100
FAForever/faf-java-server#99 ·
-
New Ban Revoke StructureForse di nuovo libera @1-alex98 l’ha presa 2902 giorni fa e non c’è nessuna pull request aperta. Apertaenhancement
FAForever/faf-java-server#95 · 1 assegnatario ·
Tutte le issue di FAForever/faf-java-server
Issue simili
-
[BUG] Re-subscribing the Mono returned by listTools/listResources/listResourceTemplates/listPrompts/listRoots duplicates the resultsForse già presa Una pull request collegata a questa issue è aperta o già unita. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
modelcontextprotocol/java-sdk#1174 · 1 reazione ·
I maintainer di solito rispondono entro 2 giorni
-
module: core
Difficoltà 2/5 1-3 ore Idoneità per principianti 77/100
bigbluebutton/bigbluebutton#26028 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
MetricsHub/metricshub-community#1390 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 86/100
apache/rocketmq-dashboard#6331 ·
I maintainer di solito rispondono entro 5 giorni
-
status: team-only type: dependency-upgrade
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
spring-projects/spring-boot#52143 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno