[Notice] Announcing the End of Demo Server [Read me]

Merge branch 'issue-2193' of dlab/hive
from pull-request 1502 * refs/heads/issue-2193: Notification: removed outdated commits from message Reviewed-by: 이응준
@07d0b07141d510508137a3d6f2d527c0b99be4ed
--- app/models/NotificationEvent.java
+++ app/models/NotificationEvent.java
... | ... | @@ -481,10 +481,12 @@ |
481 | 481 |
builder.append(Messages.get("notification.pullrequest.current.commits")); |
482 | 482 |
builder.append("\n"); |
483 | 483 |
for (PullRequestCommit commit : commits) { |
484 |
- builder.append(commit.getCommitShortId()); |
|
485 |
- builder.append(" "); |
|
486 |
- builder.append(commit.getCommitShortMessage()); |
|
487 |
- builder.append("\n"); |
|
484 |
+ if (commit.state == PullRequestCommit.State.CURRENT) { |
|
485 |
+ builder.append(commit.getCommitShortId()); |
|
486 |
+ builder.append(" "); |
|
487 |
+ builder.append(commit.getCommitShortMessage()); |
|
488 |
+ builder.append("\n"); |
|
489 |
+ } |
|
488 | 490 |
} |
489 | 491 |
return builder.toString(); |
490 | 492 |
} |
Add a comment
Delete comment
Once you delete this comment, you won't be able to recover it. Are you sure you want to delete this comment?