Corriger les logo - Githubissues

BettorLeague / spring-boot-server-deprecated

Bettor League API REST

http://api.bettorleague.ovh/swagger-ui.html

0 stars 0 forks source link

Corriger les logo #11

Open RedHotMan opened 5 years ago

RedHotMan commented 5 years ago

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

CNadjim commented 5 years ago

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 commented 5 years ago

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

CNadjim commented 5 years ago

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

RedHotMan commented 5 years ago

Go pour le JSON file

CNadjim commented 5 years ago

https://www.oodlestechnologies.com/blogs/Simple-Steps-To-Read-JSON-file-and-save-it-to-database-in-spring-boot

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

gurnavdeep commented 5 years ago

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 commented 5 years ago

Tu t'en occupes @gurnavdeep ?

gurnavdeep commented 5 years ago

Okey je vois ça la semaine prochaine

CNadjim commented 5 years ago

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