You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Describe what should be investigated or refactored
Related to #959 and #1323. store.ts has some hard-to-test code for migrating values when updating the store cache. Find a way to extract the migration logic that's part of PeprControllerStore.#migrateAndSetupWatch() into an isolated function that can be maintained independently.
You'll need to examine the type definitions and manage differences between cache entries of type Record<string, Operation> and DataStore entries which are defined as Record<string, string>.
The text was updated successfully, but these errors were encountered:
Describe what should be investigated or refactored
Related to #959 and #1323.
store.ts
has some hard-to-test code for migrating values when updating the store cache. Find a way to extract the migration logic that's part ofPeprControllerStore.#migrateAndSetupWatch()
into an isolated function that can be maintained independently.Start this issue after #1259 is merged.
Links to any relevant code
Reference branch 1324-migration-refactor-reference for a starting point. Examine
storeCache.ts
andstoreCache.test.ts
to start out.Additional context
You'll need to examine the type definitions and manage differences between cache entries of type
Record<string, Operation>
andDataStore
entries which are defined asRecord<string, string>
.The text was updated successfully, but these errors were encountered: