Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[beta v18] Empêche de créer un tag en double #3542

Merged
merged 5 commits into from
Apr 26, 2016
Merged

[beta v18] Empêche de créer un tag en double #3542

merged 5 commits into from
Apr 26, 2016

Conversation

artragis
Copy link
Member

Q R
Type de modification correction de bug / nouvelle fonctionnalité / évolution
Ticket(s) (issue(s)) concerné(s) (#3535)

QA

  • Créez deux tags qui ont le même slug (C/C++/C#)
  • Publiez un tuto dans chaque tag
  • vérifiez que si vous allez dans la liste des tutos et que vous ajoutez "?tag=c" dans l'url, tous les tuto apparaissent bien que le tag qui leur soit associé est bien celui qui leur est propre (C/C++)

Il s'agit d'un quickfix le temps de la béta, un travail plus conséquent sera fait pour la v19.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.03%) to 70.845% when pulling 0f95c48 on artragis:no-tag-doubly into fb73363 on zestedesavoir:release-v18.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.04%) to 70.838% when pulling ba7cbde on artragis:no-tag-doubly into fb73363 on zestedesavoir:release-v18.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.03%) to 70.845% when pulling 73ba63d on artragis:no-tag-doubly into fb73363 on zestedesavoir:release-v18.

@coveralls
Copy link

Coverage Status

Coverage increased (+16.4%) to 87.282% when pulling 778395f on artragis:no-tag-doubly into fb73363 on zestedesavoir:release-v18.

@artragis
Copy link
Member Author

Après moulte difficultés, QA can start

@coveralls
Copy link

Coverage Status

Coverage increased (+16.4%) to 87.282% when pulling be3f4a0 on artragis:no-tag-doubly into fb73363 on zestedesavoir:release-v18.

@orandin
Copy link
Contributor

orandin commented Apr 19, 2016

Q.A : ❌

En suivant chacun de tes points, @artragis, la Q.A est OK. Mais hélas, je suis sorti un peu du sentier. J'ai cliqué sur "tous les tags" et j'ai découvert un petit soucis. L'interface me liste ces tags :

c
1 contenu
c++
1 contenu
d
1 contenu

Sauf que si l'on clique sur C ou C++, on est redirigé vers la même adresse et il y a plus qu'un contenu.

De même qu'au niveau des tags les plus utilisés du menu "Tutoriels", j'ai le tag C et le tag C++ qui pointent tous deux vers le même tag, techniquement parlant. Un peu perturbant pour un utilisateur qui s'attend à ne voir que du C ou du C++.

(cc @GerardPaligot @SpaceFox)

@artragis
Copy link
Member Author

j'ai dit @orandin de vous pinger @GerardPaligot et @SpaceFox pour savoir si on se contentait du quickfix que j'ai proposé mais qui génère le bug découvert par orandin et qu'on se propose de faire un truc plus complet pour la v19 ou bien je passe du temps tout de suite quitte à ralentir la v18 qui est déjà en retard.

@gustavi
Copy link
Contributor

gustavi commented Apr 21, 2016

Je suis pour faire quelque chose de propre maintenant.

@SpaceFox
Copy link
Contributor

Ça prendrait combien de temps de sortir quelque chose de propre ? C'est quoi les risques d'avoir quelque chose de pas propre ?

Question subsidiaire à plus long terme : pourquoi on se fait chier avec des slug de merde, qui nous ont posés des tas de problèmes, alors que toute la chaîne de traitement comprends Unicode, et que donc on a pas besoin de se taper des transformations non maîtrisées ?

@artragis
Copy link
Member Author

Ça prendrait combien de temps de sortir quelque chose de propre ?

je n'en ai strictement aucune idée car... je ne sais pas où on a du slug et
où on a du title.

Le 21 avril 2016 à 08:42, SpaceFox notifications@github.com a écrit :

Ça prendrait combien de temps de sortir quelque chose de propre ? C'est
quoi les risques d'avoir quelque chose de pas propre ?

Question subsidiaire à plus long terme : pourquoi on se fait chier avec
des slug de merde, qui nous ont posés des tas de problèmes, alors que toute
la chaîne de traitement comprends Unicode, et que donc on a pas besoin de
se taper des transformations non maîtrisées ?

—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
#3542 (comment)

@gustavi
Copy link
Contributor

gustavi commented Apr 21, 2016

Sur le projet on a du slug partout sauf pour les membres.

@artragis
Copy link
Member Author

La preuve que non, ce bug prouve que la création de tag se fait par titre.

2016-04-21 9:16 GMT+02:00 Laville Augustin notifications@github.com:

Sur le projet on a du slug partout sauf pour les membres.

—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
#3542 (comment)

@gustavi
Copy link
Contributor

gustavi commented Apr 21, 2016

Je parlais niveau URL.

@artragis
Copy link
Member Author

et moi niveau du code (c'est ça qui prend du temps).

2016-04-21 9:25 GMT+02:00 Laville Augustin notifications@github.com:

Je parlais niveau URL.

—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
#3542 (comment)

@GerardPaligot
Copy link
Member

@artragis On en est où ici ?

@artragis
Copy link
Member Author

Je vais voir ce que je peux faire pour mettre les title au lieu des slugs.

2016-04-25 18:03 GMT+02:00 Gérard Paligot notifications@github.com:

@artragis https://github.com/artragis On en est où ici ?

—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
#3542 (comment)

@GerardPaligot
Copy link
Member

Merci @artragis !

@artragis
Copy link
Member Author

J'ai tenté de mettre les titres à la place des slugs. Je me suis limité à la page des tags. Ce qui peut être incomplet (typiquement, en écrivant ce message, je pense au cartouche des contenus). Du coup je vas voir, mais je risque d'en oublier pas mal.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.003%) to 70.875% when pulling 89372a8 on artragis:no-tag-doubly into fb73363 on zestedesavoir:release-v18.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.01%) to 70.868% when pulling 89372a8 on artragis:no-tag-doubly into fb73363 on zestedesavoir:release-v18.

