Skip to content
This repository was archived by the owner on Apr 19, 2023. It is now read-only.
This repository was archived by the owner on Apr 19, 2023. It is now read-only.

Refaktorere API #66

@sirhaug

Description

@sirhaug

En seksjon fra api/README omtalte refaktoreringer. Disse er nå fjernet fra README og lagt til her som et eget issue.


Potensiell refakturering

  • getVurderingerByPlaceId(String placeId, Pagable pagable) i VurderingServiceImpl:
    Denne metoden er nok ikke gjort på den peneste og mest effektive måten slik som den er nå, men den funker.
    Dersom det er tid, gjerne se litt på denne og prøv å skriv litt penere kode. Kort fortalt henter den en liste med vurderinger (paginering), og deler denne lista opp i flere lister med kriteriet om at bruker og dato skal være den samme for alle elementer i hver liste. Videre blir
    en JSON formattert, slik at output blir følgende:
[
{
    "dato": "2018-07-18",
    "registrator": 2, (eller hele objectet)
    "vurderinger": {
       "teleslynge": {"kommentar": "Noe", "rangering": true},
       "lydforhold": {"kommentar": "Noe annet", "rangering": false},
       ...
     }
},
{
    "dato": "2018-07-17",
    "registrator": 3,
    ...
},
...
]
  • Finne ut om det er mulig med dynamiske queries i repoene. Jeg fant ingen god måte å gjøre dette på så nå er det fire ulike spørringer i VurderingRepo (findXXXByPlaceId). Fint om de kan bli én spørring som er dynamisk (tror ikke dette er mulig med @Query da...)

  • Gå gjennom pom.xml filen og ta vekk det som ikke trenger å være der. Jeg tror det er en del der nå som ikke er nødvendig for applikasjonen slik den er nå.

  • En del av testene ble slettet etter Bruker ble endret. Fint om noen nye tester kan skrives her.

Metadata

Metadata

Assignees

No one assigned

    Labels

    Task - Low PriorityA nice to have task or feature that can be done if there is time

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions