-
Notifications
You must be signed in to change notification settings - Fork 66
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
issue#1479 Sponsor ticket transport #1511
issue#1479 Sponsor ticket transport #1511
Conversation
e451f06
to
bb6679e
Compare
{ | ||
public function change() | ||
{ | ||
$this->execute("ALTER TABLE `afup_forum_sponsors_tickets` ADD `transport_mode` TINYINT, ADD `transport_distance` SMALLINT"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pourquoi y a t-il besoin de modifier cette table ci ? Normalement c'est la table qui contient les tokens et qui est potentiellement liée à la table avec les billets (afup_inscription_forum
), où celle-ci est déjà modifiée.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Les ticket sponsors sont stockés dans cette table il me semble afup_forum_sponsors_tickets
, non ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Dans la table afup_forum_sponsors_tickets on génère un token sponsor, qui donne le droit de saisir X places, et permet de générer un lien, qu'on partage au sponsor en question (le mail indiqué est le mail de la personne qui gère le sponsoring chez le sponsor, pas forcément le mail de la personne qui utilisera la/les places). En cliquant dessus il arrive sur la page de saisie des places, et une fois saisies les places arrivent dans afup_inscription_forum.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK j'avais pas compris ça comme ça.
Je corrige la PR alors.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
C'est revert.
Ni vu ni vu 😄
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
:D
je vois toujours la migration sur la PR il n'y aurait pas un souci ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Arf oui, il y avait un soucis lors du push.
Je viens de vérifier, tout est bon pour moi maintenant.
c8101b7
to
afd7330
Compare
On ajoute la possibilité de renseigner son mode de transport pour la certification LEAD (#1479).
Voici ce que cela donne :