Projet

Général

Profil

Development #38597

récupérer le slug des "cubes"

Ajouté par Valentin Deniaud il y a plus de 4 ans. Mis à jour il y a plus de 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Valentin Deniaud
Version cible:
-
Début:
17 décembre 2019
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Même esprit que #30858, qui n'a jamais marché à cause d'un bug dans #30859. C'est important à noter car ça veut dire qu'on peut toujours faire confiance à 'name' pour contenir un nom de domaine.

Fichiers

0001-Revert-misc-allow-cubes-to-be-given-a-slug-30858-385.patch (929 octets) 0001-Revert-misc-allow-cubes-to-be-given-a-slug-30858-385.patch Valentin Deniaud, 17 décembre 2019 17:46
0002-misc-add-slug-to-cubes-38597.patch (742 octets) 0002-misc-add-slug-to-cubes-38597.patch Valentin Deniaud, 17 décembre 2019 17:46
0001-misc-add-slug-to-cubes-38597.patch (1,75 ko) 0001-misc-add-slug-to-cubes-38597.patch Valentin Deniaud, 18 décembre 2019 10:36

Demandes liées

Lié à BiJoe - Development #38596: Utiliser un slug plutôt qu'une url pour récupérer les données des visualisationsFermé17 décembre 2019

Actions

Révisions associées

Révision c304e0f8 (diff)
Ajouté par Valentin Deniaud il y a plus de 4 ans

misc: add slug to cubes (#38597)

This also reverts commit 084b0d06b4e8bf5f9eeaee02cd71623e527ca68b,
which has never been actually used.

Historique

#2

Mis à jour par Valentin Deniaud il y a plus de 4 ans

  • Lié à Development #38596: Utiliser un slug plutôt qu'une url pour récupérer les données des visualisations ajouté
#3

Mis à jour par Thomas Noël il y a plus de 4 ans

Y'a un crash des tests, qui sont à modifier, mais aussi, envoie tout ça dans un seul commit, ça sera plus facile de comprendre l'idée àmha.

#5

Mis à jour par Valentin Deniaud il y a plus de 4 ans

Je trouve ça plus propre qu'un 'slug': None ne soit pas ajouté en fait, mais je modifie quand même le test.

#6

Mis à jour par Thomas Noël il y a plus de 4 ans

  • Statut changé de Solution proposée à Solution validée

Valentin Deniaud a écrit :

Je trouve ça plus propre qu'un 'slug': None ne soit pas ajouté en fait

Les goûts et les couleurs, je discute pas, mais ça rend le code un peu moche quand même.

mais je modifie quand même le test.

Yep

#7

Mis à jour par Valentin Deniaud il y a plus de 4 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit c304e0f8f8fef0acaa876d39c2ce09b34a46be4f
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date:   Tue Dec 17 17:40:57 2019 +0100

    misc: add slug to cubes (#38597)

    This also reverts commit 084b0d06b4e8bf5f9eeaee02cd71623e527ca68b,
    which has never been actually used.
#8

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

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF