Plots2: Les liens externes contenant un symbole « @ » sont actuellement analysés comme des noms d'utilisateur de laboratoire public.

Notes de Jeff :

@jywarren : Je pense que nous devons changer le matcher de modèle https://github.com/publiclab/plots2/blob/acd5a4910632c8cc80f9fadb54d8cc64c9600519/config/initializers/constants.rb#L2

Voici une démonstration de l'échec de la correspondance de modèle basée sur les liens originaux de Joe ; il doit être modifié pour ne PAS reconnaître les noms d'utilisateur à l'intérieur des URL : https://rubular.com/r/HsxxuTG8zG0zSL

@emilyashley puis-je travailler dessus ?

@Tlazypanda ce serait super, merci !!!

Salut @Tlazypanda , avez-vous eu l'occasion d'essayer des modifications à l'aide de cet éditeur d'expression ? https://rubular.com/r/HsxxuTG8zG0zSL Cela ressemble à un petit jeu, pour essayer d'ajuster la façon dont les ' @names ' sont reconnus sur notre site mais pas quand il y a un '/' de chaque côté.

Hey @ebarry, je suis vraiment désolé d'avoir totalement oublié ce problème en travaillant sur d'autres problèmes ... est-ce que ça va si je le reprends après quelques jours depuis que mes examens universitaires sont en cours :sweat_smile:

Bien sûr!!! Bonne chance pour les examens !!!

([^`\w]|^)(?<!\/)\@([\w-]+)\b(?!\/)

Semble fonctionner correctement. Pouvez-vous s'il vous plaît confirmer si cette regex est correcte ou non @ebarry ?

@jywarren Pouvez-vous le confirmer ?

J'aimerais pouvoir confirmer de toute façon, mais je ne connais pas regex, désolé pour ma réponse tardive, et MERCI d'avoir travaillé là-dessus !

Fonctionne parfaitement! https://rubular.com/r/jfKfgjdqi00qy4

Cela peut maintenant être inséré à constants.rb -- cela ferait aussi un bon problème pour les débutants !

Cela a été marqué comme un bon candidat pour devenir un problème réservé aux débutants comme celui- ci , ce qui signifie qu'il est simple, autonome et avec un formatage supplémentaire, pourrait être un excellent point d'entrée pour un nouveau contributeur. Si vous connaissez suffisamment ce code, envisagez de le reformater ou de le republier en tant que problème réservé aux débutants , puis envoyez un ping à @publiclab/reviewers pour qu'il soit étiqueté. Ou, si ce n'est pas votre première fois, essayez de le résoudre vous-même !

Merci d'avoir évalué ce @jywarren ! Merci Gaurav Ahlawat d'avoir fait

cette!

+1 336-269-1539 / @lizbarry http://twitter.com/lizbarry / lizbarry.net

Le mar. 5 janvier 2021 à 16h55 Jeffrey Warren [email protected]
a écrit:

Fonctionne parfaitement! https://rubular.com/r/jfKfgjdqi00qy4

[image : image]
https://user-images.githubusercontent.com/24359/103704067-a4875580-4f76-11eb-81ae-51ab06863ecb.png

Cela peut maintenant être inséré dans constants.rb -- cela ferait un bon
problème réservé aux débutants aussi !

Cela a été marqué comme un bon candidat pour devenir un débutant uniquement
problème http://www.firsttimersally.com comme ceux-ci
https://code.publiclab.org/#r=all , ce qui signifie que c'est simple,
autonome et avec un formatage supplémentaire, pourrait être une excellente entrée
point pour un nouveau contributeur. Si vous connaissez suffisamment ce code,
veuillez envisager de le reformater ou de le republier en tant que problème réservé aux débutants
https://publiclab.org/notes/warren/10-31-2016/create-a-welcoming-first-timers-only-issue-to-invite-new-software-contributors ,
puis ping @publiclab/reviewers
https://github.com/orgs/publiclab/teams/reviewers pour le faire étiqueter.

Ou, si ce n'est pas votre première fois, essayez de le résoudre vous-même !

-
Vous recevez ceci parce que vous avez été mentionné.
Répondez directement à cet e-mail, consultez-le sur GitHub
https://github.com/publiclab/plots2/issues/7675#issuecomment-754925581 ,
ou se désinscrire
https://github.com/notifications/unsubscribe-auth/AABHNH47FTWY45FYH34CQCDSYODDNANCNFSM4LN2YLEA
.

@ebarry Pourriez-vous s'il vous plaît le convertir en FTO, afin que d'autres puissent le résoudre ? Merci!

excellent travail @ gauravahlawat81 Je peux le faire pour vous... ou peut-être voudriez-vous essayer de le convertir vous-même ?... Je suis heureux de vous aider là où coincé... qu'en pensez-vous ?

Oui, bien sûr, j'aimerais le convertir moi-même. Comment je fais ça ?

Dois-je créer un nouveau problème ou modifier celui-ci ?

Super :tada:, oui, en créer un nouveau serait plus simple, lorsque vous cliquez sur un nouveau numéro ici https://github.com/publiclab/plots2/issues/new/choose sélectionnez le First timer Only Issue et il se préremplira un modèle... après cette ligne où vous avez **Update** the file [$FILENAME]($BRANCH_URL) in the `$REPO` repository (press the little pen Icon) and edit the line as shown below. indiquez le fichier qui doit être modifié et les modifications nécessaires... Donnez-lui un titre et ce devrait être tout

D' accord , merci

@cesswairimu Comment générer le diff ?

Je ne sais pas s'il existe une autre façon de le faire ... la façon dont je le fais utilise le premier bot
Donc, avec cette approche, vous accédez au fichier que vous souhaitez modifier dans ce cas, je pense qu'il s'agit de https://github.com/publiclab/plots2/blob/main/config/initializers/constants.rb et cliquez sur le Modifier le bouton (icône de crayon), apportez les modifications nécessaires, puis la zone de texte sous ce Create a new branch for this commit and start a pull request modifiez-la et assurez-vous qu'elle commence par first-timers- puis cliquez sur valider les modifications...

après cela, vous devriez voir votre problème créé ici https://github.com/publiclab/plots2/issues/ avec l'auteur en tant que premier-timer-bot
plus de documentation à ce sujet est ici si cela n'a pas de sens

J'ai créé le problème, même si j'ai dû utiliser une approche différente. Veuillez voir si tout va bien là-bas, et si quelque chose doit être changé, faites-le moi savoir.

Le problème est parfait..merci encore

Lien FTO https://github.com/publiclab/plots2/issues/8947

Corrigé dans https://github.com/publiclab/plots2/pull/8972 Merci à tous

Merci à tous!

Flux de travail très soigné

+1 336-269-1539 / @lizbarry http://twitter.com/lizbarry / lizbarry.net

Le vendredi 8 janvier 2021 à 4 h 59, Cess [email protected] a écrit :

Fermé #7675 https://github.com/publiclab/plots2/issues/7675 .

-
Vous recevez ceci parce que vous avez été mentionné.
Répondez directement à cet e-mail, consultez-le sur GitHub
https://github.com/publiclab/plots2/issues/7675#event-4183687540 , ou
Se désabonner
https://github.com/notifications/unsubscribe-auth/AABHNH2CH642A3GQR5SCQR3SY3JPFANCNFSM4LN2YLEA
.