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

Bytt payload-typen for nokre endpunkt som ikkje klarte å deserialisere JSON string-nodes riktig #1840

Merged
merged 2 commits into from
Jan 22, 2025

Conversation

slovrid
Copy link
Contributor

@slovrid slovrid commented Jan 22, 2025

Describe your changes

Før hadde vi @RequestBody String variabelNavn, som skulle tilseie at vi forventa JSON payloads på formen "<noko tekst>". Dette likte oppsettet vårt dårleg. Endrar dette til å wrappe i request-typar (dvs. JSON payload på formen {"property1":"<noko tekst">}) som vi veit vi klarar å deserialisere riktig.

Trello ticket number and link

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)

Checklist before requesting a review

  • I have performed a self-review of my code

@slovrid slovrid requested a review from a team as a code owner January 22, 2025 12:06
@github-actions github-actions bot added the not dependabot PRs opened by humans (and other bots) label Jan 22, 2025
Copy link
Contributor

@Mathiamu Mathiamu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🙌

@slovrid slovrid merged commit ce05b61 into dev Jan 22, 2025
10 checks passed
@slovrid slovrid deleted the last-inn-oppfolgingsbruker-bug branch January 22, 2025 12:16
slovrid added a commit that referenced this pull request Jan 22, 2025
Synce master med dev/prodsette #1840
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
not dependabot PRs opened by humans (and other bots)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants