fix: apply perk failed sometimes.

cangzhang · Apr 16, 2020 · 405e282 · 405e282
1 parent 85ce32b
commit 405e282
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 7 deletions.
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "champ-r",
-  "version": "0.9.0-pre.5",
+  "version": "0.9.0-pre.6",
   "license": "MIT",
   "description": "Get champion items for lol, Windows only.",
   "productName": "ChampR",
@@ -146,7 +146,7 @@
       "package.json"
     ],
     "win": {
-      "icon": "./src/assets/app-icon.ico",
+      "icon": "./src/assets/app-icon.png",
       "requestedExecutionLevel": "highestAvailable"
     }
   },

diff --git a/src/assets/app-icon.ico b/src/assets/app-icon.ico
diff --git a/src/assets/app-icon.png b/src/assets/app-icon.png
diff --git a/src/modules/popup/index.js b/src/modules/popup/index.js
@@ -22,11 +22,6 @@ export default function Popup() {
   const [position, setPosition] = useState('');
   const [perks, setPerkList] = useState([]);
 
-  useEffect(() => {
-    lcu.current = new LCUService(lolDir);
-    lcu.current.getAuthToken();
-  }, [lolDir]);
-
   useEffect(() => {
     getChampions(lolVer)
       .then(championList => {
@@ -64,6 +59,8 @@ export default function Popup() {
     if (!lcu.current.applyPerk || !lcu.current.active)
       return;
 
+    lcu.current = new LCUService(lolDir);
+    await lcu.current.getAuthToken();
     const res = await lcu.current.applyPerk({
       ...perk,
       name: `${perk.alias} @ ${perk.position}`,