[PR #1771] Clean up progress sync from server to mobile client #1690

Open
opened 2026-04-25 00:00:38 +02:00 by adam · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/advplyr/audiobookshelf-app/pull/1771
Author: @hskrtich
Created: 1/4/2026
Status: 🔄 Open

Base: masterHead: progress-syncing-logs


📝 Commits (1)

  • 918c327 Clean up userMediaProgressUpdated

📊 Changes

1 file changed (+47 additions, -33 deletions)

View changed files

📝 layouts/default.vue (+47 -33)

📄 Description

Brief summary

Add more logging around syncing sessions from the server (web client) to the mobile app.
Reworked the logging to include id's for library item and progress id.
Also converted a few more console.log/console.error messages to use the abs logger.
Also simplify logic around syncing in general to make sure we are logging on edge cases.

Which issue is fixed?

This is to help diagnose sync problems. Such as https://github.com/advplyr/audiobookshelf-app/issues/1161

Pull Request Type

Both android and ios, fronend vue code only

In-depth Description

How have you tested this?

Load app the mobile app
Go to logs
Play a library item on the web client
See the logs!

Screenshots

image

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/advplyr/audiobookshelf-app/pull/1771 **Author:** [@hskrtich](https://github.com/hskrtich) **Created:** 1/4/2026 **Status:** 🔄 Open **Base:** `master` ← **Head:** `progress-syncing-logs` --- ### 📝 Commits (1) - [`918c327`](https://github.com/advplyr/audiobookshelf-app/commit/918c32752a5687bc008fba2224773d86bc4230e2) Clean up userMediaProgressUpdated ### 📊 Changes **1 file changed** (+47 additions, -33 deletions) <details> <summary>View changed files</summary> 📝 `layouts/default.vue` (+47 -33) </details> ### 📄 Description <!-- For Work In Progress Pull Requests, please use the Draft PR feature, see https://github.blog/2019-02-14-introducing-draft-pull-requests/ for further details. If you do not follow this template, the PR may be closed without review. Please ensure all checks pass. If you are a new contributor, the workflows will need to be manually approved before they run. --> ## Brief summary Add more logging around syncing sessions from the server (web client) to the mobile app. Reworked the logging to include id's for library item and progress id. Also converted a few more console.log/console.error messages to use the abs logger. Also simplify logic around syncing in general to make sure we are logging on edge cases. ## Which issue is fixed? This is to help diagnose sync problems. Such as https://github.com/advplyr/audiobookshelf-app/issues/1161 ## Pull Request Type Both android and ios, fronend vue code only <!-- Does this affect only Android, only iOS, or both? Does this change the frontend or the backend of the apps? --> ## In-depth Description <!-- Describe your solution in more depth. How does it work? Why is this the best solution? Does it solve a problem that affects multiple users or is this an edge case for your setup? --> ## How have you tested this? Load app the mobile app Go to logs Play a library item on the web client See the logs! ## Screenshots <img width="602" height="411" alt="image" src="https://github.com/user-attachments/assets/76941231-e6b3-4f19-8f7b-50addfe03d88" /> --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
adam added the pull-request label 2026-04-25 00:00:38 +02:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/audiobookshelf-app#1690