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

add controller for handling generic events #69

Merged

Conversation

chrrust
Copy link
Contributor

@chrrust chrrust commented Nov 27, 2023

Description

The purpose of this PR is to synchronize the endpoints for events between Altinn Platform and localtest. In Altinn Events API, there are two endpoints for processing events. One for app events, and one for generic events. Currently, only the app events endpoint is present in localtest.

The PR adds an endpoint for processing generic events. Validation works the same as in Altinn Events API. However, as with app events, the events are just saved to a file and not done anything with.

The added endpoint for generic events is the same as the app events endpoint with some minor tweaks for validation and routing.

Verification

  • Your code builds clean without any errors or warnings
  • Manual testing done (required)
  • Relevant automated test added (if you find this hard, leave it and we'll help out)
  • All tests run green

Copy link
Member

@ivarne ivarne left a comment

Choose a reason for hiding this comment

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

Ser greit ut.

@chrrust chrrust closed this Nov 28, 2023
@chrrust chrrust reopened this Nov 28, 2023
@chrrust
Copy link
Contributor Author

chrrust commented Nov 28, 2023

@ivarne Har du mulighet til å merge? Har ikke rettigheter til det

@ivarne ivarne merged commit 2200dd5 into Altinn:main Nov 28, 2023
2 checks passed
@chrrust chrrust deleted the chore/add-missing-controller-for-generic-events branch November 28, 2023 15:48
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