Skip to content

fix: Reindex inserted note links reliably on notebook import - EXO-88788 - #1738

Merged
hakermi merged 1 commit into
stable/7.2.x-exofrom
backport-88788
Jul 28, 2026
Merged

fix: Reindex inserted note links reliably on notebook import - EXO-88788#1738
hakermi merged 1 commit into
stable/7.2.x-exofrom
backport-88788

Conversation

@hakermi

@hakermi hakermi commented Jul 28, 2026

Copy link
Copy Markdown
Member

Inserted note links were rendered as "Content has been deleted" after import, mostly for notes whose names contain special characters (accents, dots, ...).

  • Export: apply processInsertedNotes() in the "export all" path so links are converted to reindex markers like the per-note export.
  • Import: resolve links against a name -> id map of the imported notes (by id), instead of a name based lookup that is unreliable for special characters. Links to notes not included in the export stay unresolved, as there is no target to reindex them to.

 (#1733)

Inserted note links were rendered as "Content has been deleted" after
import, mostly for notes whose names contain special characters
(accents, dots, ...).
- Export: apply processInsertedNotes() in the "export all" path so links
are converted to reindex markers like the per-note export.
- Import: resolve links against a name -> id map of the imported notes
(by id), instead of a name based lookup that is unreliable for special
characters. Links to notes not included in the export stay unresolved,
as there is no target to reindex them to.

fix: Reindex content-link anchor note ids on import - EXO-88788 (#1736)

"Insert note" links stored as anchors carry the note id in their
attributes (data-object="notes:<id>" and href=".../notes/<id>"), but the
import reindex only handled the legacy "content-link" element format.
Those anchors kept the source ids after import and rendered as "Content
has been deleted".

Build a source id -> imported id map during import and remap the ids
carried by content-link anchors. References to notes outside the import
are left unchanged.
@hakermi
hakermi requested review from Jihed525 and ahamdi July 28, 2026 14:39
@hakermi
hakermi enabled auto-merge (squash) July 28, 2026 14:39
@sonarqubecloud

Copy link
Copy Markdown

@hakermi
hakermi merged commit d7c329f into stable/7.2.x-exo Jul 28, 2026
11 checks passed
@hakermi
hakermi deleted the backport-88788 branch July 28, 2026 14:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants