Skip to content

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 die onIntersect-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 ein ModalButton (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 und ModalButton 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 - der FileUpload-Komponente ein buttonLabel-Prop hinzugefügt.
  • TransectInspectionMainData.spec.ts wurde angepasst. Dafür gab es hauptsächlich drei Gründe:
    1. 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)
    2. Die Tests Test onIntersect method und Tests 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.
    3. Hinzufügen der Unit-Tests für das aktuelle Ticket.
Edited by Michael Voigt

Merge request reports

Loading