forked from eclipse-tractusx/traceability-foss
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge pull request #1207 from catenax-ng/chore/849-notifications-to-m…
…ultiple-bpns chore(concept): #849 create concept
- Loading branch information
Showing
5 changed files
with
98 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
97 changes: 97 additions & 0 deletions
97
...cept/#849-notifications-to-multiple-bpns/#849-notifications-to-multiple-bpns.md
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,97 @@ | ||
# \[Concept\] \[#849\] Quality notifications to multiple BPNs | ||
|
||
| Key | Value | | ||
|---------------|--------------------------------------------------------------------------| | ||
| Author | @ds-crehm | | ||
| Creation date | 22.04.2024 | | ||
| Ticket Id | [#849](https://github.com/eclipse-tractusx/traceability-foss/issues/849) | | ||
| State | DRAFT | | ||
|
||
# Table of Contents | ||
1. [Overview](#overview) | ||
2. [Requirements](#requirements) | ||
3. [Out of scope](#out-of-scope) | ||
4. [Concept](#concept) | ||
5. [Additional Details](#additional-details) | ||
|
||
# Overview | ||
Instead of having "messages" that can be sent to multiple BPNs within one notification, multiple notifications must be created. | ||
Those can be individually edited and sent to the specific BPN. | ||
|
||
# Requirements | ||
- For alerts, when selecting multiple BPNs, a duplicated alert for each of the selected BPNs is created. | ||
- For investigations, when selecting parts from different BPNs, a duplicated investigation for each of the BPNs is created. | ||
- A case attribute is added to notifications, that can be set during creation and edit processes. | ||
- The case attribute is shown in the inbox table and can be searched and filtered for. | ||
- Notifications are tied to single BPNs and cannot be sent to multiple BPNs anymore. | ||
- BPN selection is only possibly for alerts. | ||
- Autocomplete filter -> Selection shows all matching BPNs (if there are more than 10, this selection is scrollable). | ||
- Selected BPNs are shown as chips that can be removed. | ||
- BPN selection is disabled for investigations, instead the BPNs are "pulled" from the selected parts. | ||
- The frontend modals appear when a user is creating notifications and triggers a notification duplication. | ||
- In the modal the user has the chance to cancel or approve the process. | ||
- During the "Edit notification" process, the "Available parts" table is filtered to show only the parts from the currently selected BPNs. | ||
|
||
# Out of scope | ||
- A duplication process for the user. In case the user wants to retroactively add BPNs to an existing notification or send a notification to a BPN that was not selected during the creation of this notification, | ||
he must create a new notification and copy the data. In the future, there might be the possibility to add a "Duplicate notification" or "Copy notification" process to Trace-X, but that is out of scope for this concept. | ||
|
||
# Concept | ||
## Backend | ||
Messages within one notification can only be sent between **two** BPNs. To make it possible to send notifications to multiple BPNs, notifications are duplicated during the creation process. | ||
An optional grouping label can be set for each notification created during the same process to identify notifications belonging to the same 'topic'. This label can be added/removed afterwards. | ||
|
||
```diff | ||
{ | ||
"id": 423, | ||
"title": null, | ||
"status": "CREATED", | ||
"description": "2024-04-25T07:46:29.523Z-requested-by-chris", | ||
+ "case": "CASE-ID123", | ||
"createdBy": null, | ||
"createdByName": null, | ||
"createdDate": "2024-04-25T07:28:42.538791Z", | ||
"assetIds": [ | ||
"urn:uuid:5205f736-8fc2-4585-b869-6bf36842369a" | ||
], | ||
"channel": "SENDER", | ||
"reason": { | ||
"close": null, | ||
"accept": null, | ||
"decline": null | ||
}, | ||
"sendTo": null, | ||
"sendToName": null, | ||
"severity": "MINOR", | ||
"type": "ALERT", | ||
"targetDate": null, | ||
"messages": [] | ||
} | ||
``` | ||
|
||
#### Alert creation | ||
When an alert is created, multiple BPNs can be selected. For each of the recipients the notification will be duplicated. | ||
The data will be identical except for the "createdFor" and "createdForName" fields. | ||
If the data must be changed, the user can do it afterward in the "edit notification" view. | ||
|
||
#### Investigation creation | ||
When an investigation is created, multiple parts can be selected. Whenever a part from a different BPN is selected, the notification will be duplicated. | ||
The data will be identical except for the "createdFor", "createdForName" and "assetIds" fields. Each notification will only contain parts from the same BPN. | ||
If the data must be changed, the user can do it afterward in the "edit notification" view. | ||
|
||
#### Edit notifications | ||
When editing notifications, only parts with the identical BPN as the "createdFor" BPN will be shown and the user may not add BPNs manually. | ||
There is no possibility to create a new notification from the "Edit notifications" process. | ||
If the user wants to send an existing notification to a different BPN, he must copy the data and use the "Create notification" process. | ||
|
||
## Frontend | ||
![multiple-bpns-modals.png](multiple-bpns-modals.png) | ||
![notification-creation-bpn-selection.png](notification-creation-bpn-selection.png) | ||
|
||
New case label: | ||
![case-label.png](case-label.png) | ||
|
||
See https://miro.com/app/board/uXjVO5JVoho=/?moveToWidget=3458764586657591852&cot=10 | ||
|
||
# Additional Details | ||
Given the dynamic nature of ongoing development, there might be variations between the conceptualization and the current implementation. For the latest status, refer to the documentation. |
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file added
BIN
+48.4 KB
docs/concept/#849-notifications-to-multiple-bpns/multiple-bpns-modals.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file added
BIN
+88.5 KB
...ept/#849-notifications-to-multiple-bpns/notification-creation-bpn-selection.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.