Merge branch 'main' into fix-docker-reference

Author: seveibar

.github/workflows/test.yml

@@ -23,22 +23,27 @@ on:
 jobs:
   test:
     runs-on: ubuntu-latest
+    permissions:
+      contents: read
+      packages: write
     env:
-      CONTAINER_REGISTRY: "registry.digitalocean.com"
+      CONTAINER_REGISTRY: "ghcr.io"
       RUN_URL: ${{ github.server_url }}/${{ github.repository }}/actions/runs/${{ github.run_id }}
     steps:
       - name: Checkout
         uses: actions/checkout@v2
         with:
           ref: ${{ github.event.inputs.sdkSha }}
-      - name: Login to DO container registry
-        uses: docker/login-action@v1
+      - name: Login to container registry
+        uses: docker/login-action@v2
         with:
           registry: ${{ env.CONTAINER_REGISTRY }}
-          username: ${{ secrets.DIGITAL_OCEAN_TOKEN }}
-          password: ${{ secrets.DIGITAL_OCEAN_TOKEN }}
+          username: ${{ github.actor }}
+          password: ${{ secrets.BOT_GH_TOKEN }}
       - name: Pre-pull Seam Connect image
-        run: docker pull ${{ env.CONTAINER_REGISTRY }}/seam/seam-connect:${{ github.event.inputs.connectSha || 'latest' }}
+        run: |
+          docker pull ${{ env.CONTAINER_REGISTRY }}/seamapi/seam-connect:${{ github.event.inputs.connectSha || 'latest' }}
+          docker tag ${{ env.CONTAINER_REGISTRY }}/seamapi/seam-connect:${{ github.event.inputs.connectSha || 'latest' }} seamapi/seam-connect
       - uses: actions/setup-python@v2
         with:
           python-version: 3.8

CHANGELOG.md

@@ -3,6 +3,20 @@
 All notable changes to this project will be documented in this file. See
 [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [1.3.0](https://github.com/seamapi/python/compare/v1.2.0...v1.3.0) (2022-09-19)
+
+
+### Features
+
+* Merge pull request [#42](https://github.com/seamapi/python/issues/42) from seamapi/implement-delete-connected-account ([c4b55b5](https://github.com/seamapi/python/commit/c4b55b5722774f6dc700679ecd03fbc12028eda0)), closes [#27](https://github.com/seamapi/python/issues/27)
+
+## [1.2.0](https://github.com/seamapi/python/compare/v1.1.5...v1.2.0) (2022-09-16)
+
+
+### Features
+
+* Merge pull request [#41](https://github.com/seamapi/python/issues/41) from seamapi/implement-device-update ([bd94e17](https://github.com/seamapi/python/commit/bd94e17bab7657d31e2f58049deb136b5d2a58a2)), closes [#29](https://github.com/seamapi/python/issues/29)
+
 ### [1.1.5](https://github.com/seamapi/python/compare/v1.1.4...v1.1.5) (2022-09-13)
 
 

pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "seamapi"
-version = "1.1.5"
+version = "1.3.0"
 description = "A Python Library for Seam's API https://getseam.com"
 authors = ["Severin Ibarluzea <[email protected]>"]
 license = "MIT"

seamapi/connected_accounts.py

@@ -110,3 +110,35 @@ def get(
             account_type=json_account["account_type"],
             errors=json_account.get("errors", []),
         )
+
+    def delete(
+        self,
+        connected_account: Union[ConnectedAccountId, ConnectedAccount],
+    ) -> bool:
+        """Deletes a connected account.
+
+        Parameters
+        ----------
+        connected_account : ConnectedAccountId or ConnectedAccount
+            Connected account id or ConnectedAccount to delete
+
+        Raises
+        ------
+        Exception
+            If the API request wasn't successful.
+
+        Returns
+            Boolean indicating if the connected account was deleted
+        """
+
+        connected_account_id = to_connected_account_id(connected_account)
+
+        res = requests.delete(
+            f"{self.seam.api_url}/connected_accounts/delete",
+            headers={"Authorization": f"Bearer {self.seam.api_key}"},
+            data={"connected_account_id": connected_account_id},
+        )
+        if not res.ok:
+            raise Exception(res.text)
+
+        return True

seamapi/devices.py

@@ -36,6 +36,8 @@ class Devices(AbstractDevices):
         Gets a list of devices
     get(device=None, name=None)
         Gets a device
+    update(device, name=None, properties=None, location=None)
+        Updates a device
     """
 
     seam: Seam
@@ -136,3 +138,53 @@ def get(
             raise Exception(res.text)
         json_device = res.json()["device"]
         return Device.from_dict(json_device)
+
+    def update(
+        self,
+        device: Union[DeviceId, Device],
+        name: Optional[str] = None,
+        properties: Optional[dict] = None,
+        location: Optional[dict] = None,
+    ) -> Device:
+        """Updates a device.
+
+        Parameters
+        ----------
+        device : DeviceId or Device
+            Device id or Device to update
+        name : str, optional
+            New device name
+        properties : dict, optional
+            New device properties
+        location : str, optional
+            New device location
+
+        Raises
+        ------
+        Exception
+            If the API request wasn't successful.
+
+        Returns
+        ------
+            Boolean
+        """
+
+        if not device:
+            raise Exception("device is required")
+
+        params = {
+            "device_id": to_device_id(device),
+            "name": name,
+            "properties": properties,
+            "location": location,
+        }
+        
+        res = requests.post(
+            f"{self.seam.api_url}/devices/update",
+            headers={"Authorization": f"Bearer {self.seam.api_key}"},
+            params=params,
+        )
+        if not res.ok:
+            raise Exception(res.text)
+
+        return True

tests/connected_accounts/test_connected_accounts.py

@@ -11,3 +11,6 @@ def test_connected_accounts(seam: Seam):
     connected_account_id = connected_accounts[0].connected_account_id
     connected_account = seam.connected_accounts.get(connected_account_id)
     assert connected_account.connected_account_id == connected_account_id
+
+    seam.connected_accounts.delete(connected_account)
+    assert len(seam.connected_accounts.list()) == 0

tests/devices/test_devices.py

@@ -27,3 +27,7 @@ def test_devices(seam: Seam):
     seam.locks.lock_door(device=(some_device))
     some_locked_lock = seam.locks.get(device=(some_device))
     assert some_locked_lock.properties.locked == True
+
+    seam.devices.update(device=(some_device), name="Updated lock")
+    some_updated_lock = seam.locks.get(device=(some_device))
+    assert some_updated_lock.properties.name == "Updated lock"