Desktop: "Copy page URL" throws "Missing graph-id" and copies nothing
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 87/100
Línea de trabajo
Comienza en src/main/frontend/components/page_menu.cljs y src/main/frontend/handler/page.cljs; después, sigue current-graph-id en src/main/frontend/handler/graph.cljs y la construcción de la URL en src/main/frontend/util/url.cljs. Reproduce primero los pasos de los grafos de DB sin sincronización. Se considera terminado cuando Copy page URL ya no falla cuando falta el ID del grafo y copia una URL válida o informa de que no hay ninguna URL disponible.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Steps
- In the desktop app, create a DB graph without sync (search "Add a DB graph", leave sync off, Submit).
- Open any page.
- Right-click the page title (or open the page's "..." menu) and click "Copy page URL".
Expected
A page URL is copied, or a message says the page has no URL to copy.
Actual
Nothing is copied and nothing is shown. The click handler throws Error: Missing graph-id, reported to the window as an uncaught exception.
Cause
The menu item (src/main/frontend/components/page_menu.cljs, shown on Electron and on native mobile) calls page-handler/copy-page-url (src/main/frontend/handler/page.cljs), which builds the URL with url-util/get-logseq-web-page-url config/app-website (graph-handler/current-graph-id) page-uuid. current-graph-id (src/main/frontend/handler/graph.cljs) returns a graph id only when the tab remembers one for the current graph, and get-logseq-web-page-url (src/main/frontend/util/url.cljs) throws (js/Error. "Missing graph-id") through required-url-part! when it is nil. copy-page-url checks only that the page uuid exists. The check came with ec0c01fd9 ("use graph identity in URLs", #12693).
Found by a monkey test (gremlins.js with trusted Playwright input) of the desktop app built from master 16c4ed1a0, seed 301, action 759 (a right-click on the page title, then "Copy page URL"). The steps above, run by hand on that build (a new graph, a new page, right-click on its title, "Copy page URL"), reproduced it 1 of 1 times.
- Lenguaje dominante
- Sin datos de lenguaje
- Estrellas
- 28
- Forks
- 2
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
Aún no hemos revisado los archivos de configuración de este proyecto. 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 logseq/db-test
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 86/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 86/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
-
Pressing Delete after double-clicking a journal page's title row throws in keydown-delete-handlerAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Todos los issues de logseq/db-test
Issues similares
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
Los mantenedores suelen responder en 1 día
-
component: split-view platform: windows
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
zen-browser/desktop#15616 · 1 reacción ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
Los mantenedores suelen responder en 1 día
-
bug 🐞
Dificultad 2/5 1-3 horas Aptitud para principiantes 64/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
Los mantenedores suelen responder en 1 día