Thomas' feedback

Author: Jon Barlow

README.md

@@ -1,12 +1,15 @@
 # sdk-oauth
 
-A sample application which uses the Settings API OAuth component, and the Fitbit
-Web API to query sleep data.
+A sample application which uses the Settings API [OAuth
+component](https://dev.fitbit.com/reference/settings-api/#oauth-button), and the
+[Fitbit Web API](https://dev.fitbit.com/reference/web-api/quickstart/) to query
+sleep data.
 
 ## Usage
 
 1. You must first register a Web Application on
-   [dev.fitbit.com](https://dev.fitbit.com/apps/new).
+   [dev.fitbit.com](https://dev.fitbit.com/apps/new) to get an OAuth ID and
+   secret. Configure the application as:
 
 - OAuth 2.0 Application Type: **Server**
 - Callback URL:
@@ -15,5 +18,10 @@ Web API to query sleep data.
 2. Enter your **OAuth 2.0 Client ID** and **Client Secret** into
    `settings/index.jsx`
 
+3. After installing the project from Fitbit Studio, you need to login to the
+   Fitbit Web API using the settings page within the Fitbit mobile application.
+
+   ***Fitbit mobile app > Ionic > Developer menu > your app > Settings***
+
 Read more in the [Reference
 documentation](https://dev.fitbit.com/reference/#overview).

companion/index.js

@@ -5,6 +5,7 @@ import { settingsStorage } from "settings";
 function fetchSleepData(accessToken)  {
   let todayDate = new Date().toISOString().slice(0,10); //YYYY-MM-DD
 
+  // Sleep API docs - https://dev.fitbit.com/reference/web-api/sleep/
   fetch(`https://api.fitbit.com/1.2/user/-/sleep/date/${todayDate}.json`, {
     method: "GET",
     headers: {
@@ -18,18 +19,13 @@ function fetchSleepData(accessToken)  {
     let myData = {
       totalMinutesAsleep: data.summary.totalMinutesAsleep
     }
-    sendVal(myData);
+    if (messaging.peerSocket.readyState === messaging.peerSocket.OPEN) {
+      messaging.peerSocket.send(myData);
+    }
   })
   .catch(err => console.log('[FETCH]: ' + err));
 }
 
-// Send data to the device
-function sendVal(data) {
-  if (messaging.peerSocket.readyState === messaging.peerSocket.OPEN) {
-    messaging.peerSocket.send(data);
-  }
-}
-
 // A user changes Settings
 settingsStorage.onchange = evt => {
   if (evt.key === "oauth") {

settings/index.jsx

@@ -12,7 +12,7 @@ function mySettings(props) {
           requestTokenUrl="https://api.fitbit.com/oauth2/token"
           clientId="XXXXXXXXXX"
           clientSecret="XXXXXXXXXX"
-          scope="profile sleep"
+          scope="sleep"
         />
       </Section>
     </Page>