Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

javaName på schema for bedre pojo-generering. #1

Open
wants to merge 1 commit into
base: gh-pages
Choose a base branch
from

Conversation

johnksv
Copy link

@johnksv johnksv commented Jun 24, 2021

Lager en draf PR for synlighet om at denne branchen benyttes (slik at den ikke blir slettet in the wild).

Legger til javaName på alle SBD-er slik at pojo-generering med jsonschema2pojo gir finere klassenavn. Optimalt hadde dette lugget utenfor schema (i en bindings-fil), men ser ikke ut som det er støttet. Derfor må dette proprietær feltet inn i schema..

Workflow ved oppdatering av https://github.com/difi/felleslosninger

  1. Fetch upstream (dvs ta inn endringene fra difi/felleslosninger inn i dette repoet)
  2. Merge gh-pages inn i denne branchen (her kun man jo brukt rebase, men gikk for merge for synligheten og historikken sin skyld)
  3. Done. Du kan nå oppdater git submodulen i prosjektet ditt

@johnksv
Copy link
Author

johnksv commented Oct 5, 2021

Siste commit: Endring i jsonschema (bruker nå 2019-09 ). Skal også påpekes at eformidlingsskjema har blitt inkludert under /schemas-mappen i en av de siste commitene.

@eivinhb eivinhb marked this pull request as ready for review May 18, 2022 19:32
hermanwh pushed a commit that referenced this pull request May 20, 2022
hermanwh pushed a commit that referenced this pull request May 20, 2022
Merge pull request #1 from felleslosninger/gh-pages
Skjemaet lager ikke så fin kode med gode navn og typer default og pr nå finner vi
ikke noe binding-mulighet fra schema til kodegeneratoren. Vi må derfor
legge endringene inn i schema.

Dette er en rework av arbeid gjort i 2021 av JohnKasper (the man himself)
men branchene ble veldig vanskelig å vedlikeholde når vi lå mange
tusen commits bak.
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.

2 participants