Skip to content

Arreglar tap de la barra de traducción y cambiar proveedores a opciones sin API-key#9

Merged
Rocket-Space merged 1 commit intomasterfrom
codex/fix-translation-bar-functionality-and-providers
Feb 24, 2026
Merged

Arreglar tap de la barra de traducción y cambiar proveedores a opciones sin API-key#9
Rocket-Space merged 1 commit intomasterfrom
codex/fix-translation-bar-functionality-and-providers

Conversation

@Rocket-Space
Copy link
Owner

Motivation

  • Reparar el comportamiento del tap en la barra de traducción para que alterne traducir/mostrar original también cuando se usan proveedores externos de SpaceGram (no solo usuarios premium/autotraducción).
  • Unificar la lista de proveedores mostrada en el menú de la barra con la lista usada en los ajustes de SpaceGram para evitar desincronías.
  • Eliminar proveedores que requieren clave/API del selector activo y sustituirlos por opciones que no necesiten API (añadiendo alternativas chinas y rusas).

Description

  • Cambié la condición de la acción del botón de traducción en ChatActivity para permitir toggleTranslatingDialog si SpaceGramConfig.translateProvider != 0 además de las condiciones previas. (TMessagesProj/src/main/java/org/telegram/ui/ChatActivity.java)
  • Reemplacé la selección hardcodeada de proveedores en el menú de la barra por un bucle que lee SpaceGramTranslator.getAllProviderIds() / getAllProviderNames() y guarda la elección en SpaceGramConfig. (TMessagesProj/src/main/java/org/telegram/ui/Components/TranslateButton.java)
  • Modifiqué SpaceGramTranslator para exponer únicamente proveedores sin dependencia de API-key en las funciones getAllProviderNames() y getAllProviderIds(), reordenando/renombrando constantes y eliminando la activación por defecto de proveedores con key. (TMessagesProj/src/main/java/org/spacegram/translator/SpaceGramTranslator.java)
  • Añadí SimplyTranslateTranslator que consulta https://simplytranslate.org/api/translate y la registré para motores bing, baidu y yandex como proveedores sin key. (TMessagesProj/src/main/java/org/spacegram/translator/SimplyTranslateTranslator.java)
  • Cambios commiteados: actualicé SpaceGramTranslator.java, TranslateButton.java, ChatActivity.java y añadí SimplyTranslateTranslator.java.

Testing

  • Comprobaciones HTTP: ejecuté varios curl contra endpoints públicos para validar opciones sin key; https://api.mymemory.translated.net/get y https://translate.googleapis.com/... devolvieron traducciones válidas, y https://simplytranslate.org/api/translate respondió correctamente para motores google/baidu/yandex en pruebas locales (outputs mostraron texto traducido). (curl checks ejecutados en shell).
  • Se realizaron pruebas ad-hoc con llamadas a https://simplytranslate.org/api/translate?engine=... y se obtuvo translated_text en la respuesta JSON, lo que valida el parseo básico en SimplyTranslateTranslator.
  • Intenté compilar localmente con ./gradlew -q :TMessagesProj:compileDebugJavaWithJavac pero la build falló por incompatibilidad del entorno JDK/Gradle (Unsupported class file major version 69), por lo que no se pudo verificar la compilación completa en este entorno; fallo de entorno, no de los cambios de código.

Codex Task

@chatgpt-codex-connector
Copy link

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@Rocket-Space Rocket-Space merged commit 429dc5f into master Feb 24, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant