New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix me-box/databox#103 and me-box/databox#109 #4

Merged
merged 4 commits into from Sep 27, 2017

Conversation

Projects
None yet
2 participants
@ktg
Contributor

ktg commented Sep 27, 2017

No description provided.

@Toshbrown

This comment has been minimized.

Show comment
Hide comment
@Toshbrown

Toshbrown Sep 27, 2017

Contributor

When installing sensing kit in the desktop browser using the "mobile sensing data" link in the toolbar. It installs than I get this js error

Uncaught (in promise) ReferenceError: SensingKit is not defined
    at showSensingStart (index.js:249)
    at fetch.then.then (index.js:241)
    at <anonymous>
Contributor

Toshbrown commented Sep 27, 2017

When installing sensing kit in the desktop browser using the "mobile sensing data" link in the toolbar. It installs than I get this js error

Uncaught (in promise) ReferenceError: SensingKit is not defined
    at showSensingStart (index.js:249)
    at fetch.then.then (index.js:241)
    at <anonymous>
@Toshbrown

This comment has been minimized.

Show comment
Hide comment
@Toshbrown

Toshbrown Sep 27, 2017

Contributor

Also, if I try to install the light graph app after enabling the light sensor I get:

app-config.js:47 Uncaught (in promise) TypeError: src/templates/app-config.pug:104
    102| 	}
    103| 
  > 104| if hasUnfulfilledDatasources()
    105| 	div.mdc-layout-grid__cell.mdc-layout-grid__cell--span-12(style="margin-top:20px")
    106| 		div.padded(style="color:#d50000; background: #DDD")
    107| 			= 'Unable to install app. Cannot find any datasources for ' + unfulfilledDatasources() + '. You may need to install and configure a driver to provide them'

Cannot read property 'Symbol(Symbol.iterator)' of undefined
    at getMetadata (app-config.js:47)
    at hasSensors (app-config.js:34)
    at unfulfilledDatasources (app-config.js:18)
    at hasUnfulfilledDatasources (app-config.js:43)
    at app-config.js:113
    at appConfigTemplate (app-config.js:630)
    at appConfigDisplay (app-install.js:26)
    at listDatasources.then (app-install.js:98)
    at <anonymous>
Contributor

Toshbrown commented Sep 27, 2017

Also, if I try to install the light graph app after enabling the light sensor I get:

app-config.js:47 Uncaught (in promise) TypeError: src/templates/app-config.pug:104
    102| 	}
    103| 
  > 104| if hasUnfulfilledDatasources()
    105| 	div.mdc-layout-grid__cell.mdc-layout-grid__cell--span-12(style="margin-top:20px")
    106| 		div.padded(style="color:#d50000; background: #DDD")
    107| 			= 'Unable to install app. Cannot find any datasources for ' + unfulfilledDatasources() + '. You may need to install and configure a driver to provide them'

Cannot read property 'Symbol(Symbol.iterator)' of undefined
    at getMetadata (app-config.js:47)
    at hasSensors (app-config.js:34)
    at unfulfilledDatasources (app-config.js:18)
    at hasUnfulfilledDatasources (app-config.js:43)
    at app-config.js:113
    at appConfigTemplate (app-config.js:630)
    at appConfigDisplay (app-install.js:26)
    at listDatasources.then (app-install.js:98)
    at <anonymous>
@Toshbrown

This comment has been minimized.

Show comment
Hide comment
@Toshbrown

Toshbrown Sep 27, 2017

Contributor

trying to install the twitter sentiment app without the driver gives:

app-config.js:47 Uncaught (in promise) TypeError: src/templates/app-config.pug:104
    102| 	}
    103| 
  > 104| if hasUnfulfilledDatasources()
    105| 	div.mdc-layout-grid__cell.mdc-layout-grid__cell--span-12(style="margin-top:20px")
    106| 		div.padded(style="color:#d50000; background: #DDD")
    107| 			= 'Unable to install app. Cannot find any datasources for ' + unfulfilledDatasources() + '. You may need to install and configure a driver to provide them'

Cannot read property 'Symbol(Symbol.iterator)' of undefined
    at getMetadata (app-config.js:47)
    at hasSensors (app-config.js:34)
    at unfulfilledDatasources (app-config.js:18)
    at hasUnfulfilledDatasources (app-config.js:43)
    at app-config.js:113
    at appConfigTemplate (app-config.js:630)
    at appConfigDisplay (app-install.js:26)
    at listDatasources.then (app-install.js:98)
    at <anonymous>
getMetadata @ app-config.js:47
hasSensors @ app-config.js:34
unfulfilledDatasources @ app-config.js:18
hasUnfulfilledDatasources @ app-config.js:43
(anonymous) @ app-config.js:113
appConfigTemplate @ app-config.js:630
appConfigDisplay @ app-install.js:26
listDatasources.then @ app-install.js:98
Promise resolved (async)
listApps.then @ app-install.js:96
Promise resolved (async)
router.on @ app-install.js:93
(anonymous) @ navigo.min.js:1
v @ navigo.min.js:1
(anonymous) @ navigo.min.js:1
(anonymous) @ navigo.min.js:1
before @ index.js:411
v @ navigo.min.js:1
resolve @ navigo.min.js:1
_onLocationChange @ navigo.min.js:1
Contributor

Toshbrown commented Sep 27, 2017

trying to install the twitter sentiment app without the driver gives:

app-config.js:47 Uncaught (in promise) TypeError: src/templates/app-config.pug:104
    102| 	}
    103| 
  > 104| if hasUnfulfilledDatasources()
    105| 	div.mdc-layout-grid__cell.mdc-layout-grid__cell--span-12(style="margin-top:20px")
    106| 		div.padded(style="color:#d50000; background: #DDD")
    107| 			= 'Unable to install app. Cannot find any datasources for ' + unfulfilledDatasources() + '. You may need to install and configure a driver to provide them'

Cannot read property 'Symbol(Symbol.iterator)' of undefined
    at getMetadata (app-config.js:47)
    at hasSensors (app-config.js:34)
    at unfulfilledDatasources (app-config.js:18)
    at hasUnfulfilledDatasources (app-config.js:43)
    at app-config.js:113
    at appConfigTemplate (app-config.js:630)
    at appConfigDisplay (app-install.js:26)
    at listDatasources.then (app-install.js:98)
    at <anonymous>
getMetadata @ app-config.js:47
hasSensors @ app-config.js:34
unfulfilledDatasources @ app-config.js:18
hasUnfulfilledDatasources @ app-config.js:43
(anonymous) @ app-config.js:113
appConfigTemplate @ app-config.js:630
appConfigDisplay @ app-install.js:26
listDatasources.then @ app-install.js:98
Promise resolved (async)
listApps.then @ app-install.js:96
Promise resolved (async)
router.on @ app-install.js:93
(anonymous) @ navigo.min.js:1
v @ navigo.min.js:1
(anonymous) @ navigo.min.js:1
(anonymous) @ navigo.min.js:1
before @ index.js:411
v @ navigo.min.js:1
resolve @ navigo.min.js:1
_onLocationChange @ navigo.min.js:1
@ktg

This comment has been minimized.

Show comment
Hide comment
@ktg

ktg Sep 27, 2017

Contributor

The Mobile Data Sensing menu item shouldn't be visible on desktop. So, I've fixed that. I think I've also fixed the other error too.

Contributor

ktg commented Sep 27, 2017

The Mobile Data Sensing menu item shouldn't be visible on desktop. So, I've fixed that. I think I've also fixed the other error too.

@Toshbrown

This comment has been minimized.

Show comment
Hide comment
@Toshbrown

Toshbrown Sep 27, 2017

Contributor

Much better 👍

RTM

Contributor

Toshbrown commented Sep 27, 2017

Much better 👍

RTM

@Toshbrown Toshbrown merged commit 00fe6e7 into me-box:master Sep 27, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment