Adding helper methods for each notification type. Fixes ember-templat…

…e-lint#68
bjornharrtell · Dec 16, 2015 · d4b093b · d4b093b
1 parent 713e7b6
commit d4b093b
Show file tree

Hide file tree

Showing 2 changed files with 30 additions and 2 deletions.
diff --git a/addon/services/notification-messages-service.js b/addon/services/notification-messages-service.js
@@ -6,7 +6,6 @@ export default Ember.ArrayProxy.extend({
     defaultClearDuration: 3200,
 
     addNotification(options) {
-
         // If no message is set, throw an error
         if (!options.message) {
             throw new Error("No notification message set");
@@ -29,6 +28,35 @@ export default Ember.ArrayProxy.extend({
         return notification;
     },
 
+    // Helper methods for each type of notification
+    error(message, options) {
+      this.addNotification(Ember.merge({
+        message: message,
+        type: 'error'
+      }, options));
+    },
+
+    success(message, options) {
+      this.addNotification(Ember.merge({
+        message: message,
+        type: 'success'
+      }, options));
+    },
+
+    info(message, options) {
+      this.addNotification(Ember.merge({
+        message: message,
+        type: 'info'
+      }, options));
+    },
+
+    warning(message, options) {
+      this.addNotification(Ember.merge({
+        message: message,
+        type: 'warning'
+      }, options));
+    },
+
     removeNotification(notification) {
         if (!notification) {
             return;

diff --git a/tests/dummy/app/templates/application.hbs b/tests/dummy/app/templates/application.hbs
@@ -41,5 +41,5 @@
     <label>Clear all</label>
   </div>
 </div>
-<button class="button" {{action 'showNotifcation'}}>Show</button>
+<button class="button" {{action "showNotifcation"}}>Show</button>
 {{outlet}}