added basic validation of recipient nin and orgno

Altinn / altinn-notifications

Altinn platform microservice for handling notifications (mail, sms, etc)

MIT License

2 stars 3 forks source link

Closed acn-sbuad closed 1 month ago

acn-sbuad commented 1 month ago

Validation of digits and length when organization number or national identity number is provided for recipient.

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

sonarcloud[bot] commented 1 month ago

Failed conditions
26 Duplicated Lines on New Code (required ≤ 5)

acn-sbuad commented 1 month ago

Keeping the duplicated lines as is. Moving common logic to new class only saves us a couple of the duplicated lines.