Translate Notifications

[ericszimmermann]

A change to translate Notification, if translation is enabled.

Implementation of Issue #366

Best Regards Eric

[zjs81]

Look good except requires some minor changes

[ericszimmermann]

Sorry didn't know that they need specific prompt. I tried and tested it, and it worked. It could be that it took a little longer to translate and that would explain it, but otherwise I got less german to german translations.

I also did the simple translation heuristics PR. But it is really simple and gives sometimes the wrong language. I tried other methods like n-gramms with weigthened scores, but the results were not better, which is why I published the simplest method. I think the best solution would be to use the model to recognize and return the language, but I can not find a prompt for that. Or someone has an other Idea that is working better, as the keyword approach.

Sincerely Eric

[zjs81]

You should use this https://pub.dev/documentation/flutter_langdetect/latest/

[ericszimmermann]

It is also a n-gramm approach, but with a lot more statistics behind it, than i could provide.
I will implement and try this. But should I make a separate Pull request or add a commit to this Pull-Request?

[zjs81]

I would say go ahead and do it in this PR. If we are going to use this lib we should just put it in our helper that way it can also be used else where. Thank you!

[ericszimmermann]

Sorry pushed a test, I didn't want to.
So I had to force push the branch back to the commit "implement flutter_langdetect"

[zjs81]

How well does it work so far?

[zjs81]

@codex review

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 75ae903b99

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[ericszimmermann]

The Language detection Library works a lot better, than the previous implementation. But through the small Texts like "Hallo", which could be FI as also DE, you still get false Positives. I don't think we can get rid of that completely.
I will have to look into codex‘es comment about executing twice and have found a problem with the replyInfo: the "@[...]" will be stripped in addMessage, but translated will be the whole string. This is good for the notification (for following the thread) but also makes a discrepancy between originalText and displayText in the MessageBubble.
I plan to modify the translated text, but sending the notification still with the replyInfo kept.