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

Ajout de champ pour l'export vers une page externe #19

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

Syxxxx
Copy link
Collaborator

@Syxxxx Syxxxx commented Oct 10, 2014

Ajout des champs :

  • lastLife (vie restante des poireaux à la fin du match)
  • Ajout de l'export de l'id farmer
  • Ajout du timestamp dans la bdd
  • Prévision de l'ajout du nom de team

Changement de la clé primaire (fightid + leekid)

Attention :

  • pour une base deja existante il faudra faire un alter table.
  • je n'ai fait la maj de la creation de table que sur le script mysql

change Primary key to `fightId`,`leekId`
Add fields farmer and lastLife to json for dataReceiverURL.
Add filed teamName (actually not filled)
@lzear
Copy link
Owner

lzear commented Oct 10, 2014

Je ne peux pas accéder à mon FTP depuis ma connexion 3G (j'ai que ça). Ça marche impec' ? Je Merge dès que j'ai une confirmation. Aussi, je pense qu'il serait judicieux de poster le "alter table" ici et sur le forum.

Je me demande s'il y a un autre moyen de signaler aux utilisateurs qu'il faut changer la BDD. En l’occurrence je pense que quasiment personne utilise la BDD, mais je pense que ça pourrait être gênant si des joueurs autoUpdatent le script et qu'il ne sont pas avertis que leur PHP/SQL est obsolète.

@Syxxxx
Copy link
Collaborator Author

Syxxxx commented Oct 10, 2014

Un test exterieur serai le bienvenu (vu que j'ai recopié mes modifs d'une version anterieur à celle sur Github).

pour le alter table j'ai pas trop de solution à part ajouter un fichier d'update sql et d'afficher un warning à la 1ere execution du scprit après update.

si tu as d'autres idées je suis preneur

@yLark
Copy link
Collaborator

yLark commented Oct 15, 2014

Je n'utilise pas les fonctions php, donc je ne peux guère vous aider...
En plus, depuis j'ai fait des modifs, donc va falloir résoudre les conflits après le merge.
@AlexClaw, tu utilises les fonctions php ?

@abourdin
Copy link

Au début oui, mais plus maintenant faute d'utilité concrète. Le code est toujours déployé sur heroku mais ne sert pas à grand chose pour l'instant.

@lzear
Copy link
Owner

lzear commented Oct 21, 2014

J'ai Internet (la fibre!!) et un peu de temps libre maintenant.
Je peux m'occuper du merge n'importe quand. Mais le soucis c'est qu'il y a eu pas mal de modifications entre temps. Je sais pas trop quelle est la solution appropriée à ce genre de problèmes.
Je te propose d'insérer tes modifications dans la version la plus récente du plugin et re-faire un pull-request. On mettra ton amélioration dans une branche à part pour pouvoir travailler dessus. Car il faut ajouter pas mal de champs maintenant que Lark a traité les données JSON :
Force, Agilité, Fréquence, Coeurs, TotalLife (avant tout blindage), PT, PM... et surement d'autres choses encore.
Aussi, il faut voir si la derniere MAJ a une influence sur les données à enregistrer

Après, j'essaierai d'améliorer ce qu'il se passe coté serveur.

(Je t'ai mis en contributeur)

@Syxxxx
Copy link
Collaborator Author

Syxxxx commented Oct 21, 2014

Félicitation pour la fibre :)
oui, je vais refaire mes modifs sur la derniere version sa sera plus simple.
je n'ai pas eu le temps de regarder cela depuis mon pull request.

sinon pour evité les bug lors des maj, je crois que le plus simple est de faire un menu de config comme celuis qui existe deja mais pour l'export des données. par défaut les champs de base et si on ajoute des champs on propose un fichier avec la structure de la base.
qu'en pense tu ?

Merci pour l'ajout des contributeurs.
Bonne soirée

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.

4 participants