From 6eecfa4b88f3478c0a33c3ac34db112f809df170 Mon Sep 17 00:00:00 2001
From: Anbraten <anton@ju60.de>
Date: Sun, 9 Jan 2022 19:28:02 +0100
Subject: [PATCH] Show changed files of pipeline in UI (#650)

---
 web/src/components/tabs/Tab.vue               |  2 +-
 web/src/router.ts                             |  5 +++
 .../views/repo/build/BuildChangedFiles.vue    | 38 +++++++++++++++++++
 web/src/views/repo/build/BuildWrapper.vue     |  9 +++++
 4 files changed, 53 insertions(+), 1 deletion(-)
 create mode 100644 web/src/views/repo/build/BuildChangedFiles.vue

diff --git a/web/src/components/tabs/Tab.vue b/web/src/components/tabs/Tab.vue
index 28a2a4aa5..6c4f7905b 100644
--- a/web/src/components/tabs/Tab.vue
+++ b/web/src/components/tabs/Tab.vue
@@ -37,7 +37,7 @@ export default defineComponent({
 
     onMounted(() => {
       tab.value = {
-        id: props.title.toLocaleLowerCase() || tabs.value.length.toString(),
+        id: props.id || props.title.toLocaleLowerCase().replace(' ', '-') || tabs.value.length.toString(),
         title: props.title,
       };
       tabs.value.push(tab.value);
diff --git a/web/src/router.ts b/web/src/router.ts
index cf866b64c..381555599 100644
--- a/web/src/router.ts
+++ b/web/src/router.ts
@@ -58,6 +58,11 @@ const routes: RouteRecordRaw[] = [
             component: (): Component => import('~/views/repo/build/Build.vue'),
             props: true,
           },
+          {
+            path: 'changed-files',
+            name: 'repo-build-changed-files',
+            component: (): Component => import('~/views/repo/build/BuildChangedFiles.vue'),
+          },
           {
             path: 'config',
             name: 'repo-build-config',
diff --git a/web/src/views/repo/build/BuildChangedFiles.vue b/web/src/views/repo/build/BuildChangedFiles.vue
new file mode 100644
index 000000000..99cd89e4b
--- /dev/null
+++ b/web/src/views/repo/build/BuildChangedFiles.vue
@@ -0,0 +1,38 @@
+<template>
+  <FluidContainer v-if="build" class="flex flex-col gap-y-6 text-gray-500 justify-between py-0">
+    <Panel>
+      <div v-if="build.changed_files === undefined || build.changed_files.length < 1" class="w-full">
+        <span class="text-gray-500">No files have been changed.</span>
+      </div>
+      <div v-for="file in build.changed_files" v-else :key="file" class="w-full">
+        <div>- {{ file }}</div>
+      </div>
+    </Panel>
+  </FluidContainer>
+</template>
+
+<script lang="ts">
+import { defineComponent, inject, Ref } from 'vue';
+
+import FluidContainer from '~/components/layout/FluidContainer.vue';
+import Panel from '~/components/layout/Panel.vue';
+import { Build } from '~/lib/api/types';
+
+export default defineComponent({
+  name: 'BuildChangedFiles',
+
+  components: {
+    FluidContainer,
+    Panel,
+  },
+
+  setup() {
+    const build = inject<Ref<Build>>('build');
+    if (!build) {
+      throw new Error('Unexpected: "build" should be provided at this place');
+    }
+
+    return { build };
+  },
+});
+</script>
diff --git a/web/src/views/repo/build/BuildWrapper.vue b/web/src/views/repo/build/BuildWrapper.vue
index 4e808997e..dae71944f 100644
--- a/web/src/views/repo/build/BuildWrapper.vue
+++ b/web/src/views/repo/build/BuildWrapper.vue
@@ -32,6 +32,7 @@
         <Tabs v-model="activeTab" disable-hash-mode>
           <Tab id="tasks" title="Tasks" />
           <Tab id="config" title="Config" />
+          <Tab id="changed-files" :title="`Changed files (${build.changed_files?.length || 0})`" />
         </Tabs>
 
         <div class="flex justify-between gap-x-4 text-gray-500 flex-shrink-0 ml-auto">
@@ -193,6 +194,10 @@ export default defineComponent({
 
     const activeTab = computed({
       get() {
+        if (route.name === 'repo-build-changed-files') {
+          return 'changed-files';
+        }
+
         if (route.name === 'repo-build-config') {
           return 'config';
         }
@@ -204,6 +209,10 @@ export default defineComponent({
           router.replace({ name: 'repo-build' });
         }
 
+        if (tab === 'changed-files') {
+          router.replace({ name: 'repo-build-changed-files' });
+        }
+
         if (tab === 'config') {
           router.replace({ name: 'repo-build-config' });
         }