Skip to content

Ensure proper timestamps are set properly for each achievements.#3382

Open
MrKeiKun wants to merge 1 commit intoHerculesWS:masterfrom
MrKeiKun:fix-2418-achievement-bug
Open

Ensure proper timestamps are set properly for each achievements.#3382
MrKeiKun wants to merge 1 commit intoHerculesWS:masterfrom
MrKeiKun:fix-2418-achievement-bug

Conversation

@MrKeiKun
Copy link
Contributor

@MrKeiKun MrKeiKun commented Sep 1, 2025

Pull Request Prelude

Changes Proposed

I reordered the operations in both functions to:

  1. Set completed_at = time(NULL) first
  2. Then call validate_achieve to complete any parent achievements

Issues addressed: #2418

Signed-off-by: Lorenzo Buitizon <the.keikun@gmail.com>
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