@powersync/attachments saveToQueue doesn't save additionalColumns
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- Mezza giornata
- Idoneità per principianti
- 50/100
- Tipo di issue
- Bug
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Ferma
- Stack tecnologico
- typescript
- Ambito
- databases
Direzione di ricerca
Start at the saveToQueue method on AbstractAttachmentQueue in the attachments package, and trace how the attachmentRecord is turned into an insert, which currently drops the additionalColumns declared in the attachment schema. Done when a record passed to saveToQueue with extra columns persists those values, checked by a test that saves a record with an additional column and reads it back. The issue has been open about 13 months with no response in the last year, so confirm the maintainers still want this fix before starting.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
The attachments package lets us define additionalColumns when we define our attachment schema.
When we save an attachment in the AbstractAttachmentQueue using the saveToQueue method, passing along the any additional columns as part of the attachmentRecord, it does not save these additional columns, I think this is a bit unexpected.
A work around I think will work is to take the record returned by saveToQueue and perform an update, however this is a bit inefficient, and I think it would be clearer to save any additional columns provided to saveToQueue
- Lingua principale
- TypeScript
- Stelle
- 729
- Fork
- 96
- Merge medio
- 1g 21h
- PR unite (30g)
- 27
Preparare l'ambiente
Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.
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 powersync-ja/powersync-js
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
powersync-ja/powersync-js#1159 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Differential watched queries ignore pure reorders (same rows, new ORDER BY) so stale order is servedAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 83/100
powersync-ja/powersync-js#1156 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 22/100
powersync-ja/powersync-js#1144 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 64/100
powersync-ja/powersync-js#1131 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 45/100
powersync-ja/powersync-js#1130 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di powersync-ja/powersync-js
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
kunchenguid/gh-axi#220 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
I maintainer di solito rispondono entro 1 giorno
-
enhancement
Difficoltà 2/5 1-3 ore Idoneità per principianti 79/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
I maintainer di solito rispondono entro 1 giorno
-
prerender: wait for env-runner worker readiness before crawlingForse già presa @sarankumar1325 l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
I maintainer di solito rispondono entro 1 giorno