Skip to content
Snippets Groups Projects

VIPSUTV-1047 Editing observations belonging to timeseries

Merged Lene Wasskog requested to merge feature/vipsutv-1047-timeseries into develop
1 unresolved thread

Culture, pest and location should not be editable for observations belonging to time series. I remove the original input fields for these values for all such observations, and ensure that they are still submitted with the form via hidden input fields.

Merge request reports

Approved by

Merged by Lene WasskogLene Wasskog 3 weeks ago (Apr 30, 2025 7:08am UTC)

Merge details

  • Changes merged into develop with 9c7daf23.
  • Did not delete the source branch.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Har også lagt til to andre (forslag til) endringer:

    • Fra- og til-dato vises nå alltid i filterskjemaet for observasjonslista. Det fikk meg til å innse at man benytter en veldig romslig periode dersom bruker legger inn bare én dato (+/- 100 år). Det tar lang tid å laste sida hvis man kun setter sluttdato, så dette bør vi kanskje endre?
    • Observation.description er ikke obligatorisk i appen, vil tro det er greit å bruke samme regel her?

    Har ikke lagt inn tekster i oversettingsfilene ennå, ville høre hva du tenker om ordlyden. Info om tidsserier skriver foreløpig rett ut, si gjerne ifra hvis du tenker jeg bør implementere det annerledes (gjerne med et eksempel et annet sted i applikasjonen som ligner)

  • assigned to @lewa

  • Lene Wasskog requested review from @treinar

    requested review from @treinar

    • Får kompileringsfeil (se vedlegg) Screenshot_from_2025-04-10_14-16-10

    • Buhu! mvn clean install funker hos meg med mvn/java:

      $ mvn -v
      Apache Maven 3.8.8 (4c87b05d9aedce574290d1acc98575ed5eb6cd39)
      Maven home: /usr/local/apache-maven
      Java version: 17.0.12, vendor: Eclipse Adoptium, runtime: /Users/lenewasskog/.sdkman/candidates/java/17.0.12-tem
      Default locale: nb_NO, platform encoding: UTF-8
      OS name: "mac os x", version: "15.3.2", arch: "aarch64", family: "mac"

      Hmmmm. Kan forske litt mer.

    • Apache Maven 3.8.7 Maven home: /usr/share/maven Java version: 21.0.6, vendor: Ubuntu, runtime: /usr/lib/jvm/java-21-openjdk-amd64 Default locale: en_US, platform encoding: UTF-8 OS name: "linux", version: "6.11.0-21-generic", arch: "amd64", family: "unix"

    • Please register or sign in to reply
  • I alle! Nå funka det!!

  • Det ser fint ut!

  • Tor-Einar Skog approved this merge request

    approved this merge request

  • Vi kan diskutere ordlyd ved nærmeste anledning!

  • Lene Wasskog added 1 commit

    added 1 commit

    • 965071a7 - feat: Add texts to i18n files

    Compare with previous version

  • Lene Wasskog merged manually

    merged manually

Please register or sign in to reply
Loading