Project

General

Profile

Bug #13791

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

Added by Frédéric Péters about 8 years ago. Updated almost 8 years ago.

Status:
Fermé
Priority:
Normal
Assignee:
Serghei Mihai
Target version:
-
Start date:
28 October 2016
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:

Description

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

Files

0001-api-use-categories-slugs-instead-of-ids-13791.patch (6.51 KB) 0001-api-use-categories-slugs-instead-of-ids-13791.patch Serghei Mihai, 20 January 2017 05:04 PM
0001-use-unique-category-slugs-in-urls-13791.patch (8.4 KB) 0001-use-unique-category-slugs-in-urls-13791.patch Serghei Mihai, 20 January 2017 06:47 PM
0001-use-unique-category-slugs-in-urls-13791.patch (9.08 KB) 0001-use-unique-category-slugs-in-urls-13791.patch Serghei Mihai, 23 January 2017 02:27 PM
0001-use-unique-category-slugs-in-urls-13791.patch (9.47 KB) 0001-use-unique-category-slugs-in-urls-13791.patch Serghei Mihai, 23 January 2017 03:03 PM

Associated revisions

Revision 40b65753 (diff)
Added by Serghei Mihai almost 8 years ago

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

Revision bf5d84a3 (diff)
Added by Serghei Mihai almost 8 years ago

use unique category slugs in urls (#13791)

History

#1

Updated by Serghei Mihai almost 8 years ago

#2

Updated by Frédéric Péters almost 8 years ago

Assurer l'unicité des slugs ?

#3

Updated by Serghei Mihai almost 8 years ago

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

Updated by Frédéric Péters almost 8 years ago

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

#5

Updated by Serghei Mihai almost 8 years ago

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

Updated by Frédéric Péters almost 8 years ago

Je rate probablement un cas.

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

#7

Updated by Serghei Mihai almost 8 years ago

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

Updated by Frédéric Péters almost 8 years ago

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)

#11

Updated by Serghei Mihai almost 8 years ago

  • Status changed from En cours to 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

Updated by Serghei Mihai almost 8 years ago

  • Status changed from Résolu (à déployer) to Fermé

Also available in: Atom PDF