Alert system overhaul

- Messages won't be shown more than once
- Improved system and quality

objects/Mana.js

@@ -50,6 +50,8 @@ class Mana {
     this.gameClient = new (require('./riot/leagueoflegends/GameClient'))();
     this.assetsProxy = new (require('./riot/leagueoflegends/GameAssetsProxy'))();
 
+    this.alertHandler = new (require('./handlers/AlertHandler'))();
+
     this.championStorageHandler = new (require('./handlers/ChampionStorageHandler'))();
     this.championSelectHandler = new (require('./handlers/ChampionSelectHandler'))();
     this.providerHandler = new (require('./handlers/ProviderHandler'))(this.devMode);
@@ -78,11 +80,9 @@ class Mana {
 
     this.assetsProxy.load();
 
-    const data = await UI.indicator(Promise.all([this.gameClient.load(), this.gameClient.queryChampionSummary(), this.gameClient.querySummonerSpells(), require('request-promise-native')('https://manaflux-server.herokuapp.com/api/alerts/v1')]), 'status-loading-resources');
+    const data = await UI.indicator(Promise.all([this.gameClient.load(), this.gameClient.queryChampionSummary(), this.gameClient.querySummonerSpells()]), 'status-loading-resources');
 
     this.preseason = data[0];
-
-    this._alert(data[3]);
     $('.version').text(`V${this.version} - V${this.gameClient.branch}`);
 
     await this.championStorageHandler.load();
@@ -95,6 +95,8 @@ class Mana {
     this.getStore().set('lastBranchSeen', this.gameClient.branch);
     document.querySelectorAll('[data-custom-component]').forEach(x => x.dispatchEvent(new Event('clientLoaded')));
 
+    this.alertHandler.load();
+
     ipcRenderer.send('lcu-preload-done');
   }
 
@@ -108,10 +110,9 @@ class Mana {
     document.querySelectorAll('[data-custom-component]').forEach(x => x.dispatchEvent(new Event('userConnected')));
 
     this.championSelectHandler.loop();
-
-    require('request-promise-native')(`https://manaflux-server.herokuapp.com/api/alerts/v1?v=${this.version}&summoner=${this.user.getSummonerId()}`).then(x => this._alert(x)).catch(err => console.error(err));
-
     UI.status('champion-select-waiting');
+
+    this.alertHandler.login();
   }
 
   onLeagueDisconnect() {
@@ -133,11 +134,6 @@ class Mana {
   getStore() {
     return this._store;
   }
-
-  _alert(message, repeat) {
-    if (!repeat && this._lastMessage === message) return;
-    alertify.warning(this._lastMessage = message);
-  }
 }
 
 module.exports = Mana;

objects/handlers/AlertHandler.js

@@ -0,0 +1,43 @@
+const rp = require('request-promise-native');
+class AlertHandler {
+  constructor() {
+
+  }
+
+  async load() {
+    return await this._check(Mana.version);
+  }
+
+  async login() {
+    return await this._check(Mana.version, Mana.user.getSummonerId());
+  }
+
+  async stop() {
+
+  }
+
+  async _check(version = Mana.version, summonerId) {
+    const alerts = Mana.getStore().get('alerts', []);
+
+    try {
+      let messages = await rp({ uri: `https://manaflux-server.herokuapp.com/api/alerts/v2?v=${this.version}${summonerId ? '&summoner=' + summonerId : ''}`, json: true });
+      console.dir(messages);
+
+      messages.filter(x => !alerts.includes(x._id)).forEach(x => {
+        this._alert(x.message);
+        alerts.push(x._id);
+      });
+
+      Mana.getStore().set('alerts', alerts);
+    }
+    catch(err) {
+      console.error(err);
+    }
+  }
+
+  _alert(message) {
+    alertify.warning(message);
+  }
+}
+
+module.exports = AlertHandler;