search

verdigado / rocketchat2matrix

Script to migrate Rocket.Chat databases to Matrix Synapse. Mirror of https://git.verdigado.com/NB-Public/rocketchat2matrix
GNU Affero General Public License v3.0
14 stars 6 forks source link

Support for attachments #9

Open chagai95 opened 4 months ago

chagai95 commented 4 months ago

If I understand correctly attachments are not supported, would be nice to support them, what does that entail?

grvn-ht commented 3 months ago

Hey @chagai95 , I also noticed it and wanted to work on it, @HerHde did someone already start something? ++

chagai95 commented 3 months ago

We at fairkom didn't really start, we only tried understanding the problem, if you want we can show you what we learned but we're still too confused to say something clear. Feel free to reach out maybe on matrix? @me:chagai.website

HerHde commented 3 months ago

I don't know of anyone working on this, either. Great to hear that you want to tackle it together :+1:

pierreozoux commented 3 months ago

I'm working on it right now. (but we have only object store, so I'll support only this case)

grvn-ht commented 3 months ago

I started citation messages @pierreozoux are images also in object store?

pierreozoux commented 3 months ago

@grvn-ht yes they are.

FYI: I'm get inspiration from: https://github.com/Worteks/RC2Matrix/blob/main/rc2matrix.py#L441-L557

pierreozoux commented 3 months ago

preliminary work is available here (sorry, no ts expert here :/ ): https://github.com/pierreozoux/rocketchat2matrix/tree/file-upload

Are you currently working on citations @grvn-ht ?

grvn-ht commented 3 months ago

I also did some work here: https://github.com/grvn-ht/rocketchat2matrix/tree/attachement-reply I would also like to mark all messages as read (otherwise when people get to their new account they will push button "mark all as read" and it makes synapse server very slow, lots of timeout ... then it is unusable for some time Does someone has idea on it? https://github.com/verdigado/rocketchat2matrix/issues/11

chagai95 commented 3 months ago

@pierreozoux hello, thx for making us aware of this great python script! May I ask why did you decide to not just use the python script directly, we are also no ts experts and we were wondering why not use the python script...

pierreozoux commented 3 months ago

@chagai95 this is a bit out of the subject of this issue, but the structure in different files and functions here is more appealing, and also the usage of an sqlite database to restart migration are 2 really nice things! (and there are tests here :) )