Skip to content

Commit b22591e

Browse files
madisparntopiayhrytsai-atl
authored andcommitted
Issue #276: Remove UI changes
1 parent 9c8ae5d commit b22591e

File tree

5 files changed

+11
-14
lines changed

5 files changed

+11
-14
lines changed

bitbucket-slack-server-integration-plugin/src/main/java/com/atlassian/bitbucket/plugins/slack/notification/NotificationUtil.java

+5-1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
package com.atlassian.bitbucket.plugins.slack.notification;
22

3+
import static com.atlassian.bitbucket.plugins.slack.notification.PullRequestNotificationTypes.RESCOPED;
4+
35
import lombok.experimental.UtilityClass;
46

57
import java.util.Collections;
@@ -12,7 +14,9 @@
1214
public class NotificationUtil {
1315
public static Set<String> ALL_NOTIFICATION_TYPE_KEYS = Collections.unmodifiableSet(Stream
1416
.of(
15-
Stream.of(PullRequestNotificationTypes.values()).map(PullRequestNotificationTypes::getKey),
17+
Stream.of(PullRequestNotificationTypes.values())
18+
.filter(type -> RESCOPED != type)
19+
.map(PullRequestNotificationTypes::getKey),
1620
Stream.of(RepositoryNotificationTypes.values()).map(RepositoryNotificationTypes::getKey),
1721
Stream.of(TaskNotificationTypes.values()).map(TaskNotificationTypes::getKey)
1822
)

bitbucket-slack-server-integration-plugin/src/main/resources/analytics/whitelist.json

+3-3
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@
1818
"notifications.slack.repotochannel.notification.enabled": ["userKeyHash", "teamIdHash", "slackUserIdHash", "repositoryId",
1919
"channelIdHash", {
2020
"notificationKey": ["PRCreated", "PRParticipantUpdated", "PRReviewersUpdated", "PRMerged", "PRDeclined",
21-
"PRReopened", "PRDeleted", "PRReScoped", "PRParticipantApproved", "PRParticipantUnapproved", "PRParticipantReviewed",
21+
"PRReopened", "PRDeleted", "PRParticipantApproved", "PRParticipantUnapproved", "PRParticipantReviewed",
2222
"PRComment", "PRCommentEdited", "PRCommentReplied", "PRCommentDeleted", "TaskCreated", "TaskUpdated",
2323
"TaskResolved", "TaskReopened", "TaskDeleted", "PRAutoMergeFailed", "PRAutoMergeResolved", "CommitPushed", "FileEdited",
2424
"CommitComment", "CommitCommentEdited", "CommitCommentReplied", "CommitCommentDeleted", "Forked", "TagCreated",
@@ -28,7 +28,7 @@
2828
"notifications.slack.repotochannel.notification.disabled": ["userKeyHash", "teamIdHash", "slackUserIdHash", "repositoryId",
2929
"channelIdHash", {
3030
"notificationKey": ["PRCreated", "PRParticipantUpdated", "PRReviewersUpdated", "PRMerged", "PRDeclined",
31-
"PRReopened", "PRDeleted", "PRReScoped", "PRParticipantApproved", "PRParticipantUnapproved", "PRParticipantReviewed",
31+
"PRReopened", "PRDeleted", "PRParticipantApproved", "PRParticipantUnapproved", "PRParticipantReviewed",
3232
"PRComment", "PRCommentEdited", "PRCommentReplied", "PRCommentDeleted", "TaskCreated", "TaskUpdated",
3333
"TaskResolved", "TaskReopened", "TaskDeleted", "PRAutoMergeFailed", "PRAutoMergeResolved", "CommitPushed", "FileEdited",
3434
"CommitComment", "CommitCommentEdited", "CommitCommentReplied", "CommitCommentDeleted", "Forked", "TagCreated",
@@ -40,7 +40,7 @@
4040

4141
"notifications.slack.notification.regular.sent": ["userKeyHash", "teamIdHash", "slackUserIdHash", {
4242
"notificationKey": ["PRCreated", "PRParticipantUpdated", "PRReviewersUpdated", "PRMerged", "PRDeclined",
43-
"PRReopened", "PRDeleted", "PRReScoped", "PRParticipantApproved", "PRParticipantUnapproved", "PRParticipantReviewed",
43+
"PRReopened", "PRDeleted", "PRParticipantApproved", "PRParticipantUnapproved", "PRParticipantReviewed",
4444
"PRComment", "PRCommentEdited", "PRCommentReplied", "PRCommentDeleted", "TaskCreated", "TaskUpdated",
4545
"TaskResolved", "TaskReopened", "TaskDeleted", "PRAutoMergeFailed", "PRAutoMergeResolved", "CommitPushed", "FileEdited",
4646
"CommitComment", "CommitCommentEdited", "CommitCommentReplied", "CommitCommentDeleted", "Forked", "TagCreated",

bitbucket-slack-server-integration-plugin/src/main/resources/com/atlassian/bitbucket/plugins/slack/bitbucket-slack.properties

+2-3
Original file line numberDiff line numberDiff line change
@@ -103,7 +103,6 @@ bitbucket.plugins.slack.notification.PRReopened=Reopened
103103
bitbucket.plugins.slack.notification.PRDeleted=Deleted
104104
bitbucket.plugins.slack.notification.PRMerged=Merged
105105
bitbucket.plugins.slack.notification.PRUpdated=Updated
106-
bitbucket.plugins.slack.notification.PRReScoped=Updated
107106
bitbucket.plugins.slack.notification.PRReviewersUpdated=Reviewers updated
108107
bitbucket.plugins.slack.notification.PRComment=Comment created
109108
bitbucket.plugins.slack.notification.PRCommentEdited=Comment edited
@@ -228,7 +227,7 @@ slack.activity.pr.reviewed.short=:warning: {0} *reviewed* pull request {1} in {2
228227
slack.activity.pr.unapproved.needswork.short={0} *removed needs work review* from pull request {1} in {2}
229228
slack.activity.pr.merged.short=:tada: {0} *merged* pull request {1} in {2}
230229
slack.activity.pr.updated.short={0} *updated* pull request {1} in {2}
231-
slack.activity.pr.rescoped.short={0} *updated* pull request {1} in {2}
230+
slack.activity.pr.rescoped.short={0} *re-scoped* pull request {1} in {2}
232231
slack.activity.pr.comment.added.short=:speech_balloon: {0} *commented* on pull request {1} in {2}
233232
slack.activity.pr.comment.deleted.short=:speech_balloon: {0} *deleted a comment* on pull request {1} in {2}
234233
slack.activity.pr.comment.edited.short=:speech_balloon: {0} *edited a comment* on pull request {1} in {2}
@@ -251,7 +250,7 @@ slack.activity.pr.reviewed.long=:warning: {0} *reviewed* pull request {1} {3} \u
251250
slack.activity.pr.unapproved.needswork.long={0} *removed needs work review* from pull request {1} {3} \u2192 {4} in {2}
252251
slack.activity.pr.merged.long=:tada: {0} *merged* pull request {1} {3} \u2192 {4} in {2}
253252
slack.activity.pr.updated.long={0} *updated* pull request {1} {3} \u2192 {4} in {2}
254-
slack.activity.pr.rescoped.long={0} *updated* pull request {1} {3} \u2192 {4} in {2}
253+
slack.activity.pr.rescoped.long={0} *re-scoped* pull request {1} {3} \u2192 {4} in {2}
255254
slack.activity.pr.comment.added.long=:speech_balloon: {0} *commented* on pull request {1} {3} \u2192 {4} in {2}
256255
slack.activity.pr.comment.deleted.long=:speech_balloon: {0} *deleted a comment* on pull request {1} {3} \u2192 {4} in {2}
257256
slack.activity.pr.comment.edited.long=:speech_balloon: {0} *edited a comment* on pull request {1} {3} \u2192 {4} in {2}

bitbucket-slack-server-integration-plugin/src/main/resources/slack-static/feature/repo-channel-table/repo-channel-table.soy

-6
Original file line numberDiff line numberDiff line change
@@ -116,12 +116,6 @@
116116
{param channelConfig: $channelConfig /}
117117
{param repository: $repository/}
118118
{/call}
119-
{call .renderItem}
120-
{param notificationKey: 'PRReScoped'/}
121-
{param label: getText('bitbucket.plugins.slack.notification.PRReScoped')/}
122-
{param channelConfig: $channelConfig /}
123-
{param repository: $repository/}
124-
{/call}
125119
{call .renderItem}
126120
{param notificationKey: 'PRReviewersUpdated'/}
127121
{param label: getText('bitbucket.plugins.slack.notification.PRReviewersUpdated')/}

bitbucket-slack-server-integration-plugin/src/test/java/com/atlassian/bitbucket/plugins/slack/notification/configuration/DefaultNotificationConfigurationServiceTest.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ public void disable_shouldPerformExpectedAction() {
5050

5151
verify(permissionService).validateForRepository(repository, Permission.REPO_ADMIN);
5252
verify(dao).delete(request);
53-
verify(eventPublisher, times(33)).publish(any(RepositoryNotificationDisabledAnalyticEvent.class));
53+
verify(eventPublisher, times(32)).publish(any(RepositoryNotificationDisabledAnalyticEvent.class));
5454
}
5555

5656
@Test

0 commit comments

Comments
 (0)