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

Guide: Correspondence integration #1950

Merged
merged 9 commits into from
Dec 20, 2024
Merged

Conversation

danielskovli
Copy link
Contributor

@danielskovli danielskovli commented Dec 20, 2024

Description

Adds a guide on correspondence integration in an Altinn app.

Also renames and tweaks the Maskinporten guide and updates the metrics table.

Note

This PR also changes the behavior of the code tags. Inline code tags are no longer implement the copy-to-clipboard functionality. This change was discussed a couple of times on Slack (1, 2).

Copy link
Contributor

@martinothamar martinothamar left a comment

Choose a reason for hiding this comment

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

LGTM! One thing we can consider is using a more "real" example, like implementing IProcessTaskEnd or something

@danielskovli
Copy link
Contributor Author

@martinothamar I didn't do a real scenario like that on purpose, because it would complicate the code example just a little bit. Other than that I think it's a good idea in general. Backlog for 2025 🎅

@danielskovli danielskovli marked this pull request as ready for review December 20, 2024 12:38
@danielskovli danielskovli merged commit c200668 into master Dec 20, 2024
4 checks passed
@danielskovli danielskovli deleted the app-correspondence-integration branch December 20, 2024 13:07
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