Resolve "#774-observation-adjust-sectionComment-and-photo"
Closes #774 (closed)
Was wurde in dem MR umgesetzt?
- Im initialen Cleanup wurde in der
TransectInspectionMainData
-Komponente überflüssiger/ungenutzter Code (ungenutzte Parameter, classes, überflüssiges Markup etc.) entfernt, any-types entfernt und dieonIntersect
-Methode gekürzt, da hier eine Bedingung enthalten war, die niemals aufgerufen wurde (diese Methode ist jetzt noch immer nicht gut umgesetzt, es wird aber deutlicher, was sie tut) - Nach Absprache mit Lars: Ein nachträglicher Fix in
ObservationSummaryTable
. Es ist jetzt sichergestellt, dass leere Observation-Tables nur noch angezeigt werden, wenn die dazugehörige Section mindestens ein Bild oder/und ein Kommentar hat. - Die lokale
FileUpload
-Komponente wurde so angepasst, dass sie wie einModalButton
(gemäß Design-Vorgabe) angezeigt werden kann. (Es wurden Props auf eine Weise hinzugefügt, die sicherstellt, dass die Darstellung der Komponente an anderen Stellen nicht beeinflusst wird.) - Die
ModalButton
-Komponente wurde um einen default-slot erweitert, sodass in das Modal beliebiger Inhalt eingefügt werden kann (wie im Ticket-UseCase für die comment textArea mit v-model) - Beide Komponenten
FileUpload
undModalButton
sind jetzt disableable, sodass nur noch Anpassungen durch den Benutzer vorgenommen werden können, wenn die zugehörige Section 'accessible' ist - Die FileUploads in der gesamten Anwendung wurden mit einer
outline
versehen. (In Absprache mit Lars, um deutlicher zu machen, dass es sich bei dem camera-Icon um einen Button handelt). Hierzu wurde - für besseres Aussehen - derFileUpload
-Komponente einbuttonLabel
-Prop hinzugefügt. -
TransectInspectionMainData.spec.ts
wurde angepasst. Dafür gab es hauptsächlich drei Gründe:- Es wurden unnötigerweise die section propsData dupliziert und bei jedem Test zwei wrapper-Instanzen erstellt, obwohl jeweils nur eine genutzt wurde (gelöst durch eine wrapper factory function)
- Die Tests
Test onIntersect method
undTests handleClick method
testeten nichts: Es wurde manuell eine Funktion (direkt in der VM) aufgerufen und dann per mock verifiziert, dass sie aufgerufen wurde. Die beiden Tests wurden so überarbeitet, dass zumindest noch ein paar Seiteneffekte geprüft sind. - Hinzufügen der Unit-Tests für das aktuelle Ticket.
Edited by Michael Voigt