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.
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.
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…
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 :
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.
L'installation de Playwright échoue dans CI (tests E2E). La version du dramaturge devrait résoudre ce problème.
Réf :
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;
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é).
// 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';
// Before let date = val.toLocaleDateString('en-US', { // After const date = val.toLocaleDateString('en-US', {
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.
Devrait corriger un tas de vulnérabilités de sécurité.
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 !
Ce PR :
Corrections #29548
// 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;
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.
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.
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.
before | after |
---|---|
// 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;
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!