Inbox: 6 and 8-digit issue refs render as colour swatches instead of links

Cerrado Apto para principiantes
  • ¿Algo no está bien?
#1,932 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

Ya se ha fusionado un pull request relacionado.

  • #2040 de @backnotprop — fusionado

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
72/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
typescript
Área
frontend

Línea de trabajo

El error está en InlineMarkdown en packages/ui/components/InlineMarkdown.tsx, donde la rama de muestras de color hexadecimal se ejecuta antes que la rama de referencias a issues #(\d+). Empieza leyendo las dos ramas y su orden, luego renderiza #186379 y #123 para confirmar la salida actual. Está terminado cuando las referencias solo numéricas de 6 y 8 dígitos se convierten en enlaces, y siguen funcionando las muestras hexadecimales reales como #a1b2c3 y el comportamiento actual de la referencia #123.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

Since #1840, issue refs in Inbox messages link to the project's repo. Refs with 6 or 8 digits don't, though: #186379 shows a colour swatch and a code pill, because it's also a valid hex colour. That's every current PR in big repos like home-assistant/core.

In InlineMarkdown, the hex swatch branch runs before the issue-ref branch. 3 and 4-digit hex needs an a-f letter so #123 stays a ref, but the 6 and 8-digit forms have no such check, so all-digit values never reach the ref branch.

Suggested fix: use the same letter rule for the 6 and 8-digit forms, or let an all-digit #n be an issue ref whenever a repo is known.

Seen on plannotator 0.28.20, Linux, OpenCode.

Image
Lenguaje dominante
TypeScript
Estrellas
8.7k
Forks
649
Merge medio
8 h 51 min
PR fusionados (30 d)
633

Preparar el entorno

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de backnotprop/plannotator

Todos los issues de backnotprop/plannotator

Issues similares

Más issues de TypeScript

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.