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

i18n - parte 2 #194

Merged
merged 2 commits into from
Nov 19, 2024
Merged

i18n - parte 2 #194

merged 2 commits into from
Nov 19, 2024

Conversation

fdodino
Copy link
Contributor

@fdodino fdodino commented Oct 18, 2024

Ahora que tenemos la función que puede internacionalizar mensajes en wollok-ts, completé los mensajes que muestra el IDE para

  • code lenses
  • mensajes de error que llegan a la UI

image

image

image

Que no (y está bien)

Decidí que el logger de eventos siga manteniéndose en inglés porque es más para nosotros:

image

Que no pude

Lamentablemente, hay dos lugares donde no estoy pudiendo customizar los mensajes

  • settings
  • command (lo que se muestra con F1)

image

image

porque se definen en el package.json. Si bien le puse i18n al name de cada task que creamos, el nombre no es lo que aparece en la UI sino el title. Busqué la documentación oficial de Command pero lamentablemente no dice nada y tampoco hay issues cargados. Voy a abrir un issue a ver qué me responden.

Lo mismo en los settings, las descripciones son las que definimos en el package.json.

@fdodino fdodino requested review from PalumboN and ivojawer October 18, 2024 10:55
@fdodino
Copy link
Contributor Author

fdodino commented Oct 18, 2024

Abrí este issue

Copy link
Contributor

@PalumboN PalumboN left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Había quedado esto colgado! 🗣️ 🇦🇷 🇺🇸 🇫🇷

@PalumboN PalumboN merged commit 70e71aa into master Nov 19, 2024
6 checks passed
@PalumboN PalumboN deleted the i18n-messages branch November 19, 2024 23:22
@fdodino
Copy link
Contributor Author

fdodino commented Nov 20, 2024

Había quedado esto colgado! 🗣️ 🇦🇷 🇺🇸 🇫🇷

C'est pas posible!! Nous avons oublié completement! Merci beaucoup!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants