Maison > interface Web > js tutoriel > Relancer un projet obsolète

Relancer un projet obsolète

Linda Hamilton
Libérer: 2025-01-22 06:30:16
original
108 Les gens l'ont consulté

Cette semaine, j'ai effectué quelques travaux de maintenance sur Starchart. Le projet n'a pas été travaillé depuis un moment, nous essayons donc de mettre à jour ses dépendances.

GitHub logo DéveloppementEspace / diagramme d'amidon

Un outil en libre-service pour gérer les domaines et certificats personnalisés

Carte des étoiles License: MIT

Starchart permet à la communauté des développeurs Seneca de créer et de gérer facilement ses propres sous-domaines personnalisés et certificats SSL, sans frais ni devoir fournir d'informations personnelles.

Pour plus d'informations sur l'exécution de Starchart, consultez notre guide de déploiement. Pour plus d’informations sur le développement, consultez notre guide de contribution. Pour plus d'informations techniques, de planification et de conceptions initiales, veuillez consulter le wiki.

Présentation

Internet évolue et ce qui était difficile est devenu simple. Par exemple, héberger votre propre site Web nécessitait auparavant des connaissances en administration de serveur, en systèmes d'exploitation, en réseau, etc. Aujourd'hui, de nombreux développeurs hébergent leurs sites Web personnels et de projet sans jamais toucher à un serveur distant, optant pour des services cloud (gratuits) comme GitHub Pages, Vercel, Netlify ou AWS.

Le modèle de sécurité d'Internet évolue également. Par exemple, les fournisseurs de navigateurs ont adopté HTTPS partout. C'est bon pour la sécurité, car cela permet un cryptage basé sur des certificats entre les clients et les serveurs. Cependant, comme avec…

Voir sur GitHub

Le plan était de réparer le flux de travail CI, dont nous avons découvert qu'il était défectueux la semaine dernière :

Image description

Mais avant que je puisse trouver une solution, l'un des développeurs précédents, Eakam, a résolu le problème - il s'est avéré que c'était simplement parce que Playwright était obsolète.

Relancer un projet obsolète Faire passer le dramaturge à la version 1.49.1 #772

Relancer un projet obsolète
Eakam1007 publié le

L'installation de Playwright échoue dans CI (tests E2E). La version du dramaturge devrait résoudre ce problème.

Réf : Installation error log

Voir sur GitHub

J'avais l'impression que je devrais compenser cela en trouvant plus de choses sur lesquelles travailler et j'ai pensé que mettre à jour davantage de dépendances serait un excellent point de départ.

Comme le projet n'avait pas été travaillé depuis 2 ans, il y avait un tas de failles de sécurité provenant de packages obsolètes. J'ai pu réparer la plupart d'entre eux avec npm audit fix.

Il y a eu quelques correctifs supplémentaires qui ont conduit à des modifications importantes dans @remix-run/eslint-config et @remix-run/react, donc je les ai corrigés manuellement.

L'une des mises à jour (je les ai cognées en même temps donc je ne peux pas le dire avec certitude mais mon pari est sur /react) a conduit à une erreur de vérification de type car [@remix-run/react].useNavigation( ).formData peut désormais être du type non défini. Je l'ai corrigé avec un chaînage optionnel.

// Before
const isLoading =
  navigation.state === 'submitting' &&
  Number(navigation.formData.get('id')) === dnsRecord.id;

// After
const isLoading =
  navigation.state === 'submitting' &&
  Number(navigation.formData?.get('id')) === dnsRecord.id;
Copier après la connexion
Copier après la connexion
Copier après la connexion

Les autres modifications que j'ai apportées étaient liées à des erreurs de charpie qui sont apparues (à ce stade, j'ai réalisé que mon extension ESLint était désactivée, mais je suis sûr que ces avertissements sont venus avec la mise à jour, car cela ne s'est jamais produit dans CI dans le passé).

  1. Instances d'importation du même module plusieurs fois dans un seul fichier :
// Before
import { getCertificateByUsername } from '~/models/certificate.server';
import { deleteCertificateById } from '~/models/certificate.server';
import { isAdmin } from '~/models/user.server';
import { getUserByUsername } from '~/models/user.server';

// After
import { getCertificateByUsername, deleteCertificateById } from '~/models/certificate.server';
import { isAdmin, getUserByUsername } from '~/models/user.server';
Copier après la connexion
  1. Utiliser let lorsque const est préféré :
// Before
let date = val.toLocaleDateString('en-US', {

// After
const date = val.toLocaleDateString('en-US', {
Copier après la connexion

Surpris qu'il ne les ait pas détectés avant.

De plus, lorsque j'ai activé l'extension ESLint, j'ai été un peu surpris car il y avait environ 900 erreurs de linter. Il s'est avéré que c'était parce qu'ESLint peluchait la sortie générée par Playwright. J'ai donc ajouté /playwright-report à .eslintignore.

Et c'était la somme de mon travail de maintenance pour ce sprint. J'ai fini par résoudre 30 problèmes de sécurité graves, donc pas mal.

Relancer un projet obsolète Mettre à jour les dépendances #775

Relancer un projet obsolète
ouday-rana publié le

Devrait corriger un tas de vulnérabilités de sécurité.

Modifications

  • [x] Dépendances de bosse
  • [x] Ajouter /playwright-report à .eslintignore
  • [x] Correction des erreurs de vérification de type et de linter
Voir sur GitHub

J'ai également réactivé Dependabot, ce qui a modifié quelques versions mineures. Ce serait bien de ne pas avoir à enquêter et à corriger manuellement les vulnérabilités de sécurité.

Par ailleurs, une de mes pull request vers Mattermost a finalement été fusionnée !

Relancer un projet obsolète [GH-29548] Évitez SELECT * dans `tokens_store.go` #29558

Relancer un projet obsolète
ouday-rana publié le

Résumé

Ce PR :

  • Bascule les requêtes SQL dans token_store.go pour utiliser SQLBuilder
  • Définit explicitement les colonnes dans les requêtes SELECT vers TokenStore.
  • Factifie les requêtes courantes dans le constructeur.

Lien du billet

Corrections #29548

Captures d'écran

Note de version

// Before
const isLoading =
  navigation.state === 'submitting' &&
  Number(navigation.formData.get('id')) === dnsRecord.id;

// After
const isLoading =
  navigation.state === 'submitting' &&
  Number(navigation.formData?.get('id')) === dnsRecord.id;
Copier après la connexion
Copier après la connexion
Copier après la connexion
Voir sur GitHub

Il avait été approuvé il y a quelque temps mais il a fallu quelques semaines pour être fusionné avec main.

En attendant, je travaille sur mes autres relations publiques. On m'a demandé d'apporter quelques modifications et j'attends une nouvelle révision.

Relancer un projet obsolète [MM-53650] Ajouter un paramètre de désactivation du rendu des émoticônes à l'application Web #29414

Relancer un projet obsolète
ouday-rana publié le

Résumé

Cette pull request ajoute un paramètre utilisateur à l'application Web pour basculer le rendu des émoticônes (:D) sous forme d'emojis (?).

Le paramètre est ajouté en tant que composant dans Components/user_settings/display/render_emoticons_as_emoji/ qui est importé dans Components/user_settings/display/user_settings_display.tsx.

J'ai ajouté une fonction renderOnOffLabel() à user_settings_display.tsx, extraite de components/user_settings/advanced/user_settings_advanced.tsx pour aider au rendu du nouveau composant.

Le paramètre est stocké en tant que préférence utilisateur à l'aide de l'action savePreferences().

J'ai ajouté des constantes pour la préférence à utils/constants.tsx et webapp/channels/src/packages/mattermost-redux/src/constants/preferences.ts.

Pour utiliser réellement le paramètre, j'ai modifié composants/post_markdown pour recevoir sa valeur en tant qu'accessoire, pour lequel j'ai utilisé getBool() et ajouté une valeur par défaut à la configuration. post_markdown transmet cette valeur à Markdown sur l'objet options, qui la transmet ensuite à utils/text_formatting.tsx, qui transmet finalement la valeur à emoticons.tsx en tant que paramètre nouvellement ajouté. emoticons.tsx vérifie si la valeur est vraie et si c'est le cas, il transforme les émoticônes en emojis.

J'ai mis à jour les tests concernés et créé des tests unitaires pour le nouveau composant. J'ai également mis à jour le fichier de traduction en anglais.

Étapes du test QA
  1. Accédez aux paramètres utilisateur.
  2. Allez dans la catégorie Affichage.
  3. Recherchez la section intitulée « Rendu automatique des émoticônes en tant qu'emoji » et cliquez sur « Modifier ».
  4. Basculez le paramètre et cliquez sur "Enregistrer".
  5. Le rendu des émoticônes sur les messages envoyés par l'utilisateur actuel et les autres utilisateurs doit être activé côté client avec le paramètre.

Lien du billet

Corrections (partiellement) https://github.com/mattermost/mattermost/issues/26504 Jira https://mattermost.atlassian.net/browse/MM-53650

Remarque le problème et le ticket décrivent également l'ajout de cette fonctionnalité à l'application mobile, ce que ce PR ne fait pas.

Captures d'écran

before after
Relancer un projet obsolète Relancer un projet obsolète

Note de version

// Before
const isLoading =
  navigation.state === 'submitting' &&
  Number(navigation.formData.get('id')) === dnsRecord.id;

// After
const isLoading =
  navigation.state === 'submitting' &&
  Number(navigation.formData?.get('id')) === dnsRecord.id;
Copier après la connexion
Copier après la connexion
Copier après la connexion
Voir sur GitHub

Travailler sur ce PR était intéressant car lorsque je l'ai soumis pour la première fois, je n'avais même pas entièrement compris mes modifications. Y revenir après une longue absence et grâce aux retours des critiques m'a aidé à le regarder sous un nouvel angle et à mieux le comprendre.

L'application Mattermost obtient l'état des paramètres utilisateur à partir des « préférences » et d'une « configuration ». J'ai ajouté mon paramètre aux deux, imitant l'un des paramètres existants qu'il m'a été conseillé de référencer, mais il s'est avéré que la "config" est destinée aux paramètres au niveau du serveur, alors que ce nouveau paramètre était destiné à être une option côté client. Les critiques m'ont aidé à comprendre où je m'étais trompé, et cela a fini par être un changement moindre que ce que je pensais nécessaire.

Dans l'ensemble, je dirais que ce fut une semaine assez productive.

Ce qui précède est le contenu détaillé de. pour plus d'informations, suivez d'autres articles connexes sur le site Web de PHP en chinois!

source:dev.to
Déclaration de ce site Web
Le contenu de cet article est volontairement contribué par les internautes et les droits d'auteur appartiennent à l'auteur original. Ce site n'assume aucune responsabilité légale correspondante. Si vous trouvez un contenu suspecté de plagiat ou de contrefaçon, veuillez contacter admin@php.cn
Derniers articles par auteur
Tutoriels populaires
Plus>
Derniers téléchargements
Plus>
effets Web
Code source du site Web
Matériel du site Web
Modèle frontal