Add `resourceid` to Recipient model

khanrn commented 9 months ago

Description

When a notification order is getting generated it may affiliated with a service. In other word the notification to person identified by national identity number with service affiliation should posses an resourceId.

Consideration

This issue is closely related to #436.

Tasks

API

[ ] Add resourceId property to external recipient model, RecipientExt
[ ] The resourceId property should be nullable as this is optional, because is could be with or without service affiliation.
[ ] Add mapping of recipient resourceId to relevant mapper methods in OrderMapper.cs
[ ] Add mapping of new properties in NotificationSummaryMappers

Core

[ ] Modify Recipient.cs
- Add resourceId property as nullable
[ ] Modify EmailRecipient.cs
- Add resourceId property as nullable
[ ] Add SmsRecipient.cs
- Add resourceId property as nullable

acn-sbuad commented 7 months ago

Not sure this needs to be linked to the recipient. Might be enough to link it to the order?

SandGrainOne commented 7 months ago

Not sure this needs to be linked to the recipient. Might be enough to link it to the order?

I was thinking the same. If the application owner want to create notifications for multiple resources they would need to create multiple order, but I don't think this will be an issue.

acn-sbuad commented 6 months ago

Closing issue as it is not relevant. resourceId has been added to NotificationOrder model

Altinn / altinn-notifications