Fix external_achievements (#6667)

open-chat-labs · Oct 24, 2024 · 94a060b · 94a060b
1 parent 139f5b6
commit 94a060b
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 2 deletions.
diff --git a/backend/canisters/user_index/CHANGELOG.md b/backend/canisters/user_index/CHANGELOG.md
@@ -15,6 +15,10 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/).
 
 - Don't remove external achievements on expiry ([#6588](https://github.com/open-chat-labs/open-chat/pull/6588))
 
+### Fixed
+
+- Fix `last_updated` field of `external_achievements` ([#6667](https://github.com/open-chat-labs/open-chat/pull/6667))
+
 ## [[2.0.1384](https://github.com/open-chat-labs/open-chat/releases/tag/v2.0.1384-user_index)] - 2024-10-11
 
 ### Added

diff --git a/backend/canisters/user_index/impl/src/queries/external_achievements.rs b/backend/canisters/user_index/impl/src/queries/external_achievements.rs
@@ -21,7 +21,6 @@ fn external_achievements_impl(args: Args, state: &RuntimeState) -> Response {
         last_updated = max([
             last_updated,
             achievement.registered,
-            achievement.expires,
             achievement.budget_exhausted.unwrap_or_default(),
         ])
         .unwrap();

diff --git a/frontend/openchat-agent/src/utils/caching.ts b/frontend/openchat-agent/src/utils/caching.ts
@@ -52,7 +52,7 @@ import type { CryptocurrencyContent } from "openchat-shared";
 import type { PrizeContent } from "openchat-shared";
 import type { P2PSwapContent } from "openchat-shared";
 
-const CACHE_VERSION = 118;
+const CACHE_VERSION = 119;
 const EARLIEST_SUPPORTED_MIGRATION = 115;
 const MAX_INDEX = 9999999999;
 
@@ -199,6 +199,14 @@ async function clearChatAndGroups(
     await clearGroupDetailsStore(_db, _principal, tx);
 }
 
+async function clearExternalAchievements(
+    _db: IDBPDatabase<ChatSchema>,
+    _principal: Principal,
+    tx: IDBPTransaction<ChatSchema, StoreNames<ChatSchema>[], "versionchange">,
+) {
+    await tx.objectStore("externalAchievements").clear();
+}
+
 const migrations: Record<number, MigrationFunction<ChatSchema>> = {
     115: clearEverything,
     116: async (db, principal, transaction) => {
@@ -214,6 +222,7 @@ const migrations: Record<number, MigrationFunction<ChatSchema>> = {
             createActivityFeed(db, principal, tx),
         ]);
     },
+    119: clearExternalAchievements,
 };
 
 async function migrate(