From c77e60e1688800a4f6b018cc6e9c1ce89033f108 Mon Sep 17 00:00:00 2001
From: Abir Majumdar <abirismyname@github.com>
Date: Thu, 9 May 2024 15:30:39 -0400
Subject: [PATCH 1/2] Add error handling for missing GitHub token and
 explicitly stating VUE_APP_MOCKED_DATA=true in the UI

---
 src/api/GitHubApi.ts             | 6 +++++-
 src/components/MainComponent.vue | 5 ++++-
 2 files changed, 9 insertions(+), 2 deletions(-)

diff --git a/src/api/GitHubApi.ts b/src/api/GitHubApi.ts
index 9d3c1aa1..d8ddb682 100644
--- a/src/api/GitHubApi.ts
+++ b/src/api/GitHubApi.ts
@@ -17,7 +17,7 @@ export const getMetricsApi = async (): Promise<Metrics[]> => {
   let metricsData;
 
   if (process.env.VUE_APP_MOCKED_DATA === "true") {
-    
+    console.log("Using mock data. Check VUE_APP_MOCKED_DATA variable.");
     if (process.env.VUE_APP_SCOPE === "organization") {
       response = organizationMockedResponse;
     } else if (process.env.VUE_APP_SCOPE === "enterprise") {
@@ -28,6 +28,10 @@ export const getMetricsApi = async (): Promise<Metrics[]> => {
 
     metricsData = response.map((item: any) => new Metrics(item));
   } else {
+    // if VUE_APP_GITHUB_TOKEN is not set, throw an error
+    if (!process.env.VUE_APP_GITHUB_TOKEN) {
+      throw new Error("VUE_APP_GITHUB_TOKEN environment variable is not set.");
+    }
     if (process.env.VUE_APP_SCOPE === "organization") {
       response = await axios.get(
         `https://api.github.com/orgs/${process.env.VUE_APP_GITHUB_ORG}/copilot/usage`,
diff --git a/src/components/MainComponent.vue b/src/components/MainComponent.vue
index ba1f935b..68661c48 100644
--- a/src/components/MainComponent.vue
+++ b/src/components/MainComponent.vue
@@ -6,7 +6,7 @@
       </v-btn>
 
       <v-toolbar-title>Copilot Metrics Viewer | {{ capitalizedItemName }} : {{ gitHubOrgName }}</v-toolbar-title>
-      <h2>  </h2>
+      <h2 class="error-message"> {{ mockedDataMessage }} </h2>
       <v-spacer></v-spacer>
 
       <template v-slot:extension>
@@ -73,6 +73,9 @@ export default defineComponent({
     },
     capitalizedItemName() {
       return this.itemName.charAt(0).toUpperCase() + this.itemName.slice(1);
+    },
+    mockedDataMessage() {
+      return process.env.VUE_APP_MOCKED_DATA === 'true' ? 'Using mock data. Check VUE_APP_MOCKED_DATA variable.' : '';
     }
   },
   data () {

From d085f23ac2faf60ce05ae101cba480d2377b8bed Mon Sep 17 00:00:00 2001
From: Marcio Tedesco <martedesco@github.com>
Date: Tue, 28 May 2024 10:04:28 +0200
Subject: [PATCH 2/2] Update src/components/MainComponent.vue

---
 src/components/MainComponent.vue | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/components/MainComponent.vue b/src/components/MainComponent.vue
index 68661c48..2f13f445 100644
--- a/src/components/MainComponent.vue
+++ b/src/components/MainComponent.vue
@@ -75,7 +75,7 @@ export default defineComponent({
       return this.itemName.charAt(0).toUpperCase() + this.itemName.slice(1);
     },
     mockedDataMessage() {
-      return process.env.VUE_APP_MOCKED_DATA === 'true' ? 'Using mock data. Check VUE_APP_MOCKED_DATA variable.' : '';
+      return process.env.VUE_APP_MOCKED_DATA === 'true' ? 'Using mock data - see README if unintended' : '';
     }
   },
   data () {