Task #442
closed
Enable sms notifications to be marked "not sent"
Added by Peter Štrunc over 7 years ago.
Updated over 7 years ago.
Description
Currently there is no way for custom implementations of AbstractSmsNotificationSender to tell rest of the application that some messages were not sent. Goal of this ticket is to do just that.
- Related to Task #409: Mark notification as partially sent added
- % Done changed from 0 to 80
I implemented method sendSms on AbstractNotificationSender to return list of IdmSmsNotificationLog. Each log contains information about one sms for one NotificationRecipient. For now, these logs are merged into one to be consistent with rest of the application but later they will be all persisted (see #409).
- Status changed from New to Needs feedback
- Assignee changed from Peter Štrunc to Ondřej Kopr
Implemented in dbddc08c please someone provide review.
- Assignee changed from Ondřej Kopr to Radek Tomiška
- Status changed from Needs feedback to In Progress
- Assignee changed from Radek Tomiška to Peter Štrunc
- Target version set to Citrine (7.3.0)
- % Done changed from 80 to 90
I did review and its ok, could be merged into develop, thx.
Notification api has to be redesigned in #409 (methods: N send(..., IdmIdentityDto recipient) => List<N> send(..., IdmNotificationRecipientDto recipient)). It will be useful for all senders without identity (email only, phone only etc.). Maybe some new manager will be created and currently used manager will be marked deprecated for compatibility issues.
- Status changed from In Progress to Closed
- % Done changed from 90 to 100
Also available in: Atom
PDF
