`Surface.View` can not access context
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 48/100
Línea de trabajo
Start with lib/surface/compiler/eex_engine.ex around lines 1095-1100, identified as the culprit, and reproduce the issue using a Surface initialized with --layouts and context in live.sface or a used component. Done means the context value is read correctly instead of returning nil; the issue does not name a test to run.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Describe the bug
When using the context in a template defined inside a Surface.View, both (old) <Context get={...}> and (new) prop name, :type, from_context: ... in a used component yields nil.
How to reproduce it
- Use surface initialized with
--layouts - Use context in
live.sfaceor a component used bylive.sface - Resulting value will always be
nil
The behavior you expected
The value is read correctly.
Your Environment
Surface: v0.8.1
LiveView: v0.17.11
Elixir: v1.14.0
Culprit
I tracked down the issue to the following line: https://github.com/surface-ui/surface/blob/b9675f9662f9d684ad4baeebdcc358ffab5264b2/lib/surface/compiler/eex_engine.ex#L1095-L1100
Since this looks like the context is intentionally set to an empty map, I decided not to attempt a PR for now.
Workaround
I built the component inclusion manually, for now, to work around the constraint / issue:
- Lenguaje dominante
- Elixir
- Estrellas
- 2.1k
- Forks
- 146
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
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 surface-ui/surface
-
Dificultad 1/5 1-3 horas Aptitud para principiantes 78/100
surface-ui/surface#634 · 1 reacción ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
surface-ui/surface#770 ·
-
Website is downAbierto
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
surface-ui/surface#768 · 4 comentarios ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
surface-ui/surface#762 · 2 comentarios · 1 reacción ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
surface-ui/surface#759 · 1 reacción ·
Todos los issues de surface-ui/surface
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
electric-sql/electric#4830 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
Los mantenedores suelen responder en 1 día
-
Gradle release-date fallback accepts `Last-Modified` from non-2xx responses, so `cooldown` lets brand-new plugin versions throughPosiblemente ocupada @v-robaiken la tomó hace 1 día. AbiertoL: java:gradle T: bug 🐞
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
dependabot/dependabot-core#16547 · 1 reacción · 2 asignados ·
Los mantenedores suelen responder en 1 día
-
github_actions
Dificultad 1/5 Menos de una hora Aptitud para principiantes 94/100
Los mantenedores suelen responder en 1 día