Linked spreadsheet: Never overwrite existing data
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 35/100
- Tipo di issue
- Funzionalità
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Ferma
- Stack tecnologico
- javascript
- Ambito
- backend
Direzione di ricerca
Inizia esaminando PR #1758 e seguendo il percorso di scrittura nel foglio di calcolo che ha introdotto. Verifica in che modo le nuove risposte selezionano una riga e come vengono mantenuti gli aggiornamenti del foglio di calcolo, quindi definisci la copertura necessaria per preservare il contenuto esistente e impedire che gli invii simultanei perdano voci. Il lavoro è completato quando le nuove risposte vengono aggiunte senza sovrascrivere le righe o le modifiche precedenti.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
I previously posted this as feedback on the PR before it was applied (https://github.com/nextcloud/forms/pull/1758#issuecomment-1896763727), but since it was out of scope for the initial PR, I'm reposting this here.
Nextcloud:
- Nextcloud-Version: 28.0.1
- Forms-Version: 4.0.0 with PR #1758 applied
Is your feature request related to a problem? Please describe.
Usecase
The usecase I want to implement is a logbook for work on the electrical installation in a building. We currently have a spreadsheet in Nextcloud with a line for each change made, with some details about what, where, who, when, etc. To lower the bar for people to enter something in the logbook (and importantly, to allow unauthenticated users to add but not edit entries), I want to use a public form that adds entries to this logbook (and then put up QR-codes in the electrical cabinets so people can find the form).
Overwriting data
Currently every time a new answer is added, the field names and all previous answers are also written to the spreadsheet (again). This means that I cannot:
- Manually add a row in the spreadsheet - it will be overwritten by the next form response
- Edit any form data in the spreadsheet (fix typo's, add extra info) - it will be reverted on the next form response
- Add extra rows above the table with some instructions for users of the spreadsheet
- Insert extra columns in between the result columns or reorder columns (adding extra columns to the right of the results works already).
The first three are problematic for my usecase - the reason I want to export to a spreadsheet is so I have the logbook in an easy to manage and edit form, and also so people that are more familiar with the tech can just add their entries directly in the spreadsheet. Problem 4 I added for completeness, fixing it would be nice but also easy to work around by adding extra data to the right if needed.
Describe the solution you'd like
Just add only values for the new entry on the first empty row in the spreadsheet, never modify any existing content. This also means that the header row with column names should be written only when the sheet is completely empty. This should fix problem 1-3 above, but leave problem 4 unfixed.
As a side effect, this does mean that if the form is modified to e.g. add extra questions, the column headers become outdated (and also old and new answers might put the same answer in different columns). Also, it leaves problem 4 unfixed. This could maybe be fixed by doing a column lookup first: When writing a new answer, look in the first row for the corresponding column name and write the answer into that column. If the column name is not found in the first row, write the column name to the first empty cell in the first row, and then write the answer in the same column. This would fix problem 4, but unfix problem 3 (since the column headers must now remain in the first row), unless we can somehow tell the forms app in what row the column headers live (using a configurable row offset, or maybe some magic tag value in the spreadsheet, not sure if that can be done invisibly to the user)?
Personally, I suspect that the extra complexity of fixing problem 4 is not worth the trouble, so it should be fine to fix just 1-3 and leave 4 unfixed. The most important part is that then existing data is never overwritten, so any weirdness in column layout will be visible to the user and they can just fix up things manually or adjust their workflow.
Challenges
One potential challenge with my proposed solution is that atomicity of writing data to the spreadsheet becomes more relevant. If each for submission only writes one entry, and two submissions happen at the same time, there might be (depending on implementation - I have not checked) a risk of one submission overwriting the previous one, dropping one entry. With the current code, where all entries are written, this risk is minimal (even if the first submission overwrites the data of the second submission, then a later third submission will write the second response again). This should probably be solved with some kind of locking to make sure that no changes are made to the spreadsheet between loading it from storage and writing the updated file.
- Lingua principale
- JavaScript
- Stelle
- 383
- Fork
- 129
- Merge medio
- 12h 20m
- PR unite (30g)
- 70
Preparare l'ambiente
Avvia il container di sviluppo del progetto nel browser, con il tuo account GitHub.
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi 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 nextcloud/forms
-
[Bug]: Forms: `"CustomMessage":"The filename fallback must only contain ASCII characters."}` errorAperta1. to develop bug upstream
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
nextcloud/forms#3684 · 6 commenti · 1 reazione ·
I maintainer di solito rispondono entro 1 giorno
-
0. Needs triage bug
Difficoltà 3/5 1-2 giorni Idoneità per principianti 62/100
nextcloud/forms#3713 · 2 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
0. Needs triage enhancement
Difficoltà 3/5 1-2 giorni Idoneità per principianti 67/100
nextcloud/forms#3683 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
1. to develop bug pending fix regression
Difficoltà 3/5 1-2 giorni Idoneità per principianti 65/100
nextcloud/forms#3682 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Replace description fields with NcRichTextForse di nuovo libera @vishruthasgowda l’ha presa 46 giorni fa e non c’è nessuna pull request aperta. Aperta1. to develop enhancement feature: 📑 form creation good first issue javascript
nextcloud/forms#3587 · 2 commenti · 1 reazione · 1 assegnatario ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di nextcloud/forms
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
daisy/a11y-meta-viewer#18 ·
-
good first issue status: needs triaging type: bug version: 2.0
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
medusajs/medusa#17094 · 2 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
browser: chrome package: @carbon/react package: styles
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 92/100
carbon-design-system/carbon#23567 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
clerk/javascript#10033 ·
I maintainer di solito rispondono entro 1 giorno
-
bug client p1
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
vercel/eve#4173 · 2 commenti ·
I maintainer di solito rispondono entro 1 giorno