Skip to content

Commit

Permalink
feat: expose sdkVersion on the controller node too (#4388)
Browse files Browse the repository at this point in the history
  • Loading branch information
AlCalzone authored Mar 18, 2022
1 parent 6e6b3f7 commit 5052d7d
Show file tree
Hide file tree
Showing 2 changed files with 15 additions and 4 deletions.
12 changes: 8 additions & 4 deletions packages/zwave-js/src/lib/commandclass/VersionCC.ts
Original file line number Diff line number Diff line change
Expand Up @@ -55,6 +55,13 @@ export function getSDKVersionValueId(): ValueID {
};
}

export function getSDKVersionMetadata(): ValueMetadata {
return {
...ValueMetadata.ReadOnlyString,
label: "SDK version",
};
}

export enum VersionCommand {
Get = 0x11,
Report = 0x12,
Expand Down Expand Up @@ -636,10 +643,7 @@ export class VersionCCZWaveSoftwareReport extends VersionCC {

private _sdkVersion: string;
@ccValue({ minVersion: 3 })
@ccValueMetadata({
...ValueMetadata.ReadOnlyString,
label: "SDK version",
})
@ccValueMetadata(getSDKVersionMetadata())
public get sdkVersion(): string {
return this._sdkVersion;
}
Expand Down
7 changes: 7 additions & 0 deletions packages/zwave-js/src/lib/controller/Controller.ts
Original file line number Diff line number Diff line change
Expand Up @@ -79,6 +79,8 @@ import {
import {
getFirmwareVersionsMetadata,
getFirmwareVersionsValueId,
getSDKVersionMetadata,
getSDKVersionValueId,
} from "../commandclass/VersionCC";
import type { Driver, RequestHandler } from "../driver/Driver";
import { cacheKeys, cacheKeyUtils } from "../driver/NetworkCache";
Expand Down Expand Up @@ -983,6 +985,11 @@ export class ZWaveController extends TypedEventEmitter<ControllerEventCallbacks>
controllerValueDB.setValue(getFirmwareVersionsValueId(), [
this._firmwareVersion,
]);
controllerValueDB.setMetadata(
getSDKVersionValueId(),
getSDKVersionMetadata(),
);
controllerValueDB.setValue(getSDKVersionValueId(), this._sdkVersion);

if (
this.type !== ZWaveLibraryTypes["Bridge Controller"] &&
Expand Down

0 comments on commit 5052d7d

Please sign in to comment.