Plots2: Los enlaces externos que contienen un símbolo `` @ '' actualmente se analizan como nombres de usuario de laboratorio público.

Creado en 17 mar. 2020  ·  24Comentarios  ·  Fuente: publiclab/plots2

Guión:
Al insertar un enlace externo en una publicación de laboratorio pública
Y el enlace contiene un @
Luego, debe analizarse como parte de la URL y no como un enlace a un nombre de usuario de laboratorio público.

Captura de pantalla de @joyofsoy :
Screen Shot 2020-03-17 at 3 47 42 PM

Aquí hay un enlace de ejemplo para probar de la captura de pantalla anterior: https://medium.com/@erinargyle/working-during-covid-19-how-to-be-good-at-video-meetings-57f49fdb8dcd

Una solución alternativa actual es utilizar un sitio como bit.ly para ofuscar los enlaces, pero lo ideal sería solucionarlo. ¡Gracias @joyofsoy por informarnos! : tada:

Ruby help wanted ws

Comentario más útil

@emilyashley ¿puedo trabajar en esto?

Todos 24 comentarios

Notas de Jeff:

@jywarren : Creo que debemos cambiar el comparador de patrones @______ para no encontrar el patrón dentro de los enlaces. El comparador de patrones está aquí: https://github.com/publiclab/plots2/blob/acd5a4910632c8cc80f9fadb54d8cc64c9600519/config/initializers/constants.rb#L2

Aquí hay una demostración del error de coincidencia de patrones basada en los enlaces originales de Joe; debe modificarse para NO reconocer los nombres de usuario dentro de las URL: https://rubular.com/r/HsxxuTG8zG0zSL
Screen Shot 2020-03-17 at 3 57 45 PM

@emilyashley ¿puedo trabajar en esto?

@Tlazypanda eso sería genial, ¡¡¡gracias !!!

Hola @Tlazypanda , ¿tuviste la oportunidad de probar algún cambio usando este editor de expresiones? https://rubular.com/r/HsxxuTG8zG0zSL Parece un pequeño juego, tratar de ajustar cómo se reconocen los ' @nombres ' en nuestro sitio, pero no cuando hay un '/' en cada lado.

Hola @ebarry , lo siento mucho, me olvidé por completo de este problema mientras trabajaba en otros problemas ... ¿está bien si lo tomo después de algunos días desde que continúan mis exámenes universitarios?

¡¡¡Por supuesto!!! ¡¡¡Buena suerte en los exámenes !!!

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

Parece que funciona correctamente. ¿Puedes confirmar si esta expresión regular es correcta o no @ebarry ?

Screenshot from 2020-12-17 23-24-02

@jywarren ¿Puedes confirmar esto?

Desearía poder confirmar de cualquier manera, pero no estoy familiarizado con las expresiones regulares, lo siento por mi respuesta retrasada y ¡GRACIAS por trabajar en esto!

¡Funciona perfectamente! https://rubular.com/r/jfKfgjdqi00qy4

image

Esto ahora se puede insertar en constants.rb ; ¡también sería un buen problema solo para principiantes!

Esto ha sido marcado como un buen candidato para convertirse en un tema exclusivo para este , lo que significa que es simple, autónomo y, con un formato adicional, podría ser un excelente punto de entrada para un nuevo colaborador. Si está lo suficientemente familiarizado con este código, considere reformatearlo o volver a


¡Gracias por revisar este @jywarren! Gracias Gaurav Ahlawat por hacer

¡esto!

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

El martes 5 de enero de 2021 a las 4:55 p.m. Jeffrey Warren [email protected]
escribió:

¡Funciona perfectamente! https://rubular.com/r/jfKfgjdqi00qy4

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

Esto ahora se puede insertar en constants.rb - sería una buena
¡Edición exclusiva para principiantes también!

Esto ha sido marcado como un buen candidato para convertirse en un usuario exclusivo para principiantes.
emitir http://www.firsttimersonly.com como estos
https://code.publiclab.org/#r=all , lo que significa que es simple,
autónomo, y con un formato adicional, podría ser una gran entrada
punto para un nuevo colaborador. Si está lo suficientemente familiarizado con este código,
considere reformatearlo o volver a publicarlo como un problema exclusivo para principiantes
https://publiclab.org/notes/warren/10-31-2016/create-a-welcoming-first-timers-only-issue-to-invite-new-software-contributors ,
y luego ping @ publiclab / revisores
https://github.com/orgs/publiclab/teams/reviewers para etiquetarlo.

O, si no es la primera vez, ¡intente resolverlo usted mismo!

-
Recibes esto porque te mencionaron.
Responda a este correo electrónico directamente, véalo en GitHub
https://github.com/publiclab/plots2/issues/7675#issuecomment-754925581 ,
o darse de baja
https://github.com/notifications/unsubscribe-auth/AABHNH47FTWY45FYH34CQCDSYODDNANCNFSM4LN2YLEA
.

@ebarry ¿Podría convertirlo en FTO para que otros puedan resolverlo? ¡Gracias!

gran trabajo @ gauravahlawat81 Puedo hacer eso por usted ... o tal vez le gustaría intentar convertir esto usted mismo? ... Estoy feliz de ayudar donde está atascado ... ¿qué piensas?

Sí, claro, me gustaría convertirlo yo mismo. Cómo puedo hacer eso ?

¿Debo crear un nuevo problema o editar este?

Genial: tada :, sí, crear uno nuevo sería más sencillo, cuando haga clic en el nuevo número aquí https://github.com/publiclab/plots2/issues/new/choose seleccione First timer Only Issue y se rellenará una plantilla ... después de esta línea donde tiene **Update** the file [$FILENAME]($BRANCH_URL) in the `$REPO` repository (press the little pen Icon) and edit the line as shown below. indique el archivo que necesita cambiarse y los cambios necesarios ... Ponle un título y eso debería ser todo

Muy bien, gracias

@cesswairimu ¿Cómo puedo generar la diferencia?

No estoy seguro de si hay otra forma de hacerlo ... la forma en que lo hago es usando el bot primer temporizador
Entonces, con este enfoque, lo que debe hacer es navegar hasta el archivo que desea editar en este caso, creo que es https://github.com/publiclab/plots2/blob/main/config/initializers/constants.rb y haga clic en el botón de edición (icono de lápiz), haga los cambios necesarios, luego el cuadro de texto debajo de este Create a new branch for this commit and start a pull request edítelo y asegúrese de que comience con first-timers- luego haga clic en confirmar cambios ...

después de eso, debería ver su problema creado aquí https://github.com/publiclab/plots2/issues/ con el autor como first-timer-bot
más documentación sobre esto está aquí si esto no tiene sentido

Creé el problema, aunque tuve que usar un enfoque diferente. Por favor, vea si todo está bien allí, y si es necesario cambiar algo, hágamelo saber.

El problema es perfecto ... gracias de nuevo

Corregido en https://github.com/publiclab/plots2/pull/8972 Gracias a todos

¡Gracias a todos!

Flujo de trabajo muy ordenado

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

El viernes 8 de enero de 2021 a las 4:59 a. M. Cess [email protected] escribió:

Cerrado # 7675 https://github.com/publiclab/plots2/issues/7675 .

-
Recibes esto porque te mencionaron.
Responda a este correo electrónico directamente, véalo en GitHub
https://github.com/publiclab/plots2/issues/7675#event-4183687540 , o
darse de baja
https://github.com/notifications/unsubscribe-auth/AABHNH2CH642A3GQR5SCQR3SY3JPFANCNFSM4LN2YLEA
.

¿Fue útil esta página
0 / 5 - 0 calificaciones