forked from google-home/smart-home-nodejs
-
Notifications
You must be signed in to change notification settings - Fork 1
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
update from source #1
Open
denwilliams
wants to merge
85
commits into
denwilliams:master
Choose a base branch
from
google-home:master
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Use cors module * Update smart-home-provider-cloud.js
Moves frontend to git submodule Rewrites server using Typescript Uses actions-on-google library Supports App Engine and Firestore Bug: 117293037 Bug: 110801702 Bug: 116674896 Change-Id: I4c49aadfa2ec38eaf11ecdd7728d0bf391c3ef36
ArmDisarm, LockUnlock, OpenClose, Timer Bug: 78479833 Bug: 78480036 Bug: 78480622 Bug: 131113124 Change-Id: Ibbb6fbeb06fcecc6a252575c5711a2cb0c3fddf0
Bug: 132893984 Bug: 132896002 Bug: 132896001 Change-Id: I0e2f363efa1d73ae34eeca977f042ce5e78b8b96
- fix typo - clarify gcloud project usage Change-Id: I8d05ffef6e9846f45b8ca338079f3f21cbf0a1b9
- remove ./node_modules prefix in scripts - add firebase-tools to dev deps and update README accordingly Change-Id: I0b0434db8f71c45ad14d286e4e8616cc82bd39cf
- otherDeviceIds Bug: 130215141 Change-Id: I37e50b22182465c9c6cd321cb914f9d15989bf0d
Bug: 134672432 Signed-off-by: Dave Smith <[email protected]> Change-Id: Ic999559a8082156794d18805df3cbc758b585deb
Bug: 137343009 Change-Id: I5db7f2af2d5181cc11504d7f295ed7ee355a54e8
Fix #426 Change-Id: I1bea4c877061545fd9e99b9afadbba8ad3a87e7d
- optionally call requestSync when localDeviceId is defined - disable reportState call when states is undefined Change-Id: Idf8882fda8455917b53e9f6bd43224d002cb174a
Fix issue that happens if device was unsuccessful Bug: 138810074 Change-Id: I061d74072bab1d53d2402d9f3360de8f0badbd63
Bug: 131229947 Change-Id: Ic6ebd90f59547a388464672185fcd217d3d700a8
Change-Id: Ia150f2035b88c1d0523c50fc30d9eb2b651d4415
Change-Id: I2ff9148782c9d0b38c68dddcb679e36bdc89cbb8
Bug: 140079274 Change-Id: If6b6060a1dcc86b51c4377206c7df28513a8809f
Fixes #481 Change-Id: I905eead74ecc130c03efc5ca895f6df35785a4ff
- move `status` property under each device object - add `errorCode` handling Bug: 146538213 Change-Id: I66ce682b73fd4088c5da31d3a4598af82c814011
Bug: 149561993 Change-Id: I49e18378bfdf6097bc366ff3b142236b201575f6
A change in the library fulfillment means `execution.params` may be undefined For each command one needs to explicitly say params is defined See actions-on-google/actions-on-google-nodejs@1c9c56b Change-Id: Ie4469e068c77c92325882ba58fdf1573fdd466c8
Bug: 142485101 Change-Id: Ibcc405ed6a15064b1f30b8089ab22edeb4744fd6
Bug: 152074367 Change-Id: I947caadb494662cab2f498daa357eeb73f2bcf2c
Bug: 140725857 Change-Id: Ie52c2d307e2e6b5e265129e89d47cc5de238920a
Bug: 150948152 Change-Id: I088e0c52d826d52166030657887ee30863168899
Bug: 145676543 Change-Id: I86dcbe41615e59d0cf2dcecdece63b9b94ee9879
- Update default unit for COOK trait - Update default project instructions for Firebase Bug: 166172744 Signed-off-by: Dave Smith <[email protected]> Change-Id: I1921801486bf6a80800fbd74daf132c64cd429a4
QUERY is now handled by the platform automatically following a SYNC to update Home Graph. Bug: 144428665 Signed-off-by: Dave Smith <[email protected]> Change-Id: I586eca1b80d5995d020bbb4879eb457d65a50734
add firebase link
Update README.md
- gts init/fix - remove tslint - fix no-case-declarations - fix no-unpublished-require - fix no-process-exit - cleanup @types dependencies - use esModuleInterop import syntax Change-Id: I6a176b83668232b3f68c24306458d5e44dcc4425
As it was only used for a long-time deprecated attributes transition. Change-Id: Ic827d6109c20e5e88c31f39e3d08f2fc2c6b3633
Change-Id: Ie1cd0bc75d1521da2805a4581714357b7d1973f2
- default export express app in './auth-provider' - move CRUD device managment to './device-manager' - move smarthome fulfillment to './fulfillment' Change-Id: I23ac63c4777d6a25068b73780a0b19002a697876
- remove ngrok support - remove appengine support - add support firebase cloud functions deployment - add support firebase hosting frontend deployment - add firebase hosting functions rewrite rules - use implicit firestore implicit configuration - remove firestore default indexes and security rules - simplify deployment instructions - removed obsolete dependencies - disable actions-on-google debug mode (messes with cloud functions logging) Change-Id: I708dd327801e6a720325c35560f3d0ade005e4cc
Bug: 175662079 Change-Id: Ic311f197f1f93881de90ce0759cf6bdaac810425
- use firebase structured logging - use logger.debug for request/request - simplify logging.debug for command execution - use logger.error for exception - localize error handling for homegraph calls - use appropriate http status code Bug: 173403262 Bug: 137852081 Change-Id: I48177dc2297b3a0bebab207488c7337f122cc4f9
Fixes #527 Change-Id: I06c53efbe3b729dc8495f9e9af5b8250b46820ed
Fixes #527 Change-Id: I8eceb67d9a1bcd560b6d62869134b6a666a428ed
Change-Id: I6acbe79bbd55a040d91e49b8acd59e58afb02d8f
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.