community icon indicating copy to clipboard operation
community copied to clipboard

Add Ecobee temperature sensors app

Open Vestride opened this issue 4 months ago • 3 comments

Description

This PR adds a new app, ecobeesensors, which displays the room temperature and occupancy for Ecobee thermostats and Smart Sensors.

ecobee_sensors


Is there a way to test oauth flows without deploying this app to production? My local version isn't working, and I think it's because the Ecobee developer dashboard requires all callback URLs to be https (ie I can't use http://localhost:8080/oauth-callback).

Copilot

copilot:all

Vestride avatar Feb 17 '24 22:02 Vestride

⚠️ The automated review process is experimental and likely has bugs. Please bear with us as we iron out the kinks and enable you to ship changes at high velocity 🚀

Next Steps

Hello! Thank you so much for your change 🤜 🤛 . There are a few things you need to do:

  • [ ] Sign the CLA if you haven't already
  • [ ] Ensure your build is green! Any problem will display a proposed solution to try out
  • [ ] Get a review, either by Tidbyt Bot or by a Tidbyt engineer

Manual Review Required

Hang tight! A Tidbyt engineer will be by shortly to review your change. Here is what they will be looking for:

Test Details
App Dir All files are in a single app directory
🟡 Modules Usage of cache.star requires review
Original Author The original author matches the PR author

Previews

apps/ecobeesensors/ecobee_sensors.star:

tidbyt[bot] avatar Feb 17 '24 22:02 tidbyt[bot]

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

tidbyt-bot avatar Feb 17 '24 22:02 tidbyt-bot

I have read the CLA Document and I hereby sign the CLA

Vestride avatar Feb 17 '24 22:02 Vestride

We're automatically closing this issue because it hasn't had any activity in 30 days. If that seems like a mistake, please feel free to re-open. Thanks!

stale[bot] avatar Mar 22 '24 20:03 stale[bot]

Please reopen and consider merging @matslina

Vestride avatar Mar 22 '24 21:03 Vestride