ngu / geonorge-nedlastingapi

An implementation in Java of the Geonorge Download API (https://nedlasting.geonorge.no/Help)
MIT License
4 stars 2 forks source link

order receipt file with more file details #54

Closed halset closed 5 years ago

halset commented 5 years ago

Dette er et forsøk på å løse #52.

halset commented 5 years ago

Kan dere sjekke om dette ble greit, @MalteStr og @bgrotan ? Jeg er litt bekymret for å endre innholdet av format-egenskapen til en fil i internal/dataset/{metadataUuid}, men det er vel heller v2/codelists/format/{metadataUuid} som blir brukt for å hente formater og der er det ordentlige format-objekter med både navn og versjon hver for seg.

bgrotan commented 5 years ago

internal/ blir kun brukt ved publisering av filer for datasett, og aldri noe geonorge-klienten ser noe til. P.t. er det da kun NGU og NIBIO som har klienter som evt bruker internal-api.

halset commented 5 years ago

Denne er ikke helt bra nå. @bgrotan Er det greit om vi endrer slik at file.format ikke blir ala "GML 3.2.1", men at vi gjør det om til et format-objekt som har name og version? Da stemmer det mer med resten av APIet og vi slipper feil ved bruk av PUT internal/dataset/{metadataUuid} som prøver å ta format og legge det inn som formatName i databasen.

halset commented 5 years ago

Eller splitte format opp i formatName og formatVersion i internal-APIet for å matche litt med slik area og projection gjøres. Eller gjøre alle tre om til objekter for å matche det ikke-internal APIet..

halset commented 5 years ago

Nå har jeg oppdatert denne slik at internal-apiet har fått en formatVersion i tillegg til den gamle format. Ser det greit ut, @bgrotan og @MalteStr ?