Pass this.state.data to then(), catch(), andThen() and andCatch() functions for more powerful side effects
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 35/100
- Tipo de issue
- Nueva funcionalidad
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- javascript, react
- Área
- frontend
Línea de trabajo
No se nombran rutas de archivos ni de tests. Empieza rastreando las implementaciones de then(), catch(), andThen() y andCatch(), además de las funciones de mapeo directo; determina cómo se ensamblan los argumentos de los handlers y cómo se puede exponer state.data. Se considera completado cuando esos handlers y mapeos pueden acceder a los datos actuales para combinar los resultados obtenidos y admitir actualizaciones optimistas, con el comportamiento relevante cubierto por tests.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Does it make any sense to have this.state.data available in those handlers? In my case, it's relevant for implementing an infinite scroller, where each fetch adds to the previous data and I need to somehow merge it.
Here's how that would look like:
const connections = {
widgets: '/api/widget/?limit=20',
loadMore: offset => ({
loadingMore: {
url: `/api/widget/?limit=20&offset=${offset}`,
andThen: (res, meta, data) => ({
widgets: { value: data.widgets.value.concat(res) },
}),
},
}),
};
My current workaround is to pass a getData() function to loadMore() from the component, which works, but isn't nice.
On another note, I would like to have that data even in direct mapping functions (loadMore()above), so I can do things like optimistic updates.
And, 👍 for this very nice library, thank you for writing it!
- Lenguaje dominante
- JavaScript
- Estrellas
- 3.4k
- Forks
- 137
- 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 heroku/react-refetch
-
Support for react@17Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 35/100
heroku/react-refetch#258 · 1 comentario ·
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 45/100
heroku/react-refetch#243 · 1 comentario ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
heroku/react-refetch#228 · 1 comentario · 3 reacciones ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 45/100
heroku/react-refetch#226 · 1 comentario ·
-
[idea] useRefetchAbierto
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
heroku/react-refetch#218 · 5 comentarios · 8 reacciones ·
Todos los issues de heroku/react-refetch
Issues similares
-
documentation
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
andromarces/agent-loops#886 ·
Los mantenedores suelen responder en 1 día
-
bug effort:low model:light plan planner:gpt-6-1-sol tests
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día
-
B: visual
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
FreeTubeApp/FreeTube#9929 ·
Los mantenedores suelen responder en 1 día
-
🐛 Bug: `--clear-screen` not very useful (at least not for dealing with large test/error output)Abiertostatus: in triage type: bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
mochajs/mocha#6428 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Fixed-position outer <svg> lands outside a clip after scroll; fixed HTML boxes are re-anchoredAbierto🔧 in development
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100