zulip
zulip copied to clipboard
zulip
WIP: Allow users to export their messages from the UI
@timabbott I pushed a few changes based on your initial feedback. I also implemented some tracking for a running export, but I'm not sure what the best way to handle failures during an export would be.
Also, like you said, the export part could definitely use some more tests and error handling. Any advice on that would be helpful too.
I merged the first commit, thanks @punchagan! Posting a next batch of comments shortly.
@timabbott I've finally been able to address the suggestion to update the status of the export for the user using an event. This PR is ready for another review.
Heads up @punchagan, we just merged some commits that conflict with the changes your made in this pull request! You can review this repository's recent commits to see where the conflicts occur. Please rebase your feature branch against the upstream/master branch and resolve your pull request's merge conflicts accordingly.
@punchagan It looks like Tim made several other suggestions/requests in his review - did you address those already?
It would be great to get this feature! :-)
Hey @brainwane thanks for the nudge/ping on this. This PR has become quite stale, and I will try to take a look at this again. (I seem to have some time on my hands owing to the pandemic situation).
Meanwhile, this PR also doesn't really implement the exact feature you requested - dump of all the messages you sent. This PR tries to create an export of all sent and received messages. I remember @timabbott mentioned on CZO that just getting the messages you sent would be easy to do using the API. Would it be useful to look into that and write a script that fetches this data, before I jump back into this PR? :thinking: