Hacktoberfest 2026: los issues que los mantenedores marcaron para octubre, abiertos y aptos para principiantes. Explorar issues de Hacktoberfest

Desktop: "Copy page URL" throws "Missing graph-id" and copies nothing

Cerrado Apto para principiantes
#1,362 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
87/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
electron
Área
desktop, frontend

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

  1. In the desktop app, create a DB graph without sync (search "Add a DB graph", leave sync off, Submit).
  2. Open any page.
  3. 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

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de logseq/db-test

Todos los issues de logseq/db-test

Issues similares

Más issues de Desktop Dev

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.