@coveralls
Copy link

Coverage Status

Coverage increased (+16.4%) to 87.282% when pulling 446bcd3 on artragis:no-tag-doubly into fb73363 on zestedesavoir:release-v18.

@gustavi
Copy link
Contributor

gustavi commented Apr 26, 2016

QA : OK MAIS j'ai ça http://i.imgur.com/DThJggR.png

Une fois corrigé je peux merger.

@artragis
Copy link
Member Author

Corrigé.

2016-04-26 9:07 GMT+02:00 Laville Augustin notifications@github.com:

QA : OK MAIS j'ai ça http://i.imgur.com/DThJggR.png

Une fois corrigé je peux merger.

—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
#3542 (comment)

@coveralls
Copy link

coveralls commented Apr 26, 2016 •
edited
Loading

Coverage Status

Coverage increased (+16.4%) to 87.294% when pulling 6fc7685 on artragis:no-tag-doubly into fb73363 on zestedesavoir:release-v18.

@gustavi gustavi merged commit 2f05fb3 into zestedesavoir:release-v18 Apr 26, 2016
gustavi pushed a commit that referenced this pull request Jun 17, 2016
@GerardPaligot @gustavi
Finish v18 (#3666)
5b8e231 
* Enlève le recalcul des votes à la volée lors de la migration

* ZEP-25 : Corrige la commande de migration + instructions MEP

* corrige le MP de la migration

* Rééquilibre le texte de présentation de la home

* Design de la page "Tous les tags"

* Trailing slash des ressources de l'API optionnel

* Fix #3503 : plus possible de creer un tag vide

* Fix #3494 : correction du menu pour les articles

* Fix #3497 : aligne les messages de la page de connexion

* Fix #3489 : corrige l'affiche des article sur la page d'un membre

* Utilise le bon sprite sur écran rétina

* fix 3513: supprime la supperposition des aides markdown

* fix #3492: Corrige le design des content-item de la home

* Affiche 3 articles et 4 tutos sur la home

Signed-off-by: Quentin Gliech <quentingliech@gmail.com>

* Enlève le nombre des commentaires sur les content-items des tutoriels

Signed-off-by: Quentin Gliech <quentingliech@gmail.com>

* Affiche le nombre de publications sur la home

* corrige une migration d'id

* Corrige la marge sur la home "connectée"

* Corrige la souscription à un sujet

* fix(migration): Corrige les souscriptions des membres.

Closes #3498

* Fix #3507 : correction de l'affichage des noms des filtres

* Typo sur la description sur la page des categories

* fix 3490

* enlève les bots

* Fix #3534 : correction de l'encodage des sorties pour la commande zep-25

* fix: Ordonne correctement les sujets des sujets suivis.

Closes #3536

* fix: Recupere la liste des sujets suivis pour la sidebar.

Closes #3498

* test: Retourne toujours bien les sujets suivis dans le bon ordre.

* fix: N'affiche plus "Suivre ce contenu" pour les visiteurs.

Closes #3545

* Fix #3542 : Empêche de créer un tag en double

* Fix #3533 : corrige le nombre de tuto par tag

* Fix #3496 : correction de la page des contenus (#3524)

* [beta v18]Rend la topbar cohérente en matière de toptags (#3564)

* rend la topbar cohérente

* typo

* typo

* typo

* typo

* typo

* typo

* Mise à jour de la configuration nginx

* [v18]Donne les instruction pour corriger #3448 (#3577)

Donne les instruction pour supprimer les anciennes tables du module des tutos #3448

* [ci skip] Correction du message de migration de la ZEP-25 (#3595)

* Fix #3592: Ajout du lien vers "CONTRIBUTING.md" (#3596)

* Fix #3587 : Index les relations generiques des modeles des notifs. (#3594)

* fix(notif): Index les relations generiques des modeles des notifs.

Pour information, Django n'index pas automatiquement les relations
generiques ce qui vient plomber les performances quand on
commence a disposer d'une grosse base de donnees.

Voir https://groups.google.com/forum/#!topic/django-users/GJ-FVh40-F4

Closes #3587

* [ci skip] Ajoute la commande d'opti mysql dans le update.md

* Fix #3584 : correction des commandes de migration de la v18 (#3586)

* Évite les tags répétés plusieurs fois (#3601)

* augmente la taille des tags et évite qu'ils soient recréés quand ils sont trop grands.

* ajoute un test unitaire

* Fix l'unicite des tags

* Corrige l'unicite des tags (again)

* change le processus de création des tags lors du sujet de beta

* fix indexerror

* pep8

* pep8

* fix indexerror

* remove useless test

* fix test

* pep8

* Correction affichage du tag (#3597)

* ZEP-25 : correction mineure de la commande de migration (#3612)

* Fix #3614 - ZEP-25 : corrige les URL dans le mp de migration (#3616)

* Fix #3626 : corrige les espaces dans l'ajout de tags sur les contenus (#3627)

* Hotfix v18 : votes negatifs (#3635)

* fix(notif): Marque comme lu une notif d'un contenu non suivi. (#3664)

Closes #3659

* fix(notif): Evite les notifs persistantes a la suppression. (#3663)

Closes #3639

* fix(3630): Migration des souscriptions par e-mail. (#3661)

Closes #3630

* fix(notif): Mise a jour de la date des notifications. (#3662)

Closes #3628
@artragis artragis deleted the no-tag-doubly branch April 13, 2017 07:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@artragis @coveralls @orandin @gustavi @SpaceFox @GerardPaligot