deepl-for-slack icon indicating copy to clipboard operation
deepl-for-slack copied to clipboard

Feature request: Automatic translation

Open cbaus opened this issue 2 years ago • 5 comments

Thank you for this great project and easy installation.

I was wondering if there is a way to automatically translate between two (pre-set) languages when the bot is included in a channel. We need a feature like the current Kiara slack bot offers so we don't have to click the flags as a reaction. It would be very helpful if you could add this.

cbaus avatar Jan 07 '22 07:01 cbaus

Hi @cbaus, thanks for writing in!

However, I'm not planning to add such a feature in the short term. If someone is interested in working on the enhancement or forking this project, I'm happy to be a code reviewer, though.

seratch avatar Jan 07 '22 09:01 seratch

OK I will work on it now and I can provide a pull request later. Please check the code though as Typescript is not really my favourite language.

In any case, DeepL currently doesn't allow to choose the target language based on the source language. I will therefore need to sometimes translate messages twice. That means translate by default to language 1, and if it was already in language 1, translate to language 2 after receiving the detected language tag. Long messages can be cut to save some API usage. I don't think there is another way unless DeepL changes this behaviour.

cbaus avatar Jan 11 '22 09:01 cbaus

Hey @cbaus, if you are fine to implement for your own use case, I would recommend forking this project or starting from scratch in your favorite language.

And then, once your folk becomes stable and you would like me to merge the changes in this repo, please reach out to me again. As you know, accepting more functionalities requires more maintenance efforts but I don't have the bandwidth for this project.

Also, many companies use this application in their workspaces. I need to avoid having any drastic and/or breaking changes in this repository. It'd be appreciated if you could understand this.

seratch avatar Jan 11 '22 09:01 seratch

That are not so many changes but feel free to decide yourself if you want to add it or not.

cbaus avatar Jan 11 '22 17:01 cbaus

Thanks for sharing your idea and working code!

I will keep this issue open for now but I'm not going to merge your PR at least in the short term. If we receive the same feature requests from more and more developers in the future, I may consider working on this.

seratch avatar Jan 21 '22 04:01 seratch