Rebase our react-data-grid fork onto current upstream
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
- 12/100
- Tipo de issue
- Refactorización
- Claridad
- Bastante claro
- Estado de actividad
- Activo
- Stack tecnológico
- git, javascript, postgresql, react
- Área
- developer-experience, frontend
Línea de trabajo
Start with web/package.json to find the pinned react-data-grid commit, then compare the fork commit 8ddaa7be against adazzle/react-data-grid at 7.0.0-beta.60. Look at the grid usages named in the issue (onSelectedCellChange, rdg-cell-copied, textEditor) in the Query Tool, View/Edit Data and schema dialog code. Done means the fork is rebased, pgAdmin's pin is updated, and the Jest and Selenium feature tests pass.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
pgAdmin installs react-data-grid from our fork, pgadmin-org/react-data-grid, pinned by commit in web/package.json. The fork is upstream 7.0.0-beta.58 plus one commit of our own (8ddaa7be, cell range selection), and is now about 255 commits behind adazzle/react-data-grid, which is at 7.0.0-beta.60. We should rebase the fork onto current upstream and update pgAdmin to match.
This isn't a fast-forward, as upstream has made breaking changes that affect both the fork commit and pgAdmin:
onSelectedCellChange(used in pgAdmin) is nowonActivePositionChange, part of aselectedPositiontoactivePositionrename;selectCell, which the range selection code relies on, has changed too.- The
rdg-cell-copiedclass, which pgAdmin styles, has been removed. textEditorhas been renamed torenderTextEditor.- Rows are now focusable and the focus sink element has been removed, which changes the keyboard and focus handling the range selection code is built on.
- The build moved from rolldown to tsdown, and biome and the old ESLint setup were replaced.
So the work is: rebase the range selection commit (and the shift + click change from pgadmin-org/react-data-grid#9, if merged by then) onto upstream, rebuild lib/, bump the pin in pgAdmin, and update the Query Tool, View/Edit Data and schema dialog grids for the API changes. The Selenium feature tests need to pass as well as Jest, as grid problems in dialogs don't show up in the unit tests. In return we get upstream's fixes since beta.58 and React 19.3 support.
- Lenguaje dominante
- Python
- Estrellas
- 3.9k
- Forks
- 910
- Merge medio
- 1 d 6 h
- PR fusionados (30 d)
- 37
Preparar el entorno
- Incluye un Dockerfile o un 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 pgadmin-org/pgadmin4
-
Feature
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
pgadmin-org/pgadmin4#10426 · 3 comentarios · 1 reacción ·
Los mantenedores suelen responder en 1 día
-
Feature
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
pgadmin-org/pgadmin4#10272 · 5 comentarios ·
Los mantenedores suelen responder en 1 día
-
Housekeeping
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
pgadmin-org/pgadmin4#10157 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dark ModeAbiertoFeature
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
pgadmin-org/pgadmin4#10640 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Feature
Dificultad 3/5 1-2 días Aptitud para principiantes 45/100
pgadmin-org/pgadmin4#10615 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
Todos los issues de pgadmin-org/pgadmin4
Issues similares
-
comp/tools P2 type/bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
NousResearch/hermes-agent#142908 · 1 comentario ·
-
[oblt-aw][autodoc] Fix malformed headings in new Buildkite docs pagesPosiblemente ocupada @github-actions la tomó hoy. Abiertodocumentation
Dificultad 1/5 Menos de una hora Aptitud para principiantes 83/100
Los mantenedores suelen responder en 1 día
-
[Bug]: fileUploadMatrix.test.ts URL case does a real DNS lookup and fails offlinePosiblemente ocupada @r2jaff la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
ComposioHQ/composio#5054 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
pytorch/tensordict#3286 ·
Los mantenedores suelen responder en 1 día