Feat 1256 runner to not use meetup api to display events #1335
Conversation
|
Par contre je n'ai pas trouvé de moyen de tester le runner (en l'éxécutant) @agallou |
vinceAmstoutz
assigned agallou, stakovicz and vinceAmstoutz and unassigned agallou and stakovicz
| $this->meetupClient = $meetupClient; | ||
| $this->officiesCollection = new OfficesCollection(); | ||
| $this->transformer = new Transformer($this->officiesCollection); | ||
| $this->meetupRepository = $meetupRepository; |
There was a problem hiding this comment.
pour moi il faut conserver l'appel au transformer. Le format de ce qu'on envoie à Algolia doit toujours être celui-ci :
(mais la source doit être la base de données et plus l'api meetup)There was a problem hiding this comment.
Ok je modifie ça à l'occasion
There was a problem hiding this comment.
C'est fait j'ai restoré le transformer par contre j'ai des doutes sur le faite qu'il fasse lui passer OfficesCollections à l'instanciation. Le contenu de la méthode transform me parait également peut adapté puis qu'on a un tableau d'objet Meetups désormais à traiter. Comment puis-je tester tout ca @agallou ? En modifiant tests/units/AppBundle/Indexation/Meetups/Transformer.php ?
There was a problem hiding this comment.
Le plus simple que je vois est de se créer un compte de test sur Algolia.
De modifier les paramètres lié à algolia dans le parameters.yml :
web/app/config/parameters.yml.dist-docker
Line 20 in 6c24a04
Et de relancer
indexing:meetups.Et ensuite aller voir cette page en local : https://afup.org/meetups/
Pour moi le officesCollection peux rester, il permet d'en récupérer les libellés, mais effectivement il faut adapter le transformer pour non plus traiter un tableau qui viens de l'API meetup, mais de la base de données.
(au besoin si tu veux on peux s'appeler si ça peux être plus clair)
There was a problem hiding this comment.
Je re-regarde ca dès que possible et je te redis
There was a problem hiding this comment.
Voilà c'est fait @agallou 🙂
…to feat-1256-runner-to-not-use-meetup-api-to-display-events
Completes #1300 to solve #1256