Skip to content

Commit

Permalink
api fixes
Browse files Browse the repository at this point in the history
  • Loading branch information
markusressel committed Sep 28, 2024
1 parent c3f5d49 commit ca01e0d
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 7 deletions.
11 changes: 6 additions & 5 deletions keel_telegram_bot/client/api_client.py
Original file line number Diff line number Diff line change
Expand Up @@ -61,17 +61,16 @@ def get_tracked_images(self) -> List[TrackedImage]:
result = [TrackedImage.from_dict(tracked) for tracked in response]
return result

def get_tracked_image(self, identifier: str) -> TrackedImage or None:
def get_tracked_image(self, namespace: str, image: str) -> TrackedImage or None:
"""
Returns a list of all tracked images
"""
trecked_images = self.get_tracked_images()
for tracked_image in trecked_images:
if tracked_image.image == identifier:
if tracked_image.namespace == namespace and tracked_image.image == image:
return tracked_image
return None


def set_tracked(self, identifier: str, provider: Provider, trigger: Trigger,
schedule: PollSchedule or None) -> None:
"""
Expand Down Expand Up @@ -123,7 +122,8 @@ def set_schedule(self, identifier: str, schedule: PollSchedule) -> None:
:param schedule: the schedule of the image
"""
resource = self.get_resource(identifier)
self.set_tracked(identifier, resource.provider, resource.trigger, schedule)
tracked_image = self.get_tracked_image(resource.namespace, resource.name)
self.set_tracked(identifier, resource.provider, tracked_image.trigger, schedule)

def set_trigger(self, identifier: str, trigger: Trigger) -> None:
"""
Expand All @@ -132,7 +132,8 @@ def set_trigger(self, identifier: str, trigger: Trigger) -> None:
:param trigger: the trigger of the image
"""
resource = self.get_resource(identifier)
self.set_tracked(identifier, resource.provider, trigger, resource.schedule)
tracked_image = self.get_tracked_image(resource.namespace, resource.name)
self.set_tracked(identifier, resource.provider, trigger, tracked_image.poll_schedule)

def get_approvals(self, rejected: bool = None, archived: bool = None) -> List[Approval]:
"""
Expand Down
4 changes: 2 additions & 2 deletions keel_telegram_bot/client/tracked_image.py
Original file line number Diff line number Diff line change
@@ -1,13 +1,13 @@
from attr import dataclass

from keel_telegram_bot.client.types import Trigger, Policy
from keel_telegram_bot.client.types import Trigger, Policy, PollSchedule


@dataclass
class TrackedImage:
image: str
trigger: Trigger
poll_schedule: str
poll_schedule: PollSchedule
provider: str
namespace: str
policy: Policy
Expand Down

0 comments on commit ca01e0d

Please sign in to comment.