-
Notifications
You must be signed in to change notification settings - Fork 303
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
Replace stubs in device views #373
Replace stubs in device views #373
Conversation
} | ||
|
||
// Add root keys, if available | ||
if (Object.keys(root_keys) > 0) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if (Object.keys(root_keys) > 0) { | |
if (Object.keys(root_keys).length > 0) { |
list: stubs.devices.list, | ||
search: stubs.devices.search, | ||
get: stubs.device.get, | ||
list: ttnClient.Applications.Devices.getAll.bind(ttnClient.Applications.Devices), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
gives Uncaught TypeError
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please try building the sdk again and then do
make js.build-dll -B; make js.serve
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
issue persists
3234732
to
d0f48c5
Compare
Summary:
This PR replaces the stubs currently in place for displaying device info. There are some small marginally related fixes as well.
Changes:
n/a
)<DateTime />
component for displaying date/time valuesNotes for Reviewers:
feature/webui-single-device-page
, which was already reviewed via Single Device Overview Page (stubbed) #97feature/webui-single-device-page
to merge into master