-
Notifications
You must be signed in to change notification settings - Fork 652
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 #74 from robfletcher/master
user-centric notifications for stale app ownership
- Loading branch information
Showing
10 changed files
with
28 additions
and
47 deletions.
There are no files selected for viewing
8 changes: 6 additions & 2 deletions
8
echo-notifications/src/main/resources/templates/appOwnerMultipleUsers/body-email.vm
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 |
---|---|---|
@@ -1,5 +1,9 @@ | ||
The owner email for the following applications matches multiple users: | ||
|
||
#foreach($application in $notification.additionalContext.multiple) | ||
$application.name in $application.accounts owned by <$application.owner.email> | ||
#foreach($event in $notification.additionalContext.events) | ||
$event.email owns: | ||
#foreach($application in $event.applications) | ||
- $application.name in $application.accounts | ||
#end | ||
|
||
#end |
11 changes: 7 additions & 4 deletions
11
echo-notifications/src/main/resources/templates/appOwnerOutdatedNoSuggestion/body-email.vm
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 |
---|---|---|
@@ -1,6 +1,9 @@ | ||
The following applications are owned by ex-employees: | ||
The following users are ex-employees but still own applications. We could not identify managers or team members to reassign to. | ||
|
||
#foreach($event in $notification.additionalContext.events) | ||
$application.owner.name<$application.owner.email> owns... | ||
#foreach($application in $event.applications) | ||
- $application.name in $application.accounts | ||
#end | ||
|
||
#foreach($application in $notification.additionalContext.outdated) | ||
$application.name in $application.accounts owned by $application.owner.name<$application.owner.email> | ||
- no suggested owner found for reassignment. | ||
#end |
12 changes: 8 additions & 4 deletions
12
echo-notifications/src/main/resources/templates/appOwnerOutdatedReassign/body-email.vm
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 |
---|---|---|
@@ -1,6 +1,10 @@ | ||
The following applications are owned by ex-employees: | ||
The following users are ex-employees but still own applications. We have identified possible managers or team members to reassign to. | ||
|
||
#foreach($event in $notification.additionalContext.events) | ||
$application.owner.name<$application.owner.email> owns... | ||
#foreach($application in $event.applications) | ||
- $application.name in $application.accounts | ||
#end | ||
Suggest reassigning to $application.suggestedOwner.name<$application.suggestedOwner.email> | ||
|
||
#foreach($application in $notification.additionalContext.applications) | ||
$application.name in $application.accounts owned by $application.owner.name<$application.owner.email> | ||
- suggest reassigning to $application.suggestedOwner.name<$application.suggestedOwner.email>. | ||
#end |
10 changes: 7 additions & 3 deletions
10
echo-notifications/src/main/resources/templates/appOwnerUnknown/body-email.vm
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 |
---|---|---|
@@ -1,5 +1,9 @@ | ||
The following applications are owned by unidentifiable users: | ||
The following email addresses are not recognized employee or group addresses but own applications. | ||
|
||
#foreach($event in $notification.additionalContext.events) | ||
$event.email owns... | ||
#foreach($application in $event.applications) | ||
- $application.name in $application.accounts | ||
#end | ||
|
||
#foreach($application in $notification.additionalContext.unknown) | ||
$application.name in $application.accounts owned by <$application.owner.email> | ||
#end |
29 changes: 0 additions & 29 deletions
29
echo-notifications/src/main/resources/templates/appOwnership/body-email.vm
This file was deleted.
Oops, something went wrong.
1 change: 0 additions & 1 deletion
1
echo-notifications/src/main/resources/templates/appOwnership/subject.vm
This file was deleted.
Oops, something went wrong.
1 change: 0 additions & 1 deletion
1
echo-notifications/src/main/resources/templates/appReassigned/body-email.vm
This file was deleted.
Oops, something went wrong.
1 change: 0 additions & 1 deletion
1
echo-notifications/src/main/resources/templates/appReassigned/subject.vm
This file was deleted.
Oops, something went wrong.
1 change: 0 additions & 1 deletion
1
echo-notifications/src/main/resources/templates/appUnknownOwner/body-email.vm
This file was deleted.
Oops, something went wrong.
1 change: 0 additions & 1 deletion
1
echo-notifications/src/main/resources/templates/appUnknownOwner/subject.vm
This file was deleted.
Oops, something went wrong.