Add Matrix API - Githubissues

radarlabs / radar-sdk-js

Web JavaScript SDK for Radar, the leading geofencing and location tracking platform

https://radar.com

Apache License 2.0

42 stars 11 forks source link

Add Matrix API #52

Closed samfader closed 2 years ago

samfader commented 2 years ago

This PR adds the Matrix API to the SDK. It also adds some notes to the README on how to spin up the SDK and run/test it locally. Finally, it adds a bit of UI to the index.html page to test out the Matrix capabilities.

cc @brettguenther

lgtm-com[bot] commented 2 years ago

This pull request introduces 3 alerts when merging fae98b1beb021fa866292606c7141f7c5fefa3cf into c9d6fa617c4456ec5916dc3299a67e8c709b91ea - view on LGTM.com

new alerts:

3 for Self assignment

lgtm-com[bot] commented 2 years ago

This pull request introduces 3 alerts when merging fae98b1beb021fa866292606c7141f7c5fefa3cf into 16ad0788acf442c005785abbba495950e952a52c - view on LGTM.com

new alerts:

3 for Self assignment

jsani-radar commented 2 years ago

as I mentioned in Slack, let's try bumping the circleci image to 12.22.7 to fix the build error

samfader commented 2 years ago

Thanks @jsani-radar, Circle tests passing now. Made some updates, and have one more question for you inline.

samfader commented 2 years ago

@corypisano @jsani-radar I believe this is ready for one more round of reviews. Thanks for all the assistance!