A signer custom message can't be removed once it has been saved
I maintainer di solito rispondono entro 1 giorno
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 20/100
- Tipo di issue
- Bug
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Ferma
- Stack tecnologico
- php
- Ambito
- full-stack
Direzione di ricerca
The issue identifies src/components/Request/IdentifySigner.vue, RequestSignatureService, and SignRequestService::populateSignRequest() as the path where clearing the message is lost. Start by tracing how an empty description moves from the component through the store and backend. Done means turning off or emptying a saved custom message removes it from the signer and subsequent notifications; linked pull request #9126 is already open.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Describe the bug
A signer custom message can't be removed once it has been saved
To reproduce
- Request a signature and add a signer with "Add custom message" turned on (e.g. "Please review the annex"). Save.
- Edit the same signer, turn "Add custom message" off, and save.
- Reopen the signer, or trigger a notification/reminder:
the old message is still stored;
it is still included in the signer's email (and SMS/Telegram via Two-Factor Gateway).
Expected behavior
Turning off the custom message (or emptying it) removes it, and later notifications use the standard text only.
Screenshots
No response
Operational System
NixOS
Browser Type and Version
No response
Libresign Version
15.0.6
Nextcloud Server Version
35.0.0
Logs from Nextcloud Server
Logs from Developers tools
No response
Additional context
Cause: clearing the message never reaches the backend as "clear".
src/components/Request/IdentifySigner.vuesendsdescription.value.trim() || undefined, and then the store drops non-string valuesRequestSignatureServiceturns the missing key into'';SignRequestService::populateSignRequest()only writes when!empty($description), so '' keeps the old value.
- Lingua principale
- PHP
- Stelle
- 830
- Fork
- 161
- Merge medio
- 17h 1m
- PR unite (30g)
- 477
Preparare l'ambiente
Avvia il container di sviluppo del progetto nel browser, con il tuo account GitHub.
- Nessun Dockerfile né file Docker Compose
- Ha un 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 LibreSign/libresign
-
frontend good first issue help wanted javascript technical debt
Difficoltà 2/5 1-3 ore Idoneità per principianti 86/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 83/100
LibreSign/libresign#9007 · 1 reazione ·
I maintainer di solito rispondono entro 1 giorno
-
backend enhancement php
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
I maintainer di solito rispondono entro 1 giorno
-
good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
LibreSign/libresign#8284 · 5 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
enhancement good first issue php
Difficoltà 3/5 1-2 giorni Idoneità per principianti 72/100
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di LibreSign/libresign
Issue simili
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
MagnaCapax/PMSS#1142 · 1 commento ·
I maintainer di solito rispondono entro 3 giorni
-
bug 🐛
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
netalertx/NetAlertX#1856 · 1 reazione ·
I maintainer di solito rispondono entro 1 giorno
-
Bug Status: Needs Triage
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
I maintainer di solito rispondono entro 1 giorno
-
feature good first issue html mcv php
Difficoltà 2/5 1-3 ore Idoneità per principianti 67/100
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 85/100
lorisleiva/laravel-docker#166 ·