Feature/508 notification with resource - Githubissues

Altinn / altinn-notifications

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

MIT License

3 stars 3 forks source link

Feature/508 notification with resource #517

Closed acn-sbuad closed 3 months ago

acn-sbuad commented 3 months ago

Description

Changed a few model interfaces to base classes to avoid repeating properties.
Moved ignoreReservations and resourceId to base class => consequence: the properties will be included in all order endpoints including /status and /notifications. Do we want this, or should I move the properties to the relevant models? Have not made relevant repository changes yet.
New profile client to retrieve unit details
Updated to contact point service to use both profile client and authorization service

Related Issue(s)

508

Verification

[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

Documentation

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

sonarcloud[bot] commented 3 months ago

Quality Gate passed

Issues
1 New issue
1 Accepted issue

Measures
0 Security Hotspots
98.2% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud