Fixed a performance issue

viktorgullmark · Mar 29, 2020 · 9afa528 · 9afa528
1 parent 8aaef9a
commit 9afa528
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 5 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,9 +1,12 @@
 # Changelog
 All notable changes to this project will be documented in this file.
 
-## [0.3.5] - NOT RELEASED
+## [0.3.5] - 2020-03-29
 ##### Added
 - Added the ability to filter stash tabs in the tab breakdown chart
+##### Fixed
+- Fixed a performance issue introduced with the last update
+    - We now limit the snapshots to a count of 50 in the tab breakdown chart
 
 ## [0.3.4] - 2020-03-28
 ##### Added

diff --git a/ExilenceNextApp/package.json b/ExilenceNextApp/package.json
@@ -1,6 +1,6 @@
 {
   "name": "exilence-next-app",
-  "version": "0.3.4",
+  "version": "0.3.5",
   "private": true,
   "dependencies": {
     "@material-ui/core": "^4.5.2",

diff --git a/ExilenceNextApp/src/config/app.config.dev.ts b/ExilenceNextApp/src/config/app.config.dev.ts
@@ -1,5 +1,5 @@
 const devConfig = {
-  baseUrl: 'https://localhost:5001',
+  baseUrl: 'https://next.exilence.app',
   production: false,
   sentryBrowserDsn: undefined,
   i18nUrl: '/i18n/{{lng}}/{{ns}}.json',

diff --git a/ExilenceNextApp/src/store/domains/profile.ts b/ExilenceNextApp/src/store/domains/profile.ts
@@ -157,11 +157,13 @@ export class Profile {
 
   @computed
   get tabChartData() {
+    let snapshots = [...this.snapshots.slice(0, 50)];
+
     const league = rootStore.leagueStore.leagues.find(
       l => l.id === this.activeLeagueId
     );
 
-    if (this.snapshots.length === 0 || !league) {
+    if (snapshots.length === 0 || !league) {
       return undefined;
     }
 
@@ -177,7 +179,7 @@ export class Profile {
 
     let stashTabSnapshots: IChartStashTabSnapshot[] = [];
 
-    this.snapshots.map(s => {
+    snapshots.map(s => {
       const data = s.stashTabSnapshots.map(sts => {
         return {
           value: sts.value,