From 07c34a005e448885187f255b9b7f65d59caf6180 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Ren=C3=A9=20Pfeuffer?= <rene.pfeuffer@cloudogu.com>
Date: Tue, 18 Dec 2018 16:55:13 +0100
Subject: [PATCH] Peer review

---
 .../scm-git-plugin/src/main/js/RepositoryConfig.js  | 13 ++++---------
 .../src/main/resources/locales/en/plugins.json      |  3 ++-
 2 files changed, 6 insertions(+), 10 deletions(-)

diff --git a/scm-plugins/scm-git-plugin/src/main/js/RepositoryConfig.js b/scm-plugins/scm-git-plugin/src/main/js/RepositoryConfig.js
index a9f2a27e37..4fa6c09930 100644
--- a/scm-plugins/scm-git-plugin/src/main/js/RepositoryConfig.js
+++ b/scm-plugins/scm-git-plugin/src/main/js/RepositoryConfig.js
@@ -8,9 +8,9 @@ import {translate} from "react-i18next";
 
 type Props = {
   repository: Repository,
-
   t: string => string
 };
+
 type State = {
   loadingBranches: boolean,
   loadingDefaultBranch: boolean,
@@ -24,10 +24,6 @@ type State = {
 const GIT_CONFIG_CONTENT_TYPE = "application/vnd.scmm-gitConfig+json";
 
 class RepositoryConfig extends React.Component<Props, State> {
-  state = {
-    branches: []
-  };
-
   componentDidMount() {
     const { repository } = this.props;
     this.setState({ ...this.state, loadingBranches: true });
@@ -56,7 +52,6 @@ class RepositoryConfig extends React.Component<Props, State> {
   }
 
   branchSelected = (branch: Branch) => {
-    console.log(branch)
     if (!branch) {
       this.setState({ ...this.state, selectedBranchName: null });
       return;
@@ -89,8 +84,8 @@ class RepositoryConfig extends React.Component<Props, State> {
   };
 
   render() {
-    const { t, error } = this.props;
-    const { loadingBranches, loadingDefaultBranch, submitPending } = this.state;
+    const { t } = this.props;
+    const { loadingBranches, loadingDefaultBranch, submitPending, error } = this.state;
 
     if (error) {
       return (
@@ -136,7 +131,7 @@ class RepositoryConfig extends React.Component<Props, State> {
               this.setState({ ...this.state, defaultBranchChanged: false })
             }
           />
-          Default branch changed!
+          {this.props.t("scm-git-plugin.repo-config.success")}
         </div>
       );
     }
diff --git a/scm-plugins/scm-git-plugin/src/main/resources/locales/en/plugins.json b/scm-plugins/scm-git-plugin/src/main/resources/locales/en/plugins.json
index a1ad9764d1..a84f44726f 100644
--- a/scm-plugins/scm-git-plugin/src/main/resources/locales/en/plugins.json
+++ b/scm-plugins/scm-git-plugin/src/main/resources/locales/en/plugins.json
@@ -30,7 +30,8 @@
       "error": {
         "title": "Error",
         "subtitle": "Something went wrong"
-      }
+      },
+      "success": "Default branch changed!"
     }
   }
 }