Skip to content

Commit

Permalink
feat: implement lazy loading of files with pagination
Browse files Browse the repository at this point in the history
Signed-off-by: Victor Bianchi <[email protected]>
  • Loading branch information
v-bianchi committed Oct 28, 2024
1 parent 933046a commit 62afbac
Show file tree
Hide file tree
Showing 2 changed files with 49 additions and 3 deletions.
27 changes: 24 additions & 3 deletions src/store/files.js
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,10 @@ export const useFilesStore = function(...args) {
filterActive: 'all',
canRequestSign: loadState('libresign', 'can_request_sign', false),
ordered: [],
paginationCurrent: '',
paginationNext: '',
paginationLast: '',
paginationLength: '',
}
},

Expand Down Expand Up @@ -222,6 +226,7 @@ export const useFilesStore = function(...args) {
}
},
async getAllFiles(filter) {
// Build filter params object
if (!filter) filter = {}
const { chips } = useFiltersStore()
if (chips?.status?.length) {
Expand All @@ -239,12 +244,28 @@ export const useFilesStore = function(...args) {
if (sortingDirection) {
filter.sortDirection = sortingDirection
}
const response = await axios.get(generateOcsUrl('/apps/libresign/api/v1/file/list'), { params: filter })
this.files = {}
this.ordered = []

// Build pagination params object
const pagination = {}
if (this.paginationNext) pagination.page = this.paginationNext
if (this.paginationLength) pagination.length = this.paginationNext

// Call API
const response = await axios.get(generateOcsUrl('/apps/libresign/api/v1/file/list'), { params: {...filter, ...pagination} })

Check failure on line 254 in src/store/files.js

View workflow job for this annotation

GitHub Actions / NPM lint

A space is required after '{'

Check failure on line 254 in src/store/files.js

View workflow job for this annotation

GitHub Actions / NPM lint

A space is required before '}'

// Update state according to API response
if (!this.paginationCurrent) {
this.files = {}
this.ordered = []
}
response.data.ocs.data.data.forEach(file => {
this.addFile(file)
})
const currentPaginationParams = new URLSearchParams(response.data.ocs.data.pagination.current.split('?')[1])
this.paginationCurrent = currentPaginationParams.get('page')
this.paginationLength = currentPaginationParams.get('length')
this.paginationNext = new URLSearchParams(response.data.ocs.data.pagination.next.split('?')[1]).get('page')
this.paginationLast = new URLSearchParams(response.data.ocs.data.pagination.last.split('?')[1]).get('page')
return this.files
},
filesSorted() {
Expand Down
25 changes: 25 additions & 0 deletions src/views/FilesList/VirtualList.vue
Original file line number Diff line number Diff line change
Expand Up @@ -47,5 +47,30 @@ export default {
userConfigStore,
}
},
data() {
return {
observer: null,
}
},
mounted() {
this.observer = new IntersectionObserver(([entry]) => {
if (entry && entry.isIntersecting) {
this.filesStore.getAllFiles()
}
})
// Is there a better way to target the last tr element? Maybe something like useRef?
if (this.$el.querySelector('table tbody tr:last-child')) {
this.observer.observe(this.$el.querySelector('table tbody tr:last-child'))
}
},
beforeDestroy() {
if (this.observer) this.observer.disconnect()
},
updated() {
if (this.observer) {
this.observer.disconnect()
this.observer.observe(this.$el.querySelector('table tbody tr:last-child'))
}
},
}
</script>

0 comments on commit 62afbac

Please sign in to comment.