Skip to content

Conversation

@jorgkv
Copy link
Contributor

@jorgkv jorgkv commented Nov 13, 2025

Siste versjon av modellene finnes i denne PR'en i API'et:
https://dev.azure.com/NVE-devops/Varsom%20Regobs/_git/regobs/pullrequest/15371

For å jobbe mot lokalt api kan man legge inn urlen til apiet i openapitools.json og generere ny kode med npm run generate-openapi-models.

TODO:

- [ ] Fikse resterende kode i appen eller apiet + generere kode på nytt^

  • Generere modeller slik at de blir mest mulig lik develop-greina. Sammenlign med modeller i develop-greina for å finne ut hva som er endret, hvis det er til noe hjelp
  • Justere API'et der det trengs, f.eks. legge på [JsonRequired] på felter som ikke skal være null
  • Rette kompileringsfeil i appen
  • Test å ta bort overstyring av nullable felter slik at de også kan være null. Hvis det fører til mange byggefeil, dropp det. NB! Mer nyttig med nullable felter i edit-modeller enn i view-modeller
  • Fjerne sortering av felter i klassene om vi ikke får til å få med JsDoc. Løst ved at vi nå sorterer i API'et i stedet
  • Prøv å sortere importer i modellene, siden de var det fra før
  • Fjerne gammel kode og npm-script (generate-swagger-api-module) + config-fil
  • Fjerne modeller som ble generert i mappa common-regobs-api/model
  • Endre api-url fra localhost til det den skal være i openapitools.json

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants