Skip to content

fix: include user ID in cache to ensure per-user data & redirect to Browse on user change#72

Open
999romain wants to merge 2 commits intoMoonfin-Client:deprecated-tizen-onlyfrom
999romain:quickfix/ensure-per-user-data
Open

fix: include user ID in cache to ensure per-user data & redirect to Browse on user change#72
999romain wants to merge 2 commits intoMoonfin-Client:deprecated-tizen-onlyfrom
999romain:quickfix/ensure-per-user-data

Conversation

@999romain
Copy link

@999romain 999romain commented Feb 15, 2026

Pull Request

Summary

Scope browse cache data per user by storing and validating "userId", ensuring each user sees their own Continue Watching for example.

Type of Change

  • Bug fix

Changes Made

  • Store "userId" alongside "serverUrl" in the browse cache payload.
  • Validate cache reads against both "serverUrl" and "userId".
  • Redirect to Browse panel on user switch by comparing previous and current userId.

Testing

  • Tested on emulator

Test Steps

  1. Switch user.
  2. Ensure “Continue Watching” matches the selected user.

Checklist

  • Code builds successfully
  • Code follows project style and conventions
  • No unnecessary commented-out code
  • No new warnings introduced

@999romain 999romain marked this pull request as ready for review February 15, 2026 13:49
@olivercastle
Copy link

Great improvement. Solves the issue for me.

Probably doesn't sit within this PR, but i noticed only the homepage is refreshed upon user switch, library pages, for example, aren't refreshed so still show the old users library until you come off and back onto that page.

@999romain
Copy link
Author

Great improvement. Solves the issue for me.

Probably doesn't sit within this PR, but i noticed only the homepage is refreshed upon user switch, library pages, for example, aren't refreshed so still show the old users library until you come off and back onto that page.

Woud be better to redirect to the browse panel at each user changes or "refresh" the current panel ?

@olivercastle
Copy link

Great improvement. Solves the issue for me.
Probably doesn't sit within this PR, but i noticed only the homepage is refreshed upon user switch, library pages, for example, aren't refreshed so still show the old users library until you come off and back onto that page.

Woud be better to redirect to the browse panel at each user changes or "refresh" the current panel ?

Yer, good point. - I guess heading to the home screen would make the most sense IMO.

@999romain 999romain changed the title fix: include user ID in cache to ensure per-user data fix: include user ID in cache to ensure per-user data & redirect to Browse on user change Feb 17, 2026
@olivercastle
Copy link

Yep, that works well. 👌

@RadicalMuffinMan
Copy link

Please clone the new repo and implement your changes. I just merged webOS and Tizen codebases

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants