Skip to content
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

Toggling storage option is broken #339

Closed
tecimovic opened this issue Dec 2, 2021 · 2 comments
Closed

Toggling storage option is broken #339

tecimovic opened this issue Dec 2, 2021 · 2 comments

Comments

@tecimovic
Copy link
Collaborator

tecimovic commented Dec 2, 2021

  1. Go to any attribute.
  2. Toggle the "Storage option" dropdown.
  3. Observe:
⇝ (node:59858) UnhandledPromiseRejectionWarning: TypeError: Cannot read property 'map' of undefined
    at /Users/fesseha/Sources/connectedhomeipfork/third_party/zap/repo/src-electron/rest/user-data.js:189:26
    at Layer.handle [as handle_request] (/Users/fesseha/Sources/connectedhomeipfork/third_party/zap/repo/node_modules/express/lib/router/layer.js:95:5)
    at next (/Users/fesseha/Sources/connectedhomeipfork/third_party/zap/repo/node_modules/express/lib/router/route.js:137:13)
    at Route.dispatch (/Users/fesseha/Sources/connectedhomeipfork/third_party/zap/repo/node_modules/express/lib/router/route.js:112:3)
    at Layer.handle [as handle_request] (/Users/fesseha/Sources/connectedhomeipfork/third_party/zap/repo/node_modules/express/lib/router/layer.js:95:5)
    at /Users/fesseha/Sources/connectedhomeipfork/third_party/zap/repo/node_modules/express/lib/router/index.js:281:22
    at Function.process_params (/Users/fesseha/Sources/connectedhomeipfork/third_party/zap/repo/node_modules/express/lib/router/index.js:335:12)
    at next (/Users/fesseha/Sources/connectedhomeipfork/third_party/zap/repo/node_modules/express/lib/router/index.js:275:10)
    at /Users/fesseha/Sources/connectedhomeipfork/third_party/zap/repo/src-electron/server/http-server.js:212:11
⇝ (node:59858) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 52)
@jmartinez-silabs
Copy link
Member

Same happens for assigning a default value to an attribute.

You can write the desired value in the text box. But once you click elsewhere,it reverts back to 0

⇝ (node:87101) UnhandledPromiseRejectionWarning: TypeError: Cannot read property 'map' of undefined
    at /Users/GitHub/connectedhomeip/third_party/zap/repo/src-electron/rest/user-data.js:189:26
    at Layer.handle [as handle_request] (/Users/GitHub/connectedhomeip/third_party/zap/repo/node_modules/express/lib/router/layer.js:95:5)
    at next (/Users/GitHub/connectedhomeip/third_party/zap/repo/node_modules/express/lib/router/route.js:137:13)
    at Route.dispatch (/Users/GitHub/connectedhomeip/third_party/zap/repo/node_modules/express/lib/router/route.js:112:3)
    at Layer.handle [as handle_request] (/Users/GitHub/connectedhomeip/third_party/zap/repo/node_modules/express/lib/router/layer.js:95:5)
    at /Users/GitHub/connectedhomeip/third_party/zap/repo/node_modules/express/lib/router/index.js:281:22
    at Function.process_params (/Users/GitHub/connectedhomeip/third_party/zap/repo/node_modules/express/lib/router/index.js:335:12)
    at next (/Users/GitHub/connectedhomeip/third_party/zap/repo/node_modules/express/lib/router/index.js:275:10)
    at /Users/GitHub/connectedhomeip/third_party/zap/repo/src-electron/server/http-server.js:212:11
(node:87101) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 88)

@jonsmirl
Copy link

jonsmirl commented Dec 8, 2021

The XML needs a 'strorage' option so that the storage class can be properly defaulted.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants