Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Edge cases for article translation #3971

Open
1 of 3 tasks
robertu7 opened this issue Jun 4, 2024 · 1 comment
Open
1 of 3 tasks

Edge cases for article translation #3971

robertu7 opened this issue Jun 4, 2024 · 1 comment
Labels
bug Something isn't working

Comments

@robertu7
Copy link
Contributor

robertu7 commented Jun 4, 2024

Checklist

  • Strip @mention in Article.language resolver;
  • Make gcp.detectLanguage isolate its API quota from gcp.translate so that Article.language of new articles wont be affected if gcp.translate runs out of quota;
  • Throw error message or return nullish value if fails to translate;

Refs

@lizhineng
Copy link
Contributor

I forked the project yesterday and am still familiarizing myself with the repository. Please correct me if I'm wrong.

  • Does the bug indicate that we need to handle situations when the translation API runs out of quota?
  • Should we replace Google Translate with another provider due to pricing concerns, or are we supposed to support multiple translation providers with load balancing behind the scenes? This would help us avoid relying on a single translation API, reducing the risk of a single point of failure and improving availability.

Additionally, I noticed that the translation feature also translates @mention. I think we should leave it as is—what are your thoughts?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants