minor fixes: do not show link when github-url annotation is missing. …

…fix job page
nais · Nov 28, 2023 · 9cdcf53 · 9cdcf53
1 parent 28a09a1
commit 9cdcf53
Show file tree

Hide file tree

Showing 6 changed files with 25 additions and 17 deletions.
diff --git a/src/routes/team/[team]/[env]/app/[app]/Image.svelte b/src/routes/team/[team]/[env]/app/[app]/Image.svelte
@@ -89,9 +89,13 @@
 	{#if deployInfo?.timestamp === PendingValue}
 		<Skeleton variant="text" width="40%" />
 	{:else if deployInfo.timestamp !== null}
-		<a href={deployInfo.url}>Deployed</a>
+		{#if deployInfo.url === ''}
+			Deployed
+		{:else}
+			<a href={deployInfo.url}>Deployed</a>
+		{/if}
 		<Time time={deployInfo.timestamp} distance={true} />
-		{#if deployInfo.deployer && deployInfo.url}
+		{#if deployInfo.deployer !== ''}
 			by
 			<a href="https://github.com/{deployInfo.deployer}">{deployInfo.deployer}</a>.
 		{/if}

diff --git a/src/routes/team/[team]/[env]/job/[job]/+layout.svelte b/src/routes/team/[team]/[env]/job/[job]/+layout.svelte
@@ -76,11 +76,6 @@
 				</div>
 			{/if}
 		{/if}
-		{#if tab === 'Status' && (state === State.NOTNAIS || state === State.FAILING)}
-			<div class="notification">
-				<NotificationBadge color={'var(--a-border-action)'} size={'8px'} />
-			</div>
-		{/if}
 	{/each}
 </Tabs>
 <slot />

diff --git a/src/routes/team/[team]/[env]/job/[job]/+page.gql b/src/routes/team/[team]/[env]/job/[job]/+page.gql
@@ -1,4 +1,4 @@
-query Job($job: String!, $team: String!, $env: String!) {
+query Job($job: String!, $team: String!, $env: String!) @cache(policy: NetworkOnly) {
 	naisjob(name: $job, team: $team, env: $env) @loading(cascade: true) {
 		name
 		schedule

diff --git a/src/routes/team/[team]/[env]/job/[job]/+page.svelte b/src/routes/team/[team]/[env]/job/[job]/+page.svelte
@@ -32,13 +32,15 @@
 	<div class="grid">
 		<Status job={$Job.data.naisjob} />
 
-		<Card columns={4}>
-			<Cost app={jobName} {env} {team} />
-		</Card>
-		<Card columns={6}>
+		<Card columns={9} rows={1}>
 			<Image {job} />
 		</Card>
-		<Card columns={6}>
+
+		<Card columns={3} rows={1}>
+			<Cost app={jobName} {env} {team} />
+		</Card>
+
+		<Card columns={3}>
 			<h4>Schedule</h4>
 			{#if job.schedule === PendingValue}
 				<Skeleton variant="text" />

diff --git a/src/routes/team/[team]/[env]/job/[job]/Image.svelte b/src/routes/team/[team]/[env]/job/[job]/Image.svelte
@@ -47,11 +47,15 @@
 </h4>
 <p class="lastActivity">
 	{#if deployInfo?.timestamp === PendingValue}
-		<Skeleton variant="text" />
+		<Skeleton variant="text" width="40%" />
 	{:else if deployInfo.timestamp !== null}
-		<a href={deployInfo.url}>Deployed</a>
+		{#if deployInfo.url === ''}
+			Deployed
+		{:else}
+			<a href={deployInfo.url}>Deployed</a>
+		{/if}
 		<Time time={deployInfo.timestamp} distance={true} />
-		{#if deployInfo.deployer && deployInfo.url}
+		{#if deployInfo.deployer !== ''}
 			by
 			<a href="https://github.com/{deployInfo.deployer}">{deployInfo.deployer}</a>.
 		{/if}

diff --git a/src/routes/team/[team]/[env]/job/[job]/Status.svelte b/src/routes/team/[team]/[env]/job/[job]/Status.svelte
@@ -87,7 +87,10 @@
 	.card {
 		border-radius: 0.5rem;
 		padding: 1rem;
-		grid-column: span 2;
+		grid-column: span 3;
+		grid-row: span 1;
+		border: 1px solid var(--a-gray-200);
+		background-color: var(--a-bg-default);
 	}
 
 	.NAIS {