Skip to content
This repository has been archived by the owner on May 25, 2019. It is now read-only.

Corriger les logo #11

Open
RedHotMan opened this issue Oct 14, 2018 · 9 comments
Open

Corriger les logo #11

RedHotMan opened this issue Oct 14, 2018 · 9 comments
Assignees
Labels
bug Something isn't working

Comments

@RedHotMan
Copy link
Member

Probléme:

  • Certains logo récupéré depuis l'API retourne des 404 (Athletic Bilbao)
  • Certains logo ne sont pas à jour (Everton)
  • Certains logo ne sont pas optimisés (PSG)

Solution ?

  • On devrait mettre les logos en base de données par exemple
@RedHotMan RedHotMan added the bug Something isn't working label Oct 14, 2018
@CNadjim
Copy link
Member

CNadjim commented Oct 14, 2018

Plutôt que de les avoir en base, et qu'il soit drop s'il y a une migration de la db, on devrait plutôt créer un fichier csv avec 2 entrées le nom de l'équipe ( la clée primaire niveau base ) et le nouveau logo ( url svg wikipedia ou autre ).
A partir de là on pourrait mettre à jour la base avec le fichier de manière automatique ( batch etc..)

@RedHotMan
Copy link
Member Author

Je pencherais plus pour un JSON plutôt qu'un csv file. Plus lisible et facile à manipuler à mon avis.
@gurnavdeep t'en pense quoi ?

@RedHotMan RedHotMan changed the title Corriger le logo Corriger les logo Oct 15, 2018
@CNadjim
Copy link
Member

CNadjim commented Oct 15, 2018

Ca me faciliterai grave le taf de lire depuis un json, à vous de voir 😄

@RedHotMan
Copy link
Member Author

Go pour le JSON file

@CNadjim
Copy link
Member

CNadjim commented Oct 15, 2018

Qui se charge de faire le json ? mdr
Parce que là on a pas mal de team en base... (210 en tout)

@gurnavdeep
Copy link
Member

Ca risque d'être long et assez chiant pour la mise en place d'un fichier csv. Donc si la facilité et l'efficacité penche pour un fichier JSON alors va pour un JSON

@CNadjim
Copy link
Member

CNadjim commented Oct 15, 2018

Tu t'en occupes @gurnavdeep ?

@gurnavdeep
Copy link
Member

Okey je vois ça la semaine prochaine

@CNadjim
Copy link
Member

CNadjim commented Oct 15, 2018

thx jte brieferai sur l'accès à la db en prod etc

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants