Apply suggestions from code review for better texts

Co-authored-by: shamoon <4887959+shamoon@users.noreply.github.com>
This commit is contained in:
Dominik Bruhn 2024-06-08 08:26:37 +02:00 committed by Dominik Bruhn
parent 6a63e77135
commit 0ad2cd1285
3 changed files with 6 additions and 8 deletions

View File

@ -2971,17 +2971,17 @@
</context-group> </context-group>
</trans-unit> </trans-unit>
<trans-unit id="2519605321077387027" datatype="html"> <trans-unit id="2519605321077387027" datatype="html">
<source> Delete original documents after merge </source> <source> Delete original documents after successful merge </source>
<context-group purpose="location"> <context-group purpose="location">
<context context-type="sourcefile">src/app/components/common/confirm-dialog/merge-confirm-dialog/merge-confirm-dialog.component.html</context> <context context-type="sourcefile">src/app/components/common/confirm-dialog/merge-confirm-dialog/merge-confirm-dialog.component.html</context>
<context context-type="linenumber">32,34</context> <context context-type="linenumber">32</context>
</context-group> </context-group>
</trans-unit> </trans-unit>
<trans-unit id="5138283234724909648" datatype="html"> <trans-unit id="5138283234724909648" datatype="html">
<source>Note that only PDFs will be included.</source> <source>Note that only PDFs will be included.</source>
<context-group purpose="location"> <context-group purpose="location">
<context context-type="sourcefile">src/app/components/common/confirm-dialog/merge-confirm-dialog/merge-confirm-dialog.component.html</context> <context context-type="sourcefile">src/app/components/common/confirm-dialog/merge-confirm-dialog/merge-confirm-dialog.component.html</context>
<context context-type="linenumber">36</context> <context context-type="linenumber">34</context>
</context-group> </context-group>
</trans-unit> </trans-unit>
<trans-unit id="8157388568390631653" datatype="html"> <trans-unit id="8157388568390631653" datatype="html">

View File

@ -28,10 +28,8 @@
</select> </select>
</div> </div>
<div class="form-check form-switch mt-4"> <div class="form-check form-switch mt-4">
<input class="form-check-input" type="checkbox" value="" id="deleteOriginals" [(ngModel)]="deleteOriginals" [disabled]="!userOwnsAllDocuments"> <input class="form-check-input" type="checkbox" role="switch" id="deleteOriginalsSwitch" [(ngModel)]="deleteOriginals" [disabled]="!userOwnsAllDocuments">
<label class="form-check-label" for="deleteOrginals" i18n> <label class="form-check-label" for="deleteOriginalsSwitch" i18n>Delete original documents after successful merge</label>
Delete original documents after merge
</label>
</div> </div>
<p class="small text-muted fst-italic mt-4" i18n>Note that only PDFs will be included.</p> <p class="small text-muted fst-italic mt-4" i18n>Note that only PDFs will be included.</p>
</div> </div>

View File

@ -293,7 +293,7 @@ def merge(
) )
if delete_originals: if delete_originals:
logger.info("Removing original documents after consumption of merged document") logger.info("Queueing removal of original documents after consumption of merged document")
chain(consume_task, delete_documents.si(affected_docs)).delay() chain(consume_task, delete_documents.si(affected_docs)).delay()
else: else:
consume_task.delay() consume_task.delay()