grundleborg / slack-advanced-exporter

A tool for exporting additional data from Slack that is missing from the official data export.
MIT License
97 stars 19 forks source link

Fix files for the new Slack export format #10

Closed kongr45gpen closed 5 years ago

kongr45gpen commented 6 years ago

This PR adds support for the files directive in exported Slack posts (fix for #9). The older file option is still supported.

Do note that Mattermost does not seem to parse files correctly yet, so a bulk import of files may still fail.

Binary files

For easier reference, these are the compiled binaries of the code from the PR that fixes the issue: slack-advanced-exporter-linux64.tar.gz
slack-advanced-exporter-win64.tar.gz

thomas-bouvier commented 6 years ago

Works for me. Should be merged!

matosconsulting commented 5 years ago

Another vote for completing this PR - just used it today to export files from Slack.

dns13 commented 5 years ago

Worked for me, too.

svelle commented 5 years ago

FYI: Here's the PR to support the new Slack export format on the server side as well: https://github.com/mattermost/mattermost-server/pull/11415

grundleborg commented 5 years ago

Sorry for letting this set for so long! Looks great - thanks for the PR and the confirmations that this works from everyone. I'll get it merged and make a new release shortly.