Amend getUser Function to give fetch request an authorisation header,…

… rather than a parameter (deprecated). Relates #35. Co-authored-by: HannahGooding <[email protected]>
fac19 · May 8, 2020 · 3a14723 · 3a14723
1 parent 0d906ac
commit 3a14723
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 13 deletions.
diff --git a/src/App.test.js b/src/App.test.js
diff --git a/src/components/header.js b/src/components/header.js
@@ -43,7 +43,7 @@ const Header = () => {
             <option value="Albadylic">Gregor</option>
             <option value="hannahgooding">Hannah</option>
             <option value="HettieM">Hettie</option>
-            <option value="Ivo-Evans">Ivo</option>w{" "}
+            <option value="Ivo-Evans">Ivo</option>
             <option value="itsina96">Ina</option>
             <option value="jackherizsmith">Jack</option>
             <option value="jamesj-0">James</option>

diff --git a/src/utils/getUser.js b/src/utils/getUser.js
@@ -9,9 +9,9 @@ const checkResponse = (response) => {
 };
 
 const getUser = (username) => {
-  return fetch(
-    `https://api.github.com/users/${username}?access_token=${accessToken}`
-  )
+  return fetch(`https://api.github.com/users/${username}?`, {
+    Authorization: `token ${accessToken}`,
+  })
     .then(checkResponse)
     .catch((error) => {
       throw new Error(`Fetch getUserData failed ${error}`);