{"action":"created","issue":{"url":"https://api.github.com/repos/zestedesavoir/zds-site/issues/3601","repository_url":"https://api.github.com/repos/zestedesavoir/zds-site","labels_url":"https://api.github.com/repos/zestedesavoir/zds-site/issues/3601/labels{/name}","comments_url":"https://api.github.com/repos/zestedesavoir/zds-site/issues/3601/comments","events_url":"https://api.github.com/repos/zestedesavoir/zds-site/issues/3601/events","html_url":"https://github.com/zestedesavoir/zds-site/pull/3601","id":153793602,"number":3601,"title":"[v18]fix_3591 évite les tags répétés plusieurs fois","user":{"login":"artragis","id":1523653,"avatar_url":"https://avatars.githubusercontent.com/u/1523653?v=3","gravatar_id":"","url":"https://api.github.com/users/artragis","html_url":"https://github.com/artragis","followers_url":"https://api.github.com/users/artragis/followers","following_url":"https://api.github.com/users/artragis/following{/other_user}","gists_url":"https://api.github.com/users/artragis/gists{/gist_id}","starred_url":"https://api.github.com/users/artragis/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/artragis/subscriptions","organizations_url":"https://api.github.com/users/artragis/orgs","repos_url":"https://api.github.com/users/artragis/repos","events_url":"https://api.github.com/users/artragis/events{/privacy}","received_events_url":"https://api.github.com/users/artragis/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":5,"created_at":"2016-05-09T14:46:48Z","updated_at":"2016-05-09T15:28:09Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/zestedesavoir/zds-site/pulls/3601","html_url":"https://github.com/zestedesavoir/zds-site/pull/3601","diff_url":"https://github.com/zestedesavoir/zds-site/pull/3601.diff","patch_url":"https://github.com/zestedesavoir/zds-site/pull/3601.patch"},"body":"*tableau à venir pycharm ne sait pas encore prendre en compte les templates*\n\n# QA\n\nLa PR augmente la taille des tags et évite qu'ils soient recréés quand ils sont trop grands.\nDu coup il faudra mettre à jour la bdd.\n\nVérifiez aussi que les tags assez grands tels que \"système d'exploitation\" passent désormais.\nEnfin vérifiez que quand on crée un tutoriel avec un tag de plus de 30 caractères puis\nqu'on ajoute un second tutoriel avec un tel tag, ça ne génère pas un doublon dans les tags.\n\n"},"comment":{"url":"https://api.github.com/repos/zestedesavoir/zds-site/issues/comments/217898043","html_url":"https://github.com/zestedesavoir/zds-site/pull/3601#issuecomment-217898043","issue_url":"https://api.github.com/repos/zestedesavoir/zds-site/issues/3601","id":217898043,"user":{"login":"artragis","id":1523653,"avatar_url":"https://avatars.githubusercontent.com/u/1523653?v=3","gravatar_id":"","url":"https://api.github.com/users/artragis","html_url":"https://github.com/artragis","followers_url":"https://api.github.com/users/artragis/followers","following_url":"https://api.github.com/users/artragis/following{/other_user}","gists_url":"https://api.github.com/users/artragis/gists{/gist_id}","starred_url":"https://api.github.com/users/artragis/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/artragis/subscriptions","organizations_url":"https://api.github.com/users/artragis/orgs","repos_url":"https://api.github.com/users/artragis/repos","events_url":"https://api.github.com/users/artragis/events{/privacy}","received_events_url":"https://api.github.com/users/artragis/received_events","type":"User","site_admin":false},"created_at":"2016-05-09T15:28:09Z","updated_at":"2016-05-09T15:28:09Z","body":"parce qu'on fetch sur le titre. Or si tu tronques un tag, ça marche pas. \nExemple :\n\nTag demandé : \"1234567891011121314151617181920\"\n\nTag obtenu  : \"1234567891011121314151\"\n\nQuand je cherchais à savoir, dans l'ancienne méthode, je demandais \n\"donne moi le tag qui a pour titre \"1234567891011121314151617181920\"\", \nil ne le trouvait pas et me créait de fait le tag qui a pour titre \n\"\"1234567891011121314151\"\" et ce pour chaque contenu.\n\n\nJ'ai donc changé la donne en mettant 30 caractères (car c'est ce que \nnous demande notre métier, 'système d'exploitation' fait un poil trop de \ncaractères par exemple) et en corrigeant la méthode d'obtention des tags \npour obtenir à chaque fois le bon.\n\nLe 09/05/2016 à 16:52, Gérard Paligot a écrit :\n>\n>       * parce que c'est le fait que du coup le tag soit tronqué qui\n>         générait des problèmes\n>\n> Pourquoi le fait de tronquer un tag générait plusieurs tags ?\n>\n>       * parce que maintenant que j'ai changé à 30 caractères plutôt\n>         que 20 le \"troncage\" se fera à 31 caractères et donc il faut\n>         tester ma méthodes \"get_tag\" dans ce moment là.\n>\n> Pourquoi ne pas tester cette méthode |get_tag| à 20 caractères ?\n>\n> —\n> You are receiving this because you authored the thread.\n> Reply to this email directly or view it on GitHub \n> <https://github.com/zestedesavoir/zds-site/pull/3601#issuecomment-217887409>\n>\n\n"}}