Git Hash
+{{ gitHash }}
+Git Hash
-{{ gitHash }}
+Unknown
Git Hash
-Unknown
-Other Versions
-Click to view other versions
-License
-{{ manifest.license }}
-Instructions
-Click to view instructions
-Source Repository
-{{ manifest['upstream-repo'] }}
-Wrapper Repository
-{{ manifest['wrapper-repo'] }}
-Support Site
-{{ manifest['support-site'] || 'Not provided' }}
-Marketing Site
-{{ manifest['marketing-site'] || 'Not provided' }}
-Other Versions
+Click to view other versions
+License
+{{ manifest.license }}
+Instructions
+Click to view instructions
+Source Repository
+{{ manifest['upstream-repo'] }}
+Wrapper Repository
+{{ manifest['wrapper-repo'] }}
+Support Site
+{{ manifest['support-site'] || 'Not provided' }}
+Marketing Site
+{{ manifest['marketing-site'] || 'Not provided' }}
+
+ ![]()
+
- {{ pkg.manifest.title }}
+{{ pkg.manifest.version | displayEmver }}
+Version
+{{ manifest.version | displayEmver }}
+Git Hash
+{{ gitHash }}
+Version
-{{ manifest.version | displayEmver }}
-Git Hash
-{{ gitHash }}
+Unknown
Git Hash
-Unknown
-License
-{{ manifest.license }}
-Source Repository
-{{ manifest['upstream-repo'] }}
-Wrapper Repository
-{{ manifest['wrapper-repo'] }}
-Support Site
-{{ manifest['support-site'] || 'Not provided' }}
-License
+{{ manifest.license }}
+Source Repository
+{{ manifest['upstream-repo'] }}
+Wrapper Repository
+{{ manifest['wrapper-repo'] }}
+Support Site
+{{ manifest['support-site'] || 'Not provided' }}
+- {{ pkg.manifest.title }} -
-{{ pkg.manifest.version | displayEmver }}
-
+
+
-
-
- All services are up to date!
-{{ pkg.manifest.title }}
++ {{ pkg.manifest.title }} +
{{ local.manifest.version }}
+
+
@@ -62,11 +73,11 @@
>
- Update
+ {{ errors[pkg.manifest.id] ? 'Retry' : 'Update' }}
diff --git a/frontend/projects/ui/src/app/pages/updates/updates.page.scss b/frontend/projects/ui/src/app/pages/updates/updates.page.scss
index 04ecc4019..546be123f 100644
--- a/frontend/projects/ui/src/app/pages/updates/updates.page.scss
+++ b/frontend/projects/ui/src/app/pages/updates/updates.page.scss
@@ -1,6 +1,7 @@
.service-avatar {
position: absolute;
top: 6px;
+ cursor: pointer;
}
ion-label {
diff --git a/frontend/projects/ui/src/app/pages/updates/updates.page.ts b/frontend/projects/ui/src/app/pages/updates/updates.page.ts
index 3669cfc35..0804d44b2 100644
--- a/frontend/projects/ui/src/app/pages/updates/updates.page.ts
+++ b/frontend/projects/ui/src/app/pages/updates/updates.page.ts
@@ -14,10 +14,18 @@ import {
MarketplacePkg,
StoreIdentity,
} from '@start9labs/marketplace'
-import { Emver } from '@start9labs/shared'
+import { Emver, isEmptyObject } from '@start9labs/shared'
import { Pipe, PipeTransform } from '@angular/core'
import { combineLatest, Observable } from 'rxjs'
import { PrimaryRendering } from '../../services/pkg-status-rendering.service'
+import {
+ AlertController,
+ LoadingController,
+ NavController,
+} from '@ionic/angular'
+import { hasCurrentDeps } from 'src/app/util/has-deps'
+import { getAllPackages } from 'src/app/util/get-package-data'
+import { Breakages } from 'src/app/services/api/api.types'
interface UpdatesData {
hosts: StoreIdentity[]
@@ -33,6 +41,7 @@ interface UpdatesData {
})
export class UpdatesPage {
queued: Record = {}
+ errors: Record = {}
readonly data$: Observable = combineLatest({
hosts: this.marketplaceService.getKnownHosts$(),
@@ -49,11 +58,115 @@ export class UpdatesPage {
private readonly marketplaceService: MarketplaceService,
private readonly api: ApiService,
private readonly patch: PatchDB,
+ private readonly navCtrl: NavController,
+ private readonly loadingCtrl: LoadingController,
+ private readonly alertCtrl: AlertController,
) {}
- async update(id: string, url: string): Promise {
+ viewInMarketplace(pkg: PackageDataEntry) {
+ const url = pkg.installed?.['marketplace-url']
+ const queryParams = url ? { url } : {}
+
+ this.navCtrl.navigateForward([`marketplace/${pkg.manifest.id}`], {
+ queryParams,
+ })
+ }
+
+ async tryUpdate(
+ manifest: MarketplaceManifest,
+ url: string,
+ local: PackageDataEntry,
+ ): Promise {
+ const { id, version } = manifest
+
+ delete this.errors[id]
this.queued[id] = true
- this.api.installPackage({ id, 'marketplace-url': url })
+
+ if (hasCurrentDeps(local)) {
+ this.dryUpdate(manifest, url)
+ } else {
+ this.update(id, version, url)
+ }
+ }
+
+ private async dryUpdate(manifest: MarketplaceManifest, url: string) {
+ const loader = await this.loadingCtrl.create({
+ message: 'Checking dependent services...',
+ })
+ await loader.present()
+
+ const { id, version } = manifest
+
+ try {
+ const breakages = await this.api.dryUpdatePackage({
+ id,
+ version: `${version}`,
+ })
+ await loader.dismiss()
+
+ if (isEmptyObject(breakages)) {
+ this.update(id, version, url)
+ } else {
+ const proceed = await this.presentAlertBreakages(
+ manifest.title,
+ breakages,
+ )
+ if (proceed) {
+ this.update(id, version, url)
+ } else {
+ delete this.queued[id]
+ }
+ }
+ } catch (e: any) {
+ delete this.queued[id]
+ this.errors[id] = e.message
+ }
+ }
+
+ private async presentAlertBreakages(
+ title: string,
+ breakages: Breakages,
+ ): Promise {
+ let message: string = `As a result of updating ${title}, the following services will no longer work properly and may crash:
- `
+ const localPkgs = await getAllPackages(this.patch)
+ const bullets = Object.keys(breakages).map(id => {
+ const title = localPkgs[id].manifest.title
+ return `
- ${title} ` + }) + message = `${message}${bullets.join('')}