Skip to content

Conversation

@flauschzelle
Copy link
Collaborator

@flauschzelle flauschzelle commented Nov 24, 2022

Parsing of direct-messages-group.js and writing to one markdown file per group (splitting up very long ones into parts).

Image embedding (and copying, etc.) is still missing here, I will add that (probably) tomorrow.
Image embedding is now also implemented here (at the same level of completion as for the non-group DMs in #116).

@timhutton
Copy link
Owner

I don't have any group DMs to test this on, so I'm hoping you do.

There must be duplicated code between this and #116? I haven't gone over it in enough detail to confirm.

timhutton
timhutton previously approved these changes Nov 26, 2022
@flauschzelle
Copy link
Collaborator Author

I don't have any group DMs to test this on, so I'm hoping you do.

Yes, I have :) And I tested it on my own group DMs many times while building this. (Of course, I could always have overlooked some bug, but overall it seems to work fine.)

There must be duplicated code between this and #116? I haven't gone over it in enough detail to confirm.

The format of group DMs and 'single' DMs in the JSON is slightly different, but I'm sure there is some duplicate code in here, yes. If you think that would be useful, I could look into it and see if any parts could be extracted and used in both?

Copy link
Collaborator

@lenaschimmel lenaschimmel left a comment

Choose a reason for hiding this comment

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

Approving this as discussed yesterday.

@flauschzelle flauschzelle merged commit 3bb150c into timhutton:main Nov 26, 2022
flauschzelle added a commit that referenced this pull request Nov 27, 2022
Update README according to #116, #118 (DMs), #126 and #136 (CLI instructions)
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.

3 participants