3240 Show anonymous reviewer rather than current one (which may be wrong) in the reviewer timeline
Use an anonymous reviewer instead of actual reviewers name to try and steer the communication to be inside the system rather than out-of-band like e-mail. In case the reviewer is actually changed all relevant communication is then available to the new reviewer.
This commit is contained in:
parent
cce458f98a
commit
7a67aac779
view/src/main/java/se/su/dsv/scipro/reviewer/timeline
@ -9,7 +9,11 @@
|
||||
<div class="messageWrap">
|
||||
<div class="forumBlueBackground" wicket:id="when"></div>
|
||||
<div class="forumGrayBackground">
|
||||
<div class="vertAlign" wicket:id="reviewer"></div>
|
||||
<div class="vertAlign">
|
||||
<wicket:link>
|
||||
<img src="../../profile/anonymous.png" class="profile-picture-sm"> Reviewer
|
||||
</wicket:link>
|
||||
</div>
|
||||
</div>
|
||||
<div class="forumMessage">
|
||||
<h4>
|
||||
|
@ -19,7 +19,6 @@ class ApprovedPanel extends Panel {
|
||||
super(id);
|
||||
add(new EnumLabel<>("topic", decision.map(Decision::getReviewerApproval).map(ReviewerApproval::getStep)));
|
||||
add(new DateLabel("when", decision.map(Decision::getDecisionDate), DateStyle.DATETIME));
|
||||
add(new UserLinkPanel("reviewer", decision.map(Decision::getReviewerApproval).map(ReviewerApproval::getProject).map(Project::getReviewer)));
|
||||
add(new MultiLineLabel("message", decision.map(Decision::getReason)));
|
||||
add(new ViewAttachmentPanel("attachment", new OrNullModel<>(decision.map(Decision::getAttachment)).map(FileReference::getFileDescription)));
|
||||
}
|
||||
|
@ -9,7 +9,11 @@
|
||||
<div class="messageWrap">
|
||||
<div class="forumBlueBackground" wicket:id="when"></div>
|
||||
<div class="forumGrayBackground">
|
||||
<div class="vertAlign" wicket:id="reviewer"></div>
|
||||
<div class="vertAlign">
|
||||
<wicket:link>
|
||||
<img src="../../profile/anonymous.png" class="profile-picture-sm"> Reviewer
|
||||
</wicket:link>
|
||||
</div>
|
||||
</div>
|
||||
<div class="forumMessage">
|
||||
<h4>
|
||||
|
@ -19,7 +19,6 @@ class DeniedPanel extends Panel {
|
||||
super(id);
|
||||
add(new EnumLabel<>("topic", decision.map(Decision::getReviewerApproval).map(ReviewerApproval::getStep)));
|
||||
add(new DateLabel("when", decision.map(Decision::getDecisionDate), DateStyle.DATETIME));
|
||||
add(new UserLinkPanel("reviewer", decision.map(Decision::getReviewerApproval).map(ReviewerApproval::getProject).map(Project::getReviewer)));
|
||||
add(new MultiLineLabel("message", decision.map(Decision::getReason)));
|
||||
add(new ViewAttachmentPanel("attachment", new OrNullModel<>(decision.map(Decision::getAttachment)).map(FileReference::getFileDescription)));
|
||||
}
|
||||
|
Loading…
x
Reference in New Issue
Block a user