Skip to content
New issue

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

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

Already on GitHub? # to your account

Varslingsmaler for varsling #305

Merged
merged 7 commits into from
Sep 30, 2024
Merged

Varslingsmaler for varsling #305

merged 7 commits into from
Sep 30, 2024

Conversation

Andreass2
Copy link
Collaborator

@Andreass2 Andreass2 commented Sep 27, 2024

Description

Templates er opprettet for engelsk, norsk og nynorsk. Tekstene bør gjennomgås.
De lagres i databasen, slik at endringer kan gjøres uten kodeendringer i prodmilljø.

Det er også satt opp validering av innhold i varslene, basert på hvilken template som blir brukt,

Related Issue(s)

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

Documentation

  • User documentation is updated with a separate linked PR in altinn-studio-docs. (if applicable)

@Andreass2 Andreass2 marked this pull request as ready for review September 30, 2024 07:17
@Andreass2 Andreass2 changed the title initial notification template functionality Varslingsmaler for varsling Sep 30, 2024
@Andreass2 Andreass2 added the kind/feature-request New feature or request label Sep 30, 2024
Copy link
Collaborator

@Ceredron Ceredron left a comment

Choose a reason for hiding this comment

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

Looks good!

@Andreass2 Andreass2 merged commit 0ab14a5 into main Sep 30, 2024
4 checks passed
@Andreass2 Andreass2 deleted the notifications/templates branch September 30, 2024 08:57
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
kind/feature-request New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants