Skip to content

Ajout d'indices#36

Open
jonag wants to merge 4 commits into
benVigie:masterfrom
jonag:source_master
Open

Ajout d'indices#36
jonag wants to merge 4 commits into
benVigie:masterfrom
jonag:source_master

Conversation

@jonag

@jonag jonag commented Apr 5, 2020

Copy link
Copy Markdown

Salut,

Comme on l'a évoqué ensemble, j'ai fait un fork du jeu qui contient pas mal de différences avec cette version avant de voir que le projet était toujours actif.
Suite à notre échange par mail j'ai implémenté cette après-midi une fonctionnalité d'indices sur mon fork que j'ai ensuite Cherry-Pick pour le proposer sur ta version, en ayant (je l'espère) supprimé tout ce qui était spécifique à mon fork (notamment la reprise de partie en cours de route).

Cela fonctionne de la façon suivante : si aucun mot n'est trouvé pendant 2 minutes le jeu affiche une lettre au hasard puis une autre toutes les 30 secondes jusqu'à ce qu'un nouveau mot soit trouvé.

J'ai bêtement c/c deux fonctions du front pour la détection de mots.

Je te laisse voir si tu veux l'intégrer à ta version du jeu ou pas 😉 !

jonag added 4 commits April 5, 2020 22:36
(cherry picked from commit ddbec24)

# Conflicts:
#	game_files/motsFleches.js
#	public/javascripts/game/grid.js
#	public/javascripts/game/mflEngine.js
(cherry picked from commit a5c13f2)

# Conflicts:
#	game_files/motsFleches.js
#	public/javascripts/game/mflEngine.js
(cherry picked from commit 58b404c)
@benVigie

benVigie commented Apr 7, 2020

Copy link
Copy Markdown
Owner

Yes !!! Alors comme ça on est bloque ?... 😛

J'en parlait justement avec une autre personne, et rajouter un indice est une chose que je voulais implémenter... Depuis 6 ans 😄 ! Merci les gars !

Je vais check ca rapidement ce soir. Si ca n'affecte rien d'autre, je merge direct. encore merci @jonag !

@jonag

jonag commented Apr 7, 2020

Copy link
Copy Markdown
Author

C'est peut-être avec moi que t'en parlait, on a échangé par mail 😜 !
Après l'avoir testé en conditions réelles sur plusieurs parties j'ai remarqué un petit changement de comportement : quand quelqu'un trouve un mot ça "écrase" sur la grille les lettres de ses petits camarades au lieu de juste venir compléter si des mots se croisent.

Ne merge peut-être donc pas en l'état !

Je vais voir pour corriger ça dans la semaine.

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