Skip to content

Conversation

@0xPb1
Copy link
Contributor

@0xPb1 0xPb1 commented Apr 9, 2025

No description provided.

@0xPb1
Copy link
Contributor Author

0xPb1 commented Apr 9, 2025

The issue has now been resolved and tested locally. It correctly skips scanning the channel_ids listed under the blacklisted_channel_ids section ->

# Filter blacklisted channels immediately
batch_channels = response.get("channels", [])
filtered_batch = [
    ch for ch in batch_channels 
    if ch['id'] not in blacklisted_channel_ids
]

However, the recent PR missed handling the scenario where no channel_ids are provided. the necessary check for this case was now included ->

if onlyArchived:
    archived_channels = True
if archived_channels:
    system.print_debug(args, f"Considering archived channels, you may want to set archived_channels to False")
else:
    system.print_debug(args, f"Skipping archived channels, you may want to set archived_channels to True")

@rohitcoder
Copy link
Owner

LGTM

@rohitcoder rohitcoder merged commit 3b74a84 into rohitcoder:main Apr 10, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants