Adrien Dorsaz a écrit 952 commentaires

Pour la réorganisation, on a, dans le modèle de news, le code:

  def reorganize(params)
    Paragraph.where(news_id: self.id).delete_all
    self.attributes = params
    create_parts
    save
    unlock
  end

On voit bien que la première étape est de supprimer tous les paragraphes.

La ligne create_parts va recréer tous les paragraphes de la première et de la seconde partie.

L'appel à create_new_version est définit pour s'exécuter après le save (after_save).

Il faut savoir que, si un lien est invalide, le modèle link lève une exception sans gestion d'erreur et donc fait arrêter nette la procédure.

Ce que je ne sais pas, c'est si le modèle de link est appelé au moment de faire self.attributes = params ou au moment de faire save.

En plus, je ne sais pas si Rails fait des transactions dans MySQL, donc c'est un peu compliqué à expliquer ce qui se passe juste en regardant le code. Il faudrait que je reproduise aussi pour mieux analyser.

Mais en tout cas, s'il n'y a pas de transaction SQL, la news serait effectivement dans un mauvais état: les paragraphes ont été détruits et les nouveaux body/wiki_body n'ont pas été sauvés.

Mon Pull Request fait en sorte que le modèle link ne lève plus une exception brute, mais remonte une erreur de validtion. Au moins, ça n'arrête plus le processus d'un coup, mais ça perturbe uniquement les modifications sur les liens.

Hello,

J'ai ajouté la validation des liens HTTP dans ce Pull Request: https://github.com/linuxfrorg/linuxfr.org/pull/309

Avant, on n'affichait aucune des erreurs de validation, j'ai ajouté les messages d'erreurs dans ces cas. J'ai dû mettre à jour un peu le rendu des liens en édition dans l'espace de rédaction:

J'ai ajouté un cadre autour du formulaire, pour bien montrer à l'utilisateur que tous ces boutons sont liés.

Dans le lien "en édition", j'ai profité d'expliquer que le lien peut être supprimé en vidant le champ Adresse.

Quand plusieurs erreurs existent, une liste à puce est affichée. Si le lien est en modification, les puces sont les drapeaux de langue. Je n'ai pas corrigé ça, parce qu'on a déjà un Pull Request pour enlever les drapeaux de langue dans les liens.

Merci pour toutes ces informations, je vais voir ce que j'arrive à faire.

Et il faut aussi faire très attention à la version de Ruby : on a une version de Ruby assez ancienne en prod et j'ai plusieurs fois dû faire des reverts à cause de ça.

Est-ce que la version de Ruby est plus vieille que celle disponible sur Debian Stretch (version 2.3.3) ?

Je comptais utiliser les images Docker que j'ai configuré et qui travaillent justement avec Debian Stretch pour l'instant.

Apple aime vendre très cher son matériel, mais il ne se gène pas non plus pour te tracer ou permettre aux applications tiers de te tracer.

Vu justement la position de Facebook suite aux changements de permission des applications dans iOS, ton affirmation demande une source.

Euh, tu viens donc justement de prouver que Facebook et les autres pouvaient ces 10 dernières années se servir ?

Et donc, malgré le prix du téléphone qui a toujours était élevé, il n'y avait pas plus de discrétion.

Que le modèle économique d’Apple et de vendre du matériel (cher), pas mes données personnelles.

C'est très intéressant comme point de vue, mais je pense que c'est certainement faux.

Apple aime vendre très cher son matériel, mais il ne se gène pas non plus pour te tracer ou permettre aux applications tiers de te tracer.

C'est pareil chez Google, mais ça fait tomber l'argument du prix.

D’autant que la confiance aveugle est généralement le plus court chemin pour se faire avoir.

Justement, vu la citation plus haut sans justification, il y a une confiance aveugle dans Apple également ;)

Préambule: c'est moi qui est ouvert l'entrée de suivi, j'ai oublié de me connecter avant :)

Comme je ne connais pas moi-même rails, j'ai cherché la manière de faire avec Rails.

D'après son tutorial, rails a déjà son propre framework de test: https://guides.rubyonrails.org/testing.html

J'ai donc commencé par chercher dans l'historique git et j'ai découvert qu'il y avait des tests dans le dossier test (dossier utilisé par rails, justement).

Puis, ce dossier a été supprimé en faveur du framework de test rspec2 qui avait ses tests dans le dossier spec.

Depuis le commit eaf8458858ab4fff7dffd86ae396c64f0d76e753, on a supprimé tous les tests.

Avant de pouvoir commencer, j'ai donc plusieurs questions:

• pourquoi est-ce que l'on est passé au framework rspec2 ?

• le changement de framework a était fait il y a très longtemps (en 2010), si je recommence de zéro, on pourrait à nouveau utiliser le framework de rails ?

  • Je préférerais utiliser celui de rails, car le tutoriel de rails est plutôt complet (pour les autres aspects du framework en tout cas) et comme ça on peut diriger les bonnes volontés sur leur tutoriel.

• est-ce que les tests qui étaient dans le dossier spec étaient vraiment trop cassé ? est-ce que je devrais recommencer de zéro ? étaient-ils utiles ?

Bonne nouvelle, j'ai repris le code et, en fait, je m'étais complètement trompé :)

Il n'y avait pas besoin de passer par JavaScript, je devais simplement couper le template de l'email en 3 parties.

Voilà le résultat:

En travaillant là-dessus, je me suis en plus rendu compte qu'il y avait besoin de faire une petite correction CSS dans l'espace de rédaction pour les écrans mobiles: il y avait trop de marge autour du texte sur les petits écrans, j'ai ajusté ça pour être raccord avec le reste du site mobile.

J'ai préparé un Pull Request pour ça: https://github.com/linuxfrorg/linuxfr.org/pull/308

Merci beaucoup pour la dépêche et le travail sur Gimp !

Je trouve l'illustration "work in progress" vraiment bien trouvée, c'est exactement l'esprit que j'aime avec les logiciels libres :)

Pour information, il y avait cette demande en double: https://linuxfr.org/suivi/suggestion-concernant-l-interface-de-redaction-des-depeches

Je n'avais pas vu que, en peu de temps, vous étiez les 2 à faire la même suggestion 😅

Hello,

Je me suis lancé aujourd'hui dans ma première contribution à une dépêche, et je trouve l'interface de rédaction vraiment bien pensée et très pratique ! Félicitations :)

Merci :)

Le seul point qui m'a un peu surpris concerne le comportement de césure de l'éditeur de texte : lorsqu'on écrit le texte au format markdown dans la boîte à cet effet, l'éditeur applique automatiquement une césure à la fin des lignes avec l'ajout d'un tiret -. Seul l'affichage est modifié : le text brut sous-jacent n'est pas modifié (par exemple, un mot affiché tronqué peut être copié-collé sur une nouvelle ligne et le tiret disparaîtra).

Ce comportement est profitable pour l'affichage du texte dans le rendu html, mais cela m'a un peu déstabilisé dans l'éditeur lui-même : je me demandais parfois si j'avais fait une faute de frappe et inséré un tiret indésirable, ou bien si le tiret était celui de l'affichage de césure.

Il me semblerait bénéfique de désactiver ce comportement d'affichage de césure dans la boîte d'édition de texte brut afin de n'afficher que le texte tapé. Curieusement, je ne retrouve pas ce comportement dans la boîte de texte utilisée pour écrire cette suggestion ? :)

Je suis tout à fait d'accord avec toi et j'ai proposé une correction dans cette entrée de suivi.

Comme c'est un doublon de demande, je vais clôturer ici et on pourra continuer à discuter là-bas si jamais.

Hello,

J'ai regardé un peu comment le code fonctionne actuellement.

L'idée actuelle est que le TEXTE DE LA RELANCE est inséré dans l'email et, également, directement dans la discussion sur le côté de la dépêche.

1) que l'on voie le texte complet de la relance quand on effectue une relance depuis l'espace de rédaction

Au début, je pensais mettre directement l'email complet dans la zone de texte.

Mais à cause de l'utilisation du message de relance dans la discussion, je ne peux pas le faire.

En plus, je dois travailler côté javascript et je crois que je n'arrive pas à récupérer le texte de l'email depuis javascript.

Je suis un peu coincé pour cette idée, désolé…

2) que le texte de la relance soit modifié ainsi

Bonjour,

vous avez créé ou contribué à la dépêche en rédaction :
TITRE DE LA DÉPÊCHE CONCERNÉE

TEXTE DE LA RELANCE

Vous pouvez retrouver et compléter votre dépêche à cette adresse :
https://linuxfr.org/redaction/news/TITRE_DE_LA_DÉPÊCHE

Merci d’avance.

--
L’équipe de LinuxFr.org

Ça c'est faisable :)

J'ai juste une question: quelle phrase générique pourrais-je mettre si aucun TEXT DE LA RELANCE n'est renseigné ?

Quelque chose comme:

L'équipe d'animation vous encourage à finaliser la dépêche.

Est-ce que ça pourrait jouer ? Est-ce que tu as une meilleure idée ?

Merci pour l'idée !

C'est vrai que, quand on édite un texte, ça ne sert pas à grand chose cette césure automatique, puisqu'elle ne sera certainement pas placée au même endroit lors du rendu réel.

J'ai préparé un merge request qui enlève la césure automatique pour tous les textarea.

Hello,

Sais-tu déjà quel langage tu souhaites initier ? Ça pourrait guider certaines propositions.

J'ai souvenir d'avoir débuté entre autre avec Notepad++ pour du PHP/HTML, mais c'était sous Windows et ce n'est pas vraiment un IDE.

Ensuite, on avait fait un peu de C et là (vers 15-16 ans je pense), j'avais pu utiliser Code::Blocks qui était pas mal. Le C était utilisé pour faire des maths appliquées, alors on n'avait pas utilisé beaucoup de fonctionnalités, mais on avait pu tester le debugger.

C'est très intéressant de voir le cheminement vers ce nouveau logo et cette nouvelle marque.

J'aime bien leur proposition finale, mais je trouve dommage d'avoir perdu le signe de l'infini. Je pense qu'il ne se voit plus et que, à la longue, on va penser que c'est juste un f stylisé.

En plus, comme le logo est placé au côté gauche du texte fedora, on perd la signification de la forme du fond bleu qui était un phylactère.

Par contre, utiliser une police de caractère libre est vraiment une évidence de nos jours :)

Bref, mon avis n'est pas très important et je pense qu'ils ont déjà eu ce genre de retours de la part de la communauté fedora.

En fait, la question que je me pose vraiment, c'est que, finalement, au lieu de changer le logo, il aurait peut être été plus simple d'assouplir les règles d'utilisation du logo et de la marque ?

Par exemple, pour leur soucis de placement sur des fonds sombres, il aurait été peut être plus simple d'accepter de mettre une bordure claire autour du phylactère ? Et même de proposer une version officielle pour gérer ce cas ?

J'ai vraiment l'impression qu'une grande partie de ce travail aurait pu être évité par la simple adaptation de 2-3 règles (et, éventuellement, la définition de plusieurs variantes officiellement acceptées).

Désolée, en ce qui me concerne, je suis absolument défavorable à un nettoyage automatique

Et pourquoi donc ? Quel est l'intérêt de garder des dépêches débutées et jamais terminées ?

De mon point de vue, c'est intéressant de trier, car c'est vraiment décourageant pour les rédacteurs de voir "Actuellement 42 dépêches sont en cours de rédaction". En tant que nouveau rédacteur, je me poserai ce genre de questions:

• Par où commencer à contribuer ?
• Est-ce que j'ose démarrer la 43ème dépêche en cours de rédaction ?
• Est-ce que c'est vraiment très difficile de proposer une dépêche ?
• Qu'est-ce qui pourrait faire que la dépêche reste 5 ans en rédaction ?

Or, je trouve que le côté le plus intéressant de LinuxFr, c'est les nouvelles collaboratives. Sinon, LinuxFr serait juste un forum / discourse communautaire comme il y en a tant sur Internet.

je pense que l'urgence est plutôt d'améliorer par exemple l'interface de saisie des commentaires

Intéressant, de ton point de vue il y a une urgence à ce sujet, alors qu'il n'y a aucune entrée de suivi ouverte liée à ça :) Ou alors je ne l'ai pas trouvée.

en permettant qu'on puisse tout voir quand on répond à un contenu ou à un commentaire, ce qui facilite les dérapages en tous genres.

Désolé, mais je ne vois pas trop le lien entre les dérapages et voir tout le contenu pendant qu'on saisi un commentaire ?

Celui qui a décidé de lancer / répondre à un troll le fera même si tous les commentaires sont affichés sur la page…

En tout cas, il faudra expliquer ce point de vue dans l'entrée de suivi :)

Effectivement, je pense que l'on n'avait simplement pas prévu que les dépêches restent 5 ans dans l'espace de rédaction :)

Pour le coup, il serait peut être vraiment intéressant que je termine le code pour le nettoyage automatique de l'espace de rédaction.

Je l'avais débuté pour cette entrée de suivi.

ah oui, bizarre, je me demande comment j'y suis arrivé, désolé pour le bruit :)

Edit:

Ah j'ai compris, j'ai cliqué sur le lien à propos des seuils d'acceptation des dépêches.

Et j'ai continué à lire celui-ci en croyant être sur celui d'avril :)

finaliser le rapport moral et préparer l’assemblée générale de LinuxFr ;

Wow, on est prévenant, on prévoit déjà celle de cet automne ? 😅

La dernière assemblée a été faite début avril et le rapport est publié dans cette dépêche.

Si j'ai bien compris la réponse, c'était "ok, on a réservé le port 22 pour SSH". Ils ne disent pas avoir audité et validé le RFC pour SSH.

C'est ça qui est fou en fait: l'auteur de SSH a pris pas mal de temps à faire son mail pour demander la réservation du port, il a pris la peine de joindre un brouillon de RFC et d'avoir une version stable presque prête et le lendemain (très rapidement) on lui répond juste "ok" 😅

Oui, tout à fait, Mathieu avait proposé d'abandonner ce système lorsque l'on passera à son nouveau design.

D'ailleurs, le travaille sur l'espace de rédaction était une partie de ce projet qui n'est pas encore terminé :)

Hello,

J'ai fermé cette entrée de suivi, parce que, comme indiqué plus bas, c'est un doublon.

L'espace de rédaction avait été fortement modifiée il y a 2 ans pour avoir une barre latérale dynamique et ça a cassé tout les thèmes alternatifs.

Hello,

Je met dans ce fil les demandes de suivis en doublon:

• RonRonnement-Classic et espace de rédaction
• L'espace de rédaction des dépêches est cassé avec le thème kaiska-short

Hello,

Je ferme la demande, parce qu'elle est en doublon à propos de la barre latérale de l'espace de rédaction.

Hello,

Les images de la demande de suivi ne sont plus accessibles.

Je ferme la demande, parce que je pense que c'est un doublon à propos de la barre latérale.

Merci !