-
Notifications
You must be signed in to change notification settings - Fork 236
Open
Description
Job listing in File delphi/scripts/delphi_cli.py is broken:
- if not filtering by status (i.e. select "ALL"), does not list anything, because of
polis/delphi/scripts/delphi_cli.py
Lines 225 to 239 in a0ced96
else: # Scan for all jobs and sort manually by created_at # THIS CODE WILL EVENTUALLY CRASH EVERYTHING # response = table.scan( # ConsistentRead=True, # Use consistent reads to immediately see new jobs # Limit=limit * 2 # Get more items since we'll sort and trim # ) # # Sort items by created_at in descending order # items = response.get('Items', []) # items.sort(key=lambda x: x.get('created_at', ''), reverse=True) # # Trim to requested limit # return items[:limit] return [] - if filtering by status, crashes like this:
Would be nice to fix it, but lesser priority right now than focusing on the comment routing.
So fielding it to remember and maybe circle back to it one day :) As well as explaining where to fix it.
Metadata
Metadata
Assignees
Labels
No labels