Fix lists UI queries overriding full lists in selections elsewhere. C…

…loses #1400.
knadh · Jul 20, 2023 · 0be5901 · 0be5901
1 parent 534c875
commit 0be5901
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 3 deletions.
diff --git a/frontend/src/App.vue b/frontend/src/App.vue
@@ -168,7 +168,7 @@ export default Vue.extend({
   mounted() {
     // Lists is required across different views. On app load, fetch the lists
     // and have them in the store.
-    this.$api.getLists({ minimal: true });
+    this.$api.getLists({ minimal: true, per_page: 'all' });
 
     window.addEventListener('resize', () => {
       this.windowWidth = window.innerWidth;

diff --git a/frontend/src/api/index.js b/frontend/src/api/index.js
@@ -115,6 +115,12 @@ export const getLists = (params) => http.get('/api/lists',
     store: models.lists,
   });
 
+export const queryLists = (params) => http.get('/api/lists',
+  {
+    params: (!params ? { per_page: 'all' } : params),
+    loading: models.lists,
+  });
+
 export const getList = async (id) => http.get(`/api/lists/${id}`,
   { loading: models.list });
 

diff --git a/frontend/src/views/Lists.vue b/frontend/src/views/Lists.vue
@@ -173,6 +173,7 @@ export default Vue.extend({
       curItem: null,
       isEditing: false,
       isFormVisible: false,
+      lists: [],
       queryParams: {
         page: 1,
         query: '',
@@ -228,11 +229,13 @@ export default Vue.extend({
     },
 
     getLists() {
-      this.$api.getLists({
+      this.$api.queryLists({
         page: this.queryParams.page,
         query: this.queryParams.query,
         order_by: this.queryParams.orderBy,
         order: this.queryParams.order,
+      }).then((resp) => {
+        this.lists = resp;
       });
     },
 
@@ -268,7 +271,7 @@ export default Vue.extend({
   },
 
   computed: {
-    ...mapState(['loading', 'lists', 'settings']),
+    ...mapState(['loading', 'settings']),
   },
 
   mounted() {