eric-taix / volley34-mobile

Application mobile pour le Volley34
3 stars 0 forks source link

Team's competitions do not return all competition #276

Closed eric-taix closed 2 years ago

eric-taix commented 2 years ago

When I use the ranking endpoint (http://api.volley34.fr/v1/Classements/equipes/{{Equipe}}) for VCVX1, 2 competitions are returned: 2022-C004 and 2022-C024.

But when I use the team's competitions endpoint (http://api.volley34.fr/v1/equipes/{{Equipe}}/competitions) then only one competition is returned:

[
    {
        "CompetitionCode": "2022-C004",
        "Saison": 2022,
        "Niveau": "LO",
        "TrancheAge": "SE",
        "RegroupementSexe": "X",
        "TypedeJeu": "4x4",
        "TypeCompetition": "T1        ",
        "LibelleCompetition": "Championnat 2022 (Brassage) - Mixte 4x4",
        "DateDebut": "2021-11-08T00:00:00",
        "DateFin": "2022-01-14T00:00:00",
        "CompetitionGroupID": "CH",
        "Created": null,
        "Modified": null,
        "division": "1",
        "poule": "2",
        "Classements": null
    }
]

It seems to be a regression as it worked few days ago. The impact is that now I'm not more able to display all competitions in club's teams tab:

Capture d’écran 2022-01-26 à 12 53 51
eric-taix commented 2 years ago

En y repensant ce n'est peut être pas une régression car auparavant je n'utilisais pas le endpoint http://api.volley34.fr/v1/equipes/{{Equipe}}/competitions mais celui retournant les classements d'une équipe (j'ai effectué le changement pour avoir les numéros de divisions correctes et non pas les EX)

eric-taix commented 2 years ago

@Lo0ping tu as vu cette issue ? Elle est assez bloquante car sans elle je ne peux remonter correctement les informations des compétitions en cours pour chaque équipe.

Lo0ping commented 2 years ago

OK, je regarde ça rapidement

Lo0ping commented 2 years ago

Franchement, faut que je debuggue en détail car la, je ne vois vraiment pas ce qui cloche

image

Lo0ping commented 2 years ago

C'est bon, j'ai trouvé ... les inscriptions ne sont reliées qu'a la poule de brassage 😒😒😒 ça va être la m... a modifier ça !!!

eric-taix commented 2 years ago

Bon ce n'est peut être pas à faire maintenant du coup ! Maintenant j'avais besoin de ce endpoint parce que pour certaines équipes le fait de relier le division EX à la division 1 ne marche pas toujours ! (Cf une autre issue la dessus). Si tu arrives à résoudre le fait que EX est relié à 1 (et idem pour les autres divisions) alors je pourrais à priori utiliser le endpoint de classement d'une équipe pour connaître les compétitions auxquels l'équipe est inscrite.

Lo0ping commented 2 years ago

Le truc, c'est que tu auras le même problème avec la coupe. Faut que je vois cette histoire de niveau, ça fout trop le bordel

Lo0ping commented 2 years ago

Hello, ça devrait être bon ...

[
    {
        "CompetitionCode": "2022-C001",
        "Saison": 2022,
        "Niveau": "LO",
        "TrancheAge": "SE",
        "RegroupementSexe": "M",
        "TypedeJeu": "4x4",
        "TypeCompetition": "T1        ",
        "LibelleCompetition": "Championnat 2022 (Brassage) - Masculin 4x4",
        "DateDebut": "2021-11-08T00:00:00",
        "DateFin": "2022-01-14T00:00:00",
        "DateLimiteReport": null,
        "CompetitionGroupID": "CH",
        "Created": null,
        "Modified": null,
        "division": "EX",
        "poule": "1",
        "Classements": null
    },
    {
        "CompetitionCode": "2022-C021",
        "Saison": 2022,
        "Niveau": "LO",
        "TrancheAge": "SE",
        "RegroupementSexe": "M",
        "TypedeJeu": "4x4",
        "TypeCompetition": "T1        ",
        "LibelleCompetition": "Championnat 2022 - Masculin 4x4",
        "DateDebut": "2022-01-24T00:00:00",
        "DateFin": "2022-04-01T00:00:00",
        "DateLimiteReport": null,
        "CompetitionGroupID": "CH",
        "Created": null,
        "Modified": null,
        "division": "HO",
        "poule": "0",
        "Classements": null
    }
]

à vérifier aussi, tu ne devrais plus avoir de valeur numérique dans Division. si c'est le cas préviens moi.

eric-taix commented 2 years ago

Nickel. Je regarde ça ce soir

eric-taix commented 2 years ago

Désolé je n'ai pas eu le temps de regarder avant maintenant.

1- http://api.volley34.fr/v1/equipes/{{Equipe}}/competitions avec VCVX1 pour l'équipe retourne

[
    {
        "CompetitionCode": "2022-C004",
        "Saison": 2022,
        "Niveau": "LO",
        "TrancheAge": "SE",
        "RegroupementSexe": "X",
        "TypedeJeu": "4x4",
        "TypeCompetition": "T1        ",
        "LibelleCompetition": "Championnat 2022 (Brassage) - Mixte 4x4",
        "DateDebut": "2021-11-08T00:00:00",
        "DateFin": "2022-01-14T00:00:00",
        "CompetitionGroupID": "CH",
        "Created": null,
        "Modified": null,
        "division": "EX",
        "poule": "2",
        "Classements": null
    }
]

donc toujours qu'une seule compétition !

Par contre effectivement j'ai bien EX comme division. Du coup pour interroger http://api.volley34.fr/v1/Classements/{{Competition}}/{{Division}}/{{Poule}} je peux utiliser EX comme division ou je dois utiliser la correspondance de la liste des divisions ? PS: Cette requête prend 5 secondes à me retourner un résultat, cela me parait énorme non ?

eric-taix commented 2 years ago

Assez bizarre, je faisais des tests sur apitest.volley34.fr et là j'ai bien 2 compétitions qui me sont retournées pour VCVX1 : 2022-C004 et 2022-C024 (avec les divisions et poules correspondantes) alors que sur api.volley34.fr je n'ai que 2022-C004 ? Les corrections que tu as faites (https://github.com/eric-taix/volley34-mobile/issues/276#issuecomment-1024045838) n'ont pas été faites sur le backup de la base que tu as remontée par hasard mais pas sur la base de prod ?

Lo0ping commented 2 years ago
Par contre effectivement j'ai bien EX comme division. Du coup pour interroger http://api.volley34.fr/v1/Classements/{{Competition}}/{{Division}}/{{Poule}} je peux utiliser EX comme division ou je dois utiliser la correspondance de la liste des divisions ?

Oui, et si tu rencontres une méthode ou il te faut encore mettre '1' à la place de 'EX', previens-moi

PS: Cette requête prend 5 secondes à me retourner un résultat, cela me parait énorme non ?

j'ai optimisé le chargement, ça devrait aller mieux

Assez bizarre, je faisais des tests sur apitest.volley34.fr et là j'ai bien 2 compétitions qui me sont retournées pour VCVX1 : 2022-C004 et 2022-C024 (avec les divisions et poules correspondantes) alors que sur api.volley34.fr je n'ai que 2022-C004 ?
Les corrections que tu as faites (#276 (comment)) n'ont pas été faites sur le backup de la base que tu as remontée par hasard mais pas sur la base de prod ?

Bah, c'est surtout que je n'avais pas déployé l'API en prod 😒 ça devrait aller mieux ...

eric-taix commented 2 years ago

Ok. @Lo0ping Depuis le déploiement je rencontre une erreur serveur (500) pour:

Lo0ping commented 2 years ago

C'est bon ... j'ai un peu trop optimisé le chargement 😜

eric-taix commented 2 years ago

Ok. J'ai maintenant http://api.volley34.fr/v1/competitions qui me renvoit null, je pense que c'est lié à tes modifications / optimisations ?

Lo0ping commented 2 years ago

c'est corrigé, désolé ...