Showing message on successful save #140

Author: tomasbjerre

CHANGELOG.md

@@ -3,10 +3,16 @@
 Changelog of Pull Request Notifier for Bitbucket.
 
 ## Unreleased
+### GitHub [#140](https://github.com/tomasbjerre/pull-request-notifier-for-bitbucket/issues/140) Show notification when settings are saved
+  Showing message on successful save
+  
+  [7f30b9cf2b6cbee](https://github.com/tomasbjerre/pull-request-notifier-for-bitbucket/commit/7f30b9cf2b6cbee) Tomas Bjerre *2016-08-17 17:46:52*
+
+## 2.36
 ### GitHub [#139](https://github.com/tomasbjerre/pull-request-notifier-for-bitbucket/issues/139) Bitbucket - PR Button created at project level shows up for other projects in the same host
   Hiding project-level buttons from other projects
 
-  [47b1aaea2806d9d](https://github.com/tomasbjerre/pull-request-notifier-for-bitbucket/commit/47b1aaea2806d9d) Tomas Bjerre *2016-08-12 15:32:23*
+  [ad97c910e5cf25f](https://github.com/tomasbjerre/pull-request-notifier-for-bitbucket/commit/ad97c910e5cf25f) Tomas Bjerre *2016-08-12 15:32:45*
 
 ### No issue
   doc

docker-build.sh

@@ -1,2 +1,3 @@
 #!/bin/bash
+docker rm prnfb
 docker build -t prnfb .

docker-logs.sh

@@ -0,0 +1,2 @@
+#!/bin/bash
+docker logs -f prnfb

src/main/resources/pr-triggerbutton.js

@@ -23,14 +23,14 @@ define('plugin/prnfb/pr-triggerbutton', [
      AJS.flag({
       close: 'auto',
       type: 'success',
-      title: notificationResponse.notificationName.replace(/<script>/g,'script'),
+      title: notificationResponse.notificationName.replace(/<script>/g, 'script'),
       body: '<p>You may check network tab in web browser for exact URL and response.</p>'
      });
     } else {
      AJS.flag({
       close: 'auto',
       type: 'error',
-      title: notificationResponse.notificationName.replace(/<script>/g,'script'),
+      title: notificationResponse.notificationName.replace(/<script>/g, 'script'),
       body: '<p>' + notificationResponse.status + ' ' + notificationResponse.uri + '</p>' +
        '<p>You may check network tab in web browser for exact URL and response.</p>'
      });
@@ -61,7 +61,7 @@ define('plugin/prnfb/pr-triggerbutton', [
  function loadSettingsAndShowButtons() {
   $.get(buttonsAdminUrl + '/repository/' + pageState.getRepository().id + '/pullrequest/' + pageState.getPullRequest().id, function(settings) {
    settings.forEach(function(item) {
-    var $buttonDropdownItem = buttonTemplate(item.name.replace(/<script>/g,'script'));
+    var $buttonDropdownItem = buttonTemplate(item.name.replace(/<script>/g, 'script'));
     $buttonDropdownItem.click(function() {
      var $this = $(this);
      $this.attr("disabled", "disabled");

src/main/resources/utils.js

@@ -17,13 +17,19 @@ define('plugin/prnfb/utils', [
     if (data && data.uuid) {
      doSetupForm(formSelector, url + '/' + data.uuid);
     }
+    AJS.flag({
+     close: 'auto',
+     type: 'success',
+     title: 'Saved',
+     body: '<p>=)</p>'
+    });
    },
    error: function(xhr, status, error) {
     AJS.messages.error(".statusresponse", {
      title: 'Error',
      body: '<p>' +
-      'Sent POST ' + url + ':<br/><code>' + jsonString.replace(/<script>/g,'script') + '</code><br/><br/>' +
-      'Got:<br/><code>' + xhr.responseText.replace(/<script>/g,'script') + '</code><br/><br/>' +
+      'Sent POST ' + url + ':<br/><code>' + jsonString.replace(/<script>/g, 'script') + '</code><br/><br/>' +
+      'Got:<br/><code>' + xhr.responseText.replace(/<script>/g, 'script') + '</code><br/><br/>' +
       '</p>'
     });
     $("html, body").animate({
@@ -205,7 +211,7 @@ define('plugin/prnfb/utils', [
     $(formSelector + ' [name=uuid]').append('<option value="">New</option>');
     for (var i = 0; i < data.length; i++) {
      var name = data[i].name;
-     name = name.replace(/<script>/g,'script')
+     name = name.replace(/<script>/g, 'script')
      $(formSelector + ' [name=uuid]').append('<option value="' + data[i].uuid + '">' + (data[i].projectKey || '') + ' ' + (data[i].repositorySlug || '') + ' ' + name + '</option>');
     }
    });