Skip to content

fix(library): owned library sort uses playtime and sane popularity - #393

Merged
Jayian1890 merged 3 commits into
devfrom
fix/owned-library-sort-playtime
May 5, 2026
Merged

fix(library): owned library sort uses playtime and sane popularity#393
Jayian1890 merged 3 commits into
devfrom
fix/owned-library-sort-playtime

Conversation

@Jayian1890

Copy link
Copy Markdown
Collaborator

Summary

  • Last played sorts by local playtime last-played timestamps, with game.lastPlayed as fallback when present.
  • Last added no longer misuses lastPlayed; list order stays the server-provided order until a real added-at field exists.
  • Most popular sorts by playtime-derived signal (total seconds, then session count) instead of membership tier label presence.

Test plan

  • Open Owned Library and confirm Last Played matches recent play order.
  • Confirm Last Added matches API/library load order (unchanged relative to fetch).
  • Confirm Most Popular ranks heavily played titles above unplayed ones.

@Jayian1890
Jayian1890 merged commit 4274821 into dev May 5, 2026
8 checks passed
@Jayian1890
Jayian1890 deleted the fix/owned-library-sort-playtime branch May 7, 2026 02:04
Jayian1890 added a commit that referenced this pull request May 16, 2026
)

* fix(library): correct owned-library sort keys

Use playtime store for last played and most popular; preserve server order for last added until addedAt exists.

* fix(library): improve sorting logic for owned library by playtime

* Merge branch 'dev' into fix/owned-library-sort-playtime
Chizuui pushed a commit to Chizuui/OpenNOW-Modified that referenced this pull request Aug 11, 2026
…penCloudGaming#393)

* fix(library): correct owned-library sort keys

Use playtime store for last played and most popular; preserve server order for last added until addedAt exists.

* fix(library): improve sorting logic for owned library by playtime

* Merge branch 'dev' into fix/owned-library-sort-playtime
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.

1 participant