TSAP-Laval / acquisition-backend

MIT License
1 stars 0 forks source link

Changer nom endpoints API [Important] #45

Closed laurentlp closed 7 years ago

laurentlp commented 7 years ago
laurentlp commented 7 years ago

PLZ !!!!! @alexDeschenes @mLaribi @Alexandrehebertvincent

laurentlp commented 7 years ago

PLZ !!!!! @alexDeschenes @mLaribi @Alexandrehebertvincent

Alexandrehebertvincent commented 7 years ago

👍

laurentlp commented 7 years ago

????? @mLaribi r.HandleFunc("/api/GetMovementType", a.GetMovementTypeHandler) r.HandleFunc("/api/GetActionType", a.GetAllActionsTypes) r.HandleFunc("/api/PostActionType", a.PostActionType) r.HandleFunc("/api/coachs/getAllCoachs", a.GetCoachsHandler) r.HandleFunc("/api/coachs/postCoach", a.PostCoachHandler)

  1. Tes nom d'enpoints sont pas pantoute uniformes (majuscules et minuscules aléatoires)
  2. Il y a des majuscules dans tes URI
  3. Ils sont préfixés par des Get, Post, etc...
  4. Un endpoint ressemble à : r.HandleFunc("/api/mouvements/types, a.GetMovementTypeHandler).methods('GET') r.HandleFunc("/api/terrains, a.GetTerrainsHandler).methods('GET') r.HandleFunc("/api/parties, a.CreerPartiesHandler).methods('POST') r.HandleFunc("/api/parties/{id}, a.GetPartiesHandler).methods('GET') ... ~r.HandleFunc("/api/GetMovementType", a.GetMovementTypeHandler)~
laurentlp commented 7 years ago

@mLaribi Toujours pas arrangé, tes nom d'endpoints ne sont toujours pas conformes ! Si tu ne les arrangent pas tu ne passeras pas les PR...

mLaribi commented 7 years ago

Mes endpoints sont modifiés!

laurentlp commented 7 years ago

@mLaribi Ca c'est bon selon toi ?

        r.HandleFunc("/api/GetMovementType", a.GetMovementTypeHandler)
        r.HandleFunc("/api/GetActionType", a.GetAllActionsTypes)
        r.HandleFunc("/api/PostActionType", a.PostActionType)
        r.HandleFunc("/api/coachs/getAllCoachs", a.GetCoachsHandler)
        r.HandleFunc("/api/coachs/postCoach", a.PostCoachHandler)
mLaribi commented 7 years ago

Pas encore push. Quelque chose comme ça?

r.HandleFunc("/api/action/movementType", a.GetMovementTypeHandler).Methods("GET")
r.HandleFunc("/api/action/actiontype", a.GetAllActionsTypes).Methods("GET")
r.HandleFunc("/api/action/addactiontype", a.PostActionType).Methods("POST")
//Coachs
r.HandleFunc("/api/coachs", a.GetCoachsHandler).Methods("GET")
r.HandleFunc("/api/coachs/addcoach", a.PostCoachHandler).Methods("POST")
r.HandleFunc("/api/coachs/addCoachTeam/{id}", a.AssignerEquipeCoach).Methods("PUT")
laurentlp commented 7 years ago

@mLaribi Fais juste lire l'issue ! J'ai déjà marqué comment ça doit être sinon check vite vite sur internet, tu vas voir que la norme c'est pas comme ça ! Garde en tête que ce sont des URI... donc tu peux voir ça comme une URL, demande toi si tu as déjà vu des URLs de même ?

laurentlp commented 7 years ago

@mLaribi Et puis ? Est-ce que ça avance ? As tu de la difficulté ?

laurentlp commented 7 years ago

@mLaribi Et puis ? Est-ce que ça avance ? As tu de la difficulté ?

laurentlp commented 7 years ago

Ça la juste pris 4 mois ! Good job...

laurentlp commented 7 years ago

Ahahah! Non c'est une blague, il a fallu que je le fasse moi même !