Project

General

Profile

Développement #38597

récupérer le slug des "cubes"

Added by Valentin Deniaud almost 5 years ago. Updated almost 5 years ago.

Status:
Fermé
Priority:
Normal
Assignee:
Valentin Deniaud
Target version:
-
Start date:
17 December 2019
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

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.

Files

0001-Revert-misc-allow-cubes-to-be-given-a-slug-30858-385.patch (929 Bytes) 0001-Revert-misc-allow-cubes-to-be-given-a-slug-30858-385.patch Valentin Deniaud, 17 December 2019 05:46 PM
0002-misc-add-slug-to-cubes-38597.patch (742 Bytes) 0002-misc-add-slug-to-cubes-38597.patch Valentin Deniaud, 17 December 2019 05:46 PM
0001-misc-add-slug-to-cubes-38597.patch (1.75 KB) 0001-misc-add-slug-to-cubes-38597.patch Valentin Deniaud, 18 December 2019 10:36 AM

Related issues

Related to BiJoe - Développement #38596: Utiliser un slug plutôt qu'une url pour récupérer les données des visualisationsFermé17 December 2019

Actions

Associated revisions

Revision c304e0f8 (diff)
Added by Valentin Deniaud almost 5 years ago

misc: add slug to cubes (#38597)

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

History

#2

Updated by Valentin Deniaud almost 5 years ago

  • Related to Développement #38596: Utiliser un slug plutôt qu'une url pour récupérer les données des visualisations added
#3

Updated by Thomas Noël (congés → 5 décembre) almost 5 years ago

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

Updated by Valentin Deniaud almost 5 years ago

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

#6

Updated by Thomas Noël (congés → 5 décembre) almost 5 years ago

  • Status changed from Solution proposée to 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

Updated by Valentin Deniaud almost 5 years ago

  • Status changed from Solution validée to 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

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

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF