Eosu 1026 achievements stop working#1257
Open
JessTello wants to merge 5 commits intorelease-6.1.0from
Open
Conversation
[Public] Release 6.0.0
Release 6.0.2
- Added a persisted refresh action s_connectLoginRefreshAction to com.playeveryware.eos/Runtime/Core/EOSManager.cs so the manager can run an up-to-date re-login strategy when connect credentials expire. - Registered refresh actions at all Connect entry points: StartConnectLoginWithEpicAccount, StartConnectLoginWithOptions(ExternalCredentialType, ...), and StartConnectLoginWithOptions(LoginOptions, ...) so the latest login path can be retried on expiration. - Replaced the previous direct call that retried the captured connectLoginOptions in the expiration callback with a centralized RefreshConnectLoginSession() that invokes the stored action. - Added CloneConnectLoginOptions helper to capture a safe snapshot for the LoginOptions path and added logging when a refresh action is missing.
matt-clarke
previously approved these changes
Mar 18, 2026
kisecrow9334
approved these changes
Mar 18, 2026
billfr-ApexSystems
previously approved these changes
Mar 18, 2026
- Restored achievements operations after auth expiry and added UI safeguards during async icon loads. - Added SemaphoreSlim-based serialized recovery via EnsureValidProductUserIdAsync to refresh ProductUserId and cached achievements. - Implemented retry logic for EOS calls through ShouldRetryAfterReconnect, updating QueryPlayerAchievementsAsync and UnlockAchievementAsync to transparently retry on transient auth connect failures. - Reworked unlock flow to return a concrete PlayerAchievement and update _playerAchievements atomically on success, enhancing error reporting for invalid inputs. - Improved icon download and cache handling: added TryGetValue checks, URI validation, null-callback early returns, and synchronous return for cached data. - Added UI guard in UIAchievementsMenu.DisplayPlayerAchievement to show descriptive messages and disable controls when ProductUserId is null or invalid.
JessTello
dismissed stale reviews from billfr-ApexSystems and matt-clarke
via
d94c066
JessTello
requested review from
billfr-ApexSystems,
kisecrow9334 and
matt-clarke
and removed request for
kisecrow9334 and
matt-clarke
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.