Thread architecture still depends on replyToEventId/rootEventId
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Aptitud para principiantes
- 25/100
Línea de trabajo
Start by reading NoteRelationModel and tracing the thread-loading, reply-discovery, parent/root resolution, and reply-visibility paths that currently use replyToEventId or rootEventId. The issue does not identify files or tests, so first locate those entry points and inspect the regressions found during testing. Done would require a defined migration that preserves those behaviors, but the scope and acceptance criteria are not specified.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Summary
While investigating a migration toward using NoteRelationModel as the single source of truth for thread relationships, I found that the current thread architecture still depends on replyToEventId and rootEventId in several places.
Findings
- NoteRelationModel currently stores untyped relation edges.
- Root edges are excluded.
- The relation graph does not contain enough information to fully reconstruct thread structure.
- Removing or bypassing replyToEventId/rootEventId causes regressions in:
- Thread loading
- Reply discovery
- Parent/root note resolution
- Reply visibility
Notes
During testing, relying only on NoteRelationModel caused thread-related regressions, suggesting that a complete migration would likely require broader changes to the relation layer.
This does not affect the fix in PR #102, but was discovered while investigating the underlying architecture.
- Lenguaje dominante
- Dart
- Estrellas
- 2
- Forks
- 2
- Merge medio
- 19 h 21 min
- PR fusionados (30 d)
- 37
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de basictech01/uniun
-
bug good first issue P2-Medium
Dificultad 2/5 1-3 horas Aptitud para principiantes 64/100
basictech01/uniun#305 ·
Los mantenedores suelen responder en 1 día
-
bug good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
basictech01/uniun#300 ·
Los mantenedores suelen responder en 1 día
-
enhancement
Dificultad 4/5 1-2 días Aptitud para principiantes 42/100
basictech01/uniun#306 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
bug good first issue P2-Medium
Dificultad 3/5 1-2 días Aptitud para principiantes 52/100
basictech01/uniun#304 ·
Los mantenedores suelen responder en 1 día
-
bug good first issue
Dificultad 3/5 1-2 días Aptitud para principiantes 52/100
basictech01/uniun#301 ·
Los mantenedores suelen responder en 1 día
Todos los issues de basictech01/uniun
Issues similares
-
Standalone dash flashes as its own RSVP step (e.g. Russian dialogue)Posiblemente ocupada @bezig01 la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
daniel-pimenta-cc/ledor#53 ·
-
Iteration CounterAbiertoenhancement firebase
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
brysonddelozier/scribbles#24 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
daohoangson/flutter_widget_from_html#1657 ·
Los mantenedores suelen responder en 3 días
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
RevenueCat/purchases-flutter#1973 ·
Los mantenedores suelen responder en 1 día
-
Needs Attention platform: android type: infrastructure umbrella
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
firebase/flutterfire#18832 ·
Los mantenedores suelen responder en 1 día