teslawhytho

joined 11 months ago
[–] [email protected] 1 points 11 months ago (1 children)

Please don’t take this as criticism, this is a great idea and I fully plan on contributing to the codebase.

With that said I spent a few hours trying to get it to work. No luck. Docker, no. Docker compose. No.

I took the code and built / run manually. That worked but then I couldn’t import a chat. I tested with one line with no attachments. From just that one line, here are the problems so far:

  • it doesn’t seem like WhatsApp has a standard way of exporting the text file. Your text file and my text file are different. In the US the format is [datetime] name msg. In your file it’s different and so it breaks the moment it hits the [.
  • unfortunately not accounting for locale. US stupidly uses mm/dd/yy. You have hardcoded the formatter for dd/mm/yy. Maybe you need to have a locale selection in the UI before import. Without that, no US messages are coming in.
  • it doesn’t account for 6/6/23. It’s expecting 06/06/2023. Again formatter and padding can fix that.
  • ui creates an entry in chats table for every attempt regardless of if a message was imported.
  • exceptions in other languages.
  • missing tests for the stuff above

Again none of that is supposed to be criticism. This is a great idea and I fully intend to help out with it.

Good job!