Skip to content

Commit

Permalink
Fixed bug from upstream update preventing publishing of location to MQTT
Browse files Browse the repository at this point in the history
  • Loading branch information
BigThunderSR authored and michaelwoods committed Feb 2, 2023
1 parent 6fb0960 commit f1c1676
Showing 1 changed file with 8 additions and 8 deletions.
16 changes: 8 additions & 8 deletions src/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -83,22 +83,22 @@ const configureMQTT = async (commands, client, mqttHA) => {
return;
}
const commandFn = cmd.bind(commands);
logger.info('Command sent', {command});
logger.info('Command sent', { command });
commandFn(options || {})
.then(data => {
// TODO refactor the response handling for commands
logger.info('Command completed', {command});
data = _.get(data, 'response.data');
if (data) {
logger.info('Command response data', {data});
logger.info('Command completed', { command });
const responseData = _.get(data, 'response.data');
if (responseData) {
logger.info('Command response data', { responseData });
const location = _.get(data, 'response.data.commandResponse.body.location');
if (location) {
const topic = mqttHA.getStateTopic({name: command});
const topic = mqttHA.getStateTopic({ name: command });
// TODO create device_tracker entity. MQTT device tracker doesn't support lat/lon and mqtt_json
// doesn't have discovery
client.publish(topic,
JSON.stringify({latitude: location.lat, longitude: location.long}), {retain: true})
.then(() => logger.info('Published location to topic.', {topic}));
JSON.stringify({ latitude: location.lat, longitude: location.long }), { retain: true })
.then(() => logger.info('Published location to topic.', { topic }));
}
}
})
Expand Down

0 comments on commit f1c1676

Please sign in to comment.