Vous avez un lien vers le code JS qui s'occupe de ça svp ? Je peux jeter un oeil.
Ca doit trainer la dedans : https://github.com/zestedesavoir/zds-site/blob/dev/assets/js/autocompletion.js
J'ai pas le bug sur un CTRL+V en créant un MP.
Ça fait bien qu'une seule requête quand on relâche le "V".
Zêtes sûrs de votre coup ?
Sur et certain pour ma part… Ci ici j’écris "Javier" et que je copie puis colle ca dans le champ participants ca me crame plein d'appel et la console js me pond un Uncaught RangeError: Maximum call stack size exceeded. (chrome Ubuntu).
Pas moi. Firefox, ArchLinux.
Ah ouais, ça me le faisait pas y'a 2 minutes. Pourtant j'avais un truc (pas un pseudo, du texte je crois) dans le presse-papier.
J'essaie d'y jeter un oeil, mais à mon avis ça se passe là-dedans le truc c'est qu'installer une version de ZdS en local avec tout le bouzin d'installation juste pour avoir les sources JS non minifiées ça m'embête un peu…
J'essaie d'y jeter un oeil, mais à mon avis ça se passe là-dedans le truc c'est qu'installer une version de ZdS en local avec tout le bouzin d'installation juste pour avoir les sources JS non minifiées ça m'embête un peu…
Si tu veux j'ai une instance cloud9 que tu peux importer il me semble (et donc tout faire dans un navigateur) (info https://zestedesavoir.com/forums/sujet/3146/cloud9-pour-jouer-avec-zds/ )
Au passage, l'autocomplétion retourne-t-elle un nombre maximal de résultats ? Quand j'entre "J" ou "Ja", je n'ai pas "Javier".
Je t'avoue ne pas comprendre c'est quoi le problème:
Qu'il n'était pas généré au moment où j'ai tenté le déploiement et que le script ne vérifie pas si le tag existe.
Quand aux scripts, non, clairement je les lis pas, pour la simple raison que c'est impossible à QA (la QA, c'est justement la release), et que quand bien même, je sais pas ce qu'il y a derrière.
Et c'est pour ça que personne ne lit les PR infra: on ne PEUT PAS savoir si ce que tu fais est juste.
Ici c'est un faux argument. N'importe qui qui connaît un minimum le Shell pouvait se rendre compte du fait que l'on checkout un tag qui n'était pas celui passé en paramètre et que rien ne garantit qu'il existe.
Qu'il n'était pas généré au moment où j'ai tenté le déploiement et que le script ne vérifie pas si le tag existe.
Au delà du fait qu'en effet, on vérifie pas que le tag existe, il est LOGIQUE que ce tag va mettre un peu de temps à être créé puisque Travis doit passer dessus. À moins de mettre un sleep() (ou son équivalent bash) dans le script, il faut séparer ledit script en deux.
Ça veut dire que c'est pas une erreur du type "personne il veut relire mes scripts", c'est une erreur de conception, et on peut pas penser à tout, surtout que la QA ne peut pas vérifier ce genre de chose (non-argument ou pas).
Ici c'est un faux argument. N'importe qui qui connaît un minimum le Shell pouvait se rendre compte du fait que l'on checkout un tag qui n'était pas celui passé en paramètre et que rien ne garantit qu'il existe.
D'accord, c'est un faux argument, mais en attendant, quand je vois une PR taggué "infra", je me dit quand même que je pourrait probablement rien y faire parce que c'est votre popote interne. Pour être sur que ce genre de chose fonctionne, il faudrait carrément que quelqu'un aie un serveur iso-prod quelque part pour tester les PRs infra ou tout ce qui touche au déploiement.
Bref. On va rentrer dans un non-débat sans fin qui n'apportera rien. La prochaine fois, on lira le script avant, et en attendant, il va falloir patcher.
Et zou un ticket est fait pour y penser, fin du non-debat 
https://github.com/zestedesavoir/zds-site/issues/2842
Au passage, la liste des tickets m'avait ete demande, la voici !
Rapport pour le jalon Version 15.6 ()
105 tickets sont compris dans ce jalon (0 ouverts et 105 fermés)
Aucun ticket
| Ticket # | Titre | Label(s) |
|---|---|---|
| #2819 | [15.6] Impossible d'indexer un tutoriel dont une partie n'est pas publié | Back, Bloquant, BUG |
| #2816 | Réinitialise le cache à chaque test de l'api | API, BUG |
| #2804 | Utilisation de urlquote pour la page de profil d'un membre | Back, BUG |
| #2801 | Corrige l'indexation des catégories des tutos | Back, Bloquant, Régression |
| #2798 | Retablit le comportement des alertes (compteur et limite) | Back, Front, Régression |
| #2796 | Le staff "voit" les articles des autres membres comme si c'etait les siens | Back, Régression |
| #2793 | Le compteur d'alertes ne va pas au delà de dix | Back, Front, Régression |
| #2792 | On ne peut plus indexer de tutoriels | Back, Bloquant, Régression |
| #2787 | Les alertes rouges sanglantes ne se voient/lisent plus dans les commentaires d'article/tutoriel | Back, Bloquant, Front, Régression |
| #2756 | Suppression du cache | BUG, Front |
| #2753 | [beta 15.5.1] L'API est kaputt => erreur 500 | API, Bloquant, Régression |
| #2752 | [beta 15.5.1] l'item article dans la speedbar est toujours souligné | Front, Régression |
| #2751 | [15.5.1] Erreur 500 lors de la desinscription | Back, Bloquant, Régression |
| #2749 | [15.5.1] Dernier message non lu d'un article | Front, Régression |
| #2727 | API des membres + Cache = KBOOOM ! | API, Bloquant, BUG |
| #2709 | Importe la bonne version de gettext | Back, Régression |
| #2690 | Les tests fronts ne passent plus sur Travis | BUG, Front |
| #2677 | [Beta 15.5.1] MAJ des dépendances | Back, BUG |
| #2676 | [Beta 15.5.1] Manque un espace dans les catégories de tutoriels sur l'accueil | BUG, Facile, Front |
| #2671 | [Beta 15.5.1] Problèmes de performances / accueil | Back, Régression |
| #2654 | correction du filtre de validation des tutoriels | Back, BUG |
| #2640 | Passer à une version supérieure à django 1.7.7 pour la prochaine release | Back, BUG |
| #2634 | Corrige deux petites erreur 500 | Back, BUG, Facile |
| #2633 | Erreur 500 lors de l'édition malencontreuse d'un message du forum | Back, BUG, Facile |
| #2632 | Impossible de remettre un champ de son profil à False |
API, BUG |
| #2631 | Erreur sur les profils quand on est admin | Back, Régression |
| #2629 | Remplacement d'un lien mort dans la doc | BUG, Documentation, Facile |
| #2624 | Liens morts dans le fichier README | BUG, Documentation, Facile |
| #2616 | Documentation qui ne compile plus : ugettext |
Back, BUG, Documentation |
| #2613 | Le champ licence passe par défaut à la première valeur de la liste lors de l'édition | Back, Régression |
| #2586 | L'objet du mail de création d'un MP n'est pas le meme que celui de la réponse | Back, Régression |
| #2562 | Plus moyen d'importer les fixtures "aides" | Back, BUG |
| #2555 | Galerie : Miniature qui subsiste après suppression d'images | Back, BUG |
| #2495 | L'item avec la pseudo de l'utilisateur dans la sidebar n'ouvre pas le profil | BUG, Front |
| #2483 | ReadTheDoc ne compile plus notre documentation | BUG, Documentation |
| #2396 | Un membre banni peut encore envoyer des MPs tant qu'il n'est pas déconnecté | Back, Régression |
| #2301 | Impossible de voir le profil de certains membres avec des caractères particuliers | Back, BUG |
| #2129 | Les titres dans la recherche sont tronqués | Back, BUG, Front |
| #1683 | Le menu de gauche de la page profil d'un membre incompréhensible sur mobile | BUG, Front |
| #1666 | Problème d'affichage sur « Tous les articles » | BUG, Front |
| #1653 | Le filtre du contenu en validation par catégorie ne marche pas | Back, BUG, Facile |
| #1634 | Les images .svg font planter la publication d'un tuto | Back, BUG |
| Ticket # | Titre | Label(s) |
|---|---|---|
| #2839 | [15.6] L'indexation des droits fait planter la recherche | Back, Evolution |
| #2786 | Personnalise le highlighter de la recherche | Back, Evolution |
| #2767 | Index les droits des posts et des forums | Back, Evolution |
| #2687 | Change l'approche des auteurs | Evolution, Facile |
| #2664 | Passer aux build dans des container sur travis | Evolution |
| #2660 | Renvoi d'email de confirmation | Back, Evolution |
| #2655 | Impossible d'activer un compte si pas d'email d'activation | Back, Evolution |
| #2646 | ZEP 4 - Nouvelle page d'accueil | Evolution, Front |
| #2630 | ZEP-23: API du module des MPs | Back, Evolution |
| #2619 | Faire un script de MAJ automatique du script de déploiement | Evolution, Facile |
| #2608 | Ajout d'un lien dans le README | Documentation, Evolution |
| #2604 | Présence de fichiers inutiles dans le dépot | Back, Evolution, Facile |
| #2592 | Suppression d'un fichier inutile | Back, Evolution |
| #2480 | Pas de formulaire sur la page de biographie | Back, Evolution, Front |
| #2471 | Build automatiquement le front a chaque release avec Travis | Evolution, Front |
| #2393 | Impossible de modifier le titre/sous-titre d'un MP | Back, Evolution |
| #983 | Les sujets de la corbeille sont visibles dans la recherche | Back, Evolution |
| Ticket # | Titre | Label(s) |
|---|---|---|
| #2835 | N'indexe plus les contenus non publiés (sur la bonne branche) | |
| #2829 | Ameliore (un peu) l'affichage des articles | Front |
| #2823 | [15.6] Met a jour la doc de deploiement | Documentation, Facile |
| #2800 | feat(api): Disables MP API. | |
| #2795 | L'affichage de mes articles en brouillon est troublant | Front |
| #2784 | Erreur 500 lorsque j'essaie de masquer un message sur le forum | Back |
| #2783 | Le compte admin ne peut plus faire de modération de topic. | Back |
| #2772 | Agrandit les champs d'url | Back |
| #2766 | Indexe uniquement les champs utiles dans les post | Back |
| #2764 | Indexe qu'une seule fois le premier post | Back |
| #2763 | Cache les posts non visible | Back |
| #2762 | Les sujets masqués sont quand même affichés dans la recherche | Back |
| #2755 | [beta 15.5.1] Impossible de créer une Une dont l'url est trop longue | Back |
| #2745 | [Beta 15.5.1]Liste des tutoriels pas toujours par 3 | Front |
| #2741 | Rajout d'un lien aide pour les maths | Front |
| #2740 | [beta 15.5.1] Le lien de présentation de l'association ne pointe vers rien | Facile, Front |
| #2735 | [Beta 15.5.1] Erreur 500 à la consultation des tutoriels d'un membre | Front |
| #2732 | [Beta 15.5.1] Les sujets non-public sont bien caché sur la home page | Back |
| #2731 | [Beta 15.5.1] Les sujets cachés sont affichés sur la home | Back |
| #2729 | Ajout de $ manquants dans les URL de certaines pages (fix #2728) | Back, Facile |
| #2728 | Erreurs 404 non générées sur certaines pages | Back, Facile |
| #2726 | Corrections typos, indentation et traductions | Front |
| #2724 | [Beta 15.5.1] Amélioration des performances des pages liste des article, liste des tutos, home | Back |
| #2712 | Ameliorations de code conseillées par landscape | Back |
| #2710 | [15.5.1] Les liens des derniers messages ne sont pas clicables | Back, Front |
| #2695 | Derniers sujets sur la Home : la date sur mobile n'est pas optimisée | Front |
| #2694 | [Beta 15.5.1] Nouvelle home et Windows Phone | Front |
| #2692 | [Beta 15.5.1] La rubrique mes tutoriels me renvoit vers la version online d'un tutoriel | Facile, Front |
| #2675 | [Beta 15.5.1] Erreur de punchline sur mobile | Facile, Front |
| #2674 | Les filtres de tags "complexes" sur les articles ne fonctionnent pas | Back, Facile |
| #2670 | [Beta 15.5.1] Ajouter la taille voulue de l'image dans l'interface des unes | Facile, Front |
| #2669 | [Beta 15.5.1] Problèmes de performances / liste des tutos | Back |
| #2667 | [Beta 15.5.1] Le design de la page d'aide aux tutos est tout cassé | Front |
| #2662 | Il y'a des boutons inutilisables dans les MPs | Front |
| #2659 | Le fichier AUTHORS n'est pas à jour | Facile |
| #2650 | Nettoyage du code de la bannière de cookie | Front |
| #2644 | Change "/api/membres/mon_profil/" en "/api/membres/mon-profil/" | API |
| #2626 | Liste des forums en doubles | Back, Facile |
| #2618 | Checkout le bon tag, avec le front buildé (deploy.sh) | Infra |
| #2576 | Ajout de la partie sur les tags | Documentation |
| #2571 | Bump des dépendances npm + clean du Gulpfile + fix les warnings JSHint | Front |
| #2544 | Problème d'affichage de la bannière cookies sur mobile | Front |
| #2509 | Avertissement anti-déterrage de sujet | Facile |
| #2410 | Tox ne gère pas correctement les BDD pour les tests | Back |
| #2247 | Pourquoi tox et comment ça marche ? | Back, Documentation, Front |
| #1511 | Les articles ne sont pas indexés par Solr | Back |
La release vient d'être mise à jour en RC2. Go go go pour le test !
@L'equipe Com' (ShigeruM, elyp', Taguan) : vos outils com' ont un peu évolué selon vos retours. Allez tester 
@Hugo : L'indexation a bien marché jusqu'au bout, Bravo !
Du coup ca veut dire que la recherche au sein du site marche beaucoup (beaucoup) mieux !
Sinon pour spacefox : je sais pas si c'est moi qui est mal géré mais le fait de changer les droits sur les scripts de deploiements (qui ne sont pas executables par defaut) ca m'a chier dans les mains au moment du git checkout (normal). Du coup le plus simple serait surement de dire a git d'ignorer les changements de droits sur les fichiers (c'est possible dans la config) pour pas que ca se reproduise…
Sinon pour spacefox : je sais pas si c'est moi qui est mal géré mais le fait de changer les droits sur les scripts de deploiements (qui ne sont pas executables par defaut) ca m'a chier dans les mains au moment du git checkout (normal). Du coup le plus simple serait surement de dire a git d'ignorer les changements de droits sur les fichiers (c'est possible dans la config) pour pas que ca se reproduise…
Ou faire une PR la prochaine fois ?
a tester en mettant les bons droits ouai…
La release vient d'être mise à jour en RC2. Go go go pour le test !
Génial, bravo à toi pour ta deuxième MEB ! On espère que tu as pas trop galéré !
@L'equipe Com' (ShigeruM, elyp', Taguan) : vos outils com' ont un peu évolué selon vos retours. Allez tester
J'ajoute que de parler de l'api des MP, qui a été désactivé pour des raisons techniques, c'est pas super. Mais sinon, ça rend drôlement bien !
@Hugo : L'indexation a bien marché jusqu'au bout, Bravo !
Un bon gros soulagement. Merci d'avoir pris le temps de faire le retour.
Du coup le plus simple serait surement de dire a git d'ignorer les changements de droits sur les fichiers (c'est possible dans la config) pour pas que ca se reproduise…
+1
Bravo pour la recherche ! elle est enfin utilisable !
La recherche "Android" en prod et sur la bêta, c'est le jour et la nuit.
Il y a encore du chemin pour que ça devienne ergonome mais ça rend la recherche enfin utilisable et un grand bravo pour ça pour les contributeurs de cette release !
@L'equipe Com' (ShigeruM, elyp', Taguan) : vos outils com' ont un peu évolué selon vos retours. Allez tester
Je n'ai pas encore pu tester tout à 100%, mais pour l'instant je n'ai remarqué aucun problème. Et c'est vraiment top ces nouveautés ! Bravo et merci !
Je crois qu'on est bon pour la mise en production là (tous les tickets bloquants ont l'air résolus) ?
Situphen doucement les chevaux, la release a encore une semaine pour finir de faire ses preuves !
En fait, je crois qu'il manque juste l'amélioration du script de déploiement mais je sais que Sandhose a une PR dans les bacs !
Situphen doucement les chevaux, la release a encore une semaine pour finir de faire ses preuves !
Ouais enfin ça fait quand même trois semaines (si on compte la précédente version abandonnée) qu'elle est en béta et elle est assez attendue ! Après je vous demande pas de la sortir demain, je secoue juste le sac pour qu'on ne l'oublie pas.
Créez un compte en une minute pour profiter pleinement de toutes les fonctionnalités de Zeste de Savoir. Ici, tout est gratuit et sans publicité.
Créer un compte