Projet

Général

Profil

Bug #13791

utiliser des slugs plutôt que des id dans les endpoints

Ajouté par Frédéric Péters il y a plus de 7 ans. Mis à jour il y a environ 7 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
28 octobre 2016
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Utiliser des id ça complique la migration de plateformes en plateformes.


Fichiers

Révisions associées

Révision 40b65753 (diff)
Ajouté par Serghei Mihai il y a environ 7 ans

api: use categories slugs instead of ids (#13791)

Révision bf5d84a3 (diff)
Ajouté par Serghei Mihai il y a environ 7 ans

use unique category slugs in urls (#13791)

Historique

#1

Mis à jour par Serghei Mihai il y a plus de 7 ans

#2

Mis à jour par Frédéric Péters il y a plus de 7 ans

Assurer l'unicité des slugs ?

#3

Mis à jour par Serghei Mihai il y a plus de 7 ans

J'avais fait ça dans un autre patch avec la migrations des données et l'utilisations des slugs dans les urls.
Le voici.

#4

Mis à jour par Frédéric Péters il y a plus de 7 ans

Ça va éclater une erreur 500 à l'usager.

#5

Mis à jour par Serghei Mihai il y a plus de 7 ans

Hm... Ce n'est pas mon cas, ni dans le backoffice, ni en communicant avec la cellule abonnements dans combo.
Je râte probablement un cas.

#6

Mis à jour par Frédéric Péters il y a plus de 7 ans

Je rate probablement un cas.

Ajouter une catégorie, ajouter une deuxième qui obtiendrait le même slug.

#7

Mis à jour par Serghei Mihai il y a plus de 7 ans

Je propose de gérer cela en ajoutant en suffixe la pk de la dernière categorie ayant le même slug, incrementée.

#8

Mis à jour par Frédéric Péters il y a plus de 7 ans

Je propose de gérer cela en ajoutant en suffixe la pk de la dernière categorie ayant le même slug, incrementée.

C'est plutôt moche de se baser sur l'id, on gagne des résultats différents en ajoutant/supprimant des catégories; je préférerais que ça soit géré comme dans wcs ou combo. (slug, slug-1, slug-2, slug-3, sans rapport avec l'id)

#10

Mis à jour par Frédéric Péters il y a environ 7 ans

Ok.

#11

Mis à jour par Serghei Mihai il y a environ 7 ans

  • Statut changé de En cours à Résolu (à déployer)
commit bf5d84a3f01a0c5ad2199833b593d541756f4c29
Author: Serghei Mihai <smihai@entrouvert.com>
Date:   Fri Jan 20 16:42:31 2017 +0100

    use unique category slugs in urls (#13791)

commit 40b6575398fe6812501103933cab924434fad092
Author: Serghei Mihai <smihai@entrouvert.com>
Date:   Fri Jan 20 15:31:02 2017 +0100

    api: use categories slugs instead of ids (#13791)
#12

Mis à jour par Serghei Mihai il y a environ 7 ans

  • Statut changé de Résolu (à déployer) à Fermé

Formats disponibles : Atom PDF