refactor(pfapi): remove unnecessary

Author: johannesjo

src/app/features/tasks/task.service.ts

@@ -939,7 +939,7 @@ export class TaskService {
   }
 
   async getArchivedTasks(): Promise<Task[]> {
-    const archiveTaskState: TaskArchive = await this._taskArchiveService.load(true);
+    const archiveTaskState: TaskArchive = await this._taskArchiveService.load();
     const ids = (archiveTaskState && (archiveTaskState.ids as string[])) || [];
     const archiveTasks = ids.map((id) => archiveTaskState.entities[id]) as Task[];
     return archiveTasks;

src/app/features/time-tracking/task-archive.service.ts

@@ -26,11 +26,11 @@ export class TaskArchiveService {
 
   constructor() {}
 
-  async load(isSkipMigration = false): Promise<TaskArchive> {
+  async load(): Promise<TaskArchive> {
     // NOTE: these are already saved in memory to speed up things
     const [archiveYoung, archiveOld] = await Promise.all([
-      this._pfapiService.m.archiveYoung.load(isSkipMigration),
-      this._pfapiService.m.archiveOld.load(isSkipMigration),
+      this._pfapiService.m.archiveYoung.load(),
+      this._pfapiService.m.archiveOld.load(),
     ]);
 
     return {

src/app/pfapi/api/model-ctrl/model-ctrl.ts

@@ -90,10 +90,9 @@ export class ModelCtrl<MT extends ModelBase> {
   // TODO implement isSkipMigration
   /**
    * Loads model data from memory cache or database
-   * @param isSkipMigration Whether to skip migration (if applicable)
    * @returns Promise resolving to model data
    */
-  async load(isSkipMigration?: boolean): Promise<MT> {
+  async load(): Promise<MT> {
     pfLog(3, `${ModelCtrl.name}.${this.load.name}()`, {
       inMemoryData: this._inMemoryData,
     });

src/app/pfapi/pfapi-config.ts

@@ -100,7 +100,7 @@ export const PFAPI_MODEL_CFGS: PfapiAllModelCfg = {
   },
 
   project: {
-    modelVersion: 1,
+    modelVersion: 1.2,
     defaultData: initialProjectState,
     isMainFileModel: true,
     validate: appDataValidators.project,