eiabea / tempea-api

Thermostat control based on node.js
MIT License
1 stars 0 forks source link

Unified status object to match a certain pattern #27

Closed eiabea closed 5 years ago

eiabea commented 5 years ago

Refactored status route response:

{
  "success": true,
  "data": {
    "master": {
      "heating": true,
      "temp": 21
    },
    "slave": {
      "temp": 12.5,
      "hum": 32
    },
    "mqtt": {
      "updated": 1550586338221,
      "temp": 12.5
    },
    "desired": {
      "temp": 25.4,
      "master": 100,
      "slave": 0
    }
  }
}
codecov[bot] commented 5 years ago

Codecov Report

Merging #27 into master will not change coverage. The diff coverage is 100%.

Impacted file tree graph

@@          Coverage Diff          @@
##           master    #27   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files          10     10           
  Lines         375    376    +1     
=====================================
+ Hits          375    376    +1
Impacted Files Coverage Δ
controller/database.controller.js 100% <ø> (ø) :arrow_up:
routes/v1/status.route.js 100% <100%> (ø) :arrow_up:
controller/slave.controller.js 100% <100%> (ø) :arrow_up:
controller/calendar.controller.js 100% <100%> (ø) :arrow_up:
app.js 100% <100%> (ø) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 338d9f6...c0f63d8. Read the comment docs.