Hotfix release 1.7 : réparation de l'encodage des URLs de membre #2535
Conversation
|
Tu es parvenu à reproduire le bug avec un TU ? Parce que, de mémoire, il en existe déjà un mais le bug était toujours présent. |
| @@ -4,7 +4,7 @@ | |||
|
|
|||
| {% with profile=member|profile %} | |||
| <a | |||
| href="{{ member.get_absolute_url }}" | |||
| href="{{ profile.get_absolute_url }}" | |||
There was a problem hiding this comment.
Ce genre de gag me fait dire que ce serait vraiment cool d'arriver à fusionner les 2 tables.
There was a problem hiding this comment.
On a une idée de comment nous pourrions faire ça ? Parce que ça me gave aussi et s'il faut prendre quelques soirées là dessus, je serais prêt à le faire.
There was a problem hiding this comment.
Théoriquement, c'est pas très-compliqué, c'est cette technique.
En pratique, vu l'état du projet, on va avoir 2 grosses difficultés :
- Le risque de conflits monstrueux (et donc de régressions) parce que ça va avoir des impacts partout
- La migration des données actuellement contenue dans la table d'authentification standard, qui ne sera sans doute pas triviale (assez simple à faire en SQL pur cela dit).
Pour moi, on va devoir faire quelque chose dans ce goût là :
- Développement de la fonctionnalité
- PR avec revue de code à fond et QA
- Bloquage de tout merge
- Rebase
- QA du rebase si y'a eu des difficultés
- Merge
- Déblocage des merges
There was a problem hiding this comment.
Même pas peur, je vais tenté de dégager du temps cette semaine pour ça.
There was a problem hiding this comment.
@GerardPaligot il me semble qu'il y a même une issue à ce sujet, si tu la retrouve :)
Il aurait fallu un TU front ici. |
Je suis certain que c'est possible d'écrire des TU front en plus. Il doit forcément y avoir des technos à ce sujet. |
|
Bah pour les tests front il faut monter une architecture de test front et Le lun. 13 avr. 2015 11:02, Gérard Paligot notifications@github.com a
|
|
Ben écoute : ça marche, donc je merge :) |
Hotfix release 1.7 : réparation de l'encodage des URLs de membre
|
Et tant pis pour le double commit. |
|
(et ca marche bien en beta \o/ ) |
Cette PR hotfix la release (Voir l'issue pour plus d'information)
Note pour QA:
"bpar exemple/membres/