Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support per channel dump-from and dump-to limitation #283

Merged
merged 3 commits into from
Apr 12, 2024

Conversation

kolsys
Copy link
Contributor

@kolsys kolsys commented Apr 11, 2024

This patch allow to configure -dump-from and -dump-to limitation per channel.

How to use

As channel name you must provide channel ID and dates with format {CHANNEL_ID}|{OLDEST_TIME}|{LATEST_TIME}

To dump only specified channel where messages not older

slackdump 'C123|2024-04-07T23:02:12' 'C321|2024-01-01T23:02:12'

To dump only specified channels where message time between specified time

slackdump 'C123|2024-01-07T23:02:12|2024-04-07T23:02:12' 'C321|2024-01-01T23:02:12|2024-03-07T23:02:12'

To dump only specified channels where message time older than specified time

slackdump 'C123||2024-04-07T23:02:12' 'C321||2024-03-07T23:02:12'

To dump all available channels but channel C123 and C321 only from specified time

slackdump '^C123|2024-04-07T23:02:12' '^C321|2024-01-01T23:02:12'

@rusq rusq self-requested a review April 12, 2024 07:51
@rusq
Copy link
Owner

rusq commented Apr 12, 2024

Hey @kolsys thanks for your contribution! It looks good, the only thing is that one test needs updating - would you be able to update it?

Once merged, I'll port this to v3.

Copy link
Owner

@rusq rusq left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

see above comment

@kolsys
Copy link
Contributor Author

kolsys commented Apr 12, 2024

Hey @kolsys thanks for your contribution! It looks good, the only thing is that one test needs updating - would you be able to update it?

Fixed.

@rusq rusq merged commit 3f3cd76 into rusq:master Apr 12, 2024
3 checks passed
@rusq rusq added this to the v3.0.0 milestone Apr 12, 2024
@kolsys kolsys changed the title Add suppor per channel dump-from dump-to limitation Add support per channel dump-from dump-to limitation Apr 12, 2024
@kolsys kolsys changed the title Add support per channel dump-from dump-to limitation Add support per channel dump-from and dump-to limitation Apr 12, 2024
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.

None yet

2 participants