publish: upload local files referenced by links, not just images
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 48/100
Direzione di ricerca
Inizia con la registrazione degli allegati forward in images.go:78, poi leggi attachname.go e il clamp condiviso withinRoot. Confronta la gestione dei link con renderImage e controlla il mapping in docs/confluence/links-and-anchors.md. Il lavoro è completato quando le destinazioni locali dei link diverse da .md che soddisfano i requisiti vengono caricate e riscritte correttamente, mentre i link .md mantengono il comportamento esistente degli URL delle pagine e il rendering inverso di ri:attachment viene aggiornato tramite sourceFor.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
read/export now pass an attachment link through as raw storage rather than converting it, because a markdown link to the exported file would break on the next publish. From the mapping table in links-and-anchors.md:
| storage | markdown | why |
|---|---|---|
ri:attachment |
passthrough | only images are uploaded, so [x](Deck.ppt) would publish as a dead relative href |
So this exports:
Grab <ac:link><ri:attachment ri:filename="Deck.ppt" /><ac:plain-text-link-body>the deck</ac:plain-text-link-body></ac:link>.
where Grab [the deck](Deck.ppt). would read better, preview locally, and reference the file export already downloads (it scans raw storage for ri:filename, not just ac:image, precisely so a link target is included).
What blocks it is the forward path, not the inverse one. Only images.go:78 registers an Attachment, so a local file named by a link destination is never uploaded, and republishing [the deck](Deck.ppt) emits <a href="Deck.ppt"> — a relative href that resolves to nothing on Confluence, with no warning and exit 0. That is the silent failure mode docs/confluence/links-and-anchors.md already calls out for unresolved doc links, and it is worse here because the link was working before the round trip.
The work is therefore in the forward direction first: treat a link destination that names a local non-.md file the way an image src is treated — resolve it page-relative, clamp it to the documentation root, register it as an attachment, and rewrite the href to the attachment. attachname.go's source-path↔attachment-name codec and the withinRoot clamp both already exist and are shared with images; what is missing is a link-side caller. Once a link target uploads, flip ri:attachment in the inverse mapping from passthrough to [body](path) via sourceFor, exactly as renderImage does.
Two things to decide when picking this up. Whether a link to a local file should upload it at all, or only when some marker says so — an unqualified rule means any [x](notes.txt) in a repo starts publishing that file as an attachment, which may be more than anyone asked for. And what .md siblings do, which must keep resolving to page URLs via the existing doc-link path rather than becoming attachments.
Split out of #88, where the passthrough decision was made.
- Lingua principale
- Go
- Stelle
- 2
- Fork
- 0
- Merge medio
- 2h 50m
- PR unite (30g)
- 52
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di mozilla/markfluence
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 55/100
mozilla/markfluence#186 ·
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
mozilla/markfluence#185 ·
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
mozilla/markfluence#184 ·
-
bug
Difficoltà 3/5 1-2 giorni Idoneità per principianti 55/100
mozilla/markfluence#181 · 1 commento ·
-
bug
Difficoltà 5/5 Più di una settimana Idoneità per principianti 48/100
mozilla/markfluence#163 ·
Tutte le issue di mozilla/markfluence
Issue simili
-
nix: vendorHash is outdated Aperta
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 90/100
-
Bob Shell support Apertaenhancement
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
santhosh-tekuri/jsonschema#276 ·