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

Traductions manquantes en anglais #34

Closed
Ckrapock opened this issue Feb 16, 2023 · 10 comments
Closed

Traductions manquantes en anglais #34

Ckrapock opened this issue Feb 16, 2023 · 10 comments
Labels
good first issue Good for newcomers

Comments

@Ckrapock
Copy link
Contributor

Ckrapock commented Feb 16, 2023

1 - Page playlist ->PR #44
Import Playlist
Export
image

2 - Page principale -> Problème isolé dans Issue #36
Musical Genre
Decades
On Screen

3 - Page principale bas de page -> PR #44
Terms and Conditions
Privacy Policy
Donate

image

4 - Page "Mon compte" -> PR #44
"Information" ne prend pas de "s" en anglais
image

5 - Popup Suggestions -> PR #44
No need to send suggestions separately. Gather them in a single suggestion, to give better readability for moderators.
image

6 - Langue -> PR #44
Connecté en anglais, au survol de la langue, le libellé "Langue" apparaît en français

7 - Recherche dans la FAQ -> PR #44
Search in FAQ
image

8 - Popup room avec mot de passe -> PR #44
This room is secured by a password.
If you are the owner of the room and have forgotten the password, you can change it on the room's editing page.
image

9 - Phrase quand pas de room trouvée -> PR #44 (celle-ci, j'ai rajouté la trad dans le fichier fr.json, mais non Index.vue car la syntaxe posait probleme - à voir pour après)
If you don't have a room on Blinest by the time you turn 50, you will have missed out on life.
Anonymous

image

10 - Prochain extrait -> PR #44
Next track in

image

11 - **Prochaine partie ** -> PR #44
Next game in

12 - Messages rigolos de réussite ou echec -> PR #44

13 - fichier validation-attributes manquant -> PR #44

14 - Messages erreurs et succès trouvés dans le code -> PR #44

15 - Copier le lien dans teams et message pour rejoindre team -> PR #44

16 - "Audios" rendu traduisible -> PR #44
Laissé tel quel en FR/EN, mais au cas où on veuille le changer dans certaines langues.

17 - Suggestions et différentes valeurs d'options rendue internationalisables -> PR #44

18 - Internationalisation de libellés survolés : PTS et Players -> PR #44

19 - **Internationalisation de libellés : Contact, Lien sponsorisé, Top 50 **PR #44

@Ckrapock Ckrapock changed the title Traducttions manquantes en anglais Traductions manquantes en anglais Feb 16, 2023
@mchev
Copy link
Owner

mchev commented Feb 16, 2023

Merci @Ckrapock pour ces nombreux retours.
Il n'est pas forcement nécessaire d'envoyer des captures d’écrans pour chaque exemple. Le mieux est de pointer le fichier qui pose problème.
Comme pour la traduction #33, une Pull Request est la solution la plus adaptée quand c'est possible.
Je réponds aux autres dès que j'ai un peu de temps.

@Ckrapock
Copy link
Contributor Author

De rien, @mchev et zéro pression je remonte les trucs mais rien d'urgent!
J'aimerais pouvoir faire plus de Pull Requests, mais mon level est trop bidon.
Si j'ai le temps d'essaierai d'en faire pour celle-ci aussi, mais je garantie absolument rien, surtout quand le libellé n'est pas encore internationalisé je sais pas trop quoi faire :)

@Ckrapock
Copy link
Contributor Author

Ckrapock commented Feb 17, 2023

J'ai ouvert une pull request pour tout les points, sauf le point 2 que j'ai isolé dans une nouvelle fiche, car je pense que définitivement je saurai pas faire !

@mchev
Copy link
Owner

mchev commented Feb 19, 2023

Idéalement il faudrait faire une PR par issue. De tout mélanger dans une seule PR pose des problème de compréhension et ne permet pas de revenir en arrière juste sur un point si l'on découvre un problème. J'ai regardé rapidement tes modifications et il y a de nombreux endroits ou il va y avoir des erreurs dans les ajouts de traductions sur les fichiers .vue.

@Ckrapock
Copy link
Contributor Author

Ckrapock commented Feb 19, 2023

Ah d'accord. Pas de probleme. Le probleme c'est que je suis un peu à l'aveugle pour ça car je ne sais pas coment démarrer l'appli. JE n'utilise que codespace.
Peux-tu me donner une marche à suivre simple pour mettre tout dans 1 PR J'avais pensé à refaire une branche clean à partir de la mienne, et ensuite gérer 1 commit.
Et sinon pour les tests, si tu as une procédure simple pour pouvoir démarrer l'app sur ma branche je suis prenaur aussi :)

@mchev mchev added the good first issue Good for newcomers label Feb 19, 2023
@mchev
Copy link
Owner

mchev commented Feb 19, 2023

Ah d'accord. Pas de probleme. Le probleme c'est que je suis un peu à l'aveugle pour ça car je ne sais pas coment démarrer l'appli. JE n'utilise que codespace. Peux-tu me donner une marche à suivre simple pour mettre tout dans 1 PR J'avais pensé à refaire une branche clean à partir de la mienne, et ensuite gérer 1 commit. Et sinon pour les tests, si tu as une procédure simple pour pouvoir démarrer l'app sur ma branche je suis prenaur aussi :)

Pour lancer l'app en développement les instructions sont dans le README.

Concernant la création de PR, voici un tuto rapide :
https://opensource.com/article/19/7/create-pull-request-github

En gros pour chaque issue, créer une nouvelle branche locale qui repart à chaque fois de la base pour avoir une PR avec juste les modifications concernées.

@Ckrapock
Copy link
Contributor Author

J'avais essayé de démarrer en suivant le readme mais pas réussi. Pas grave on verra après. Au pire si toi tu sais la démarrer et me filer un lien externe pour faires les vérifs de libellé peut etre que ça suffira
Dans un premier temps, je vais essayer de créer une nouvelle branche, virer tous les trucs relatif au PT et DE, et on verra à partir de là.

@Ckrapock
Copy link
Contributor Author

Ckrapock commented Feb 21, 2023

Nouvelle PR : #44
Uniquement les points de cette fiche, en ayant viré les trucs pour PT et DE.
(Je ne sais toujours pas comment vérifier car pas réussi à démarrer l'app dans codespace... MAIS à un moment pendant mes essais j'ai vu 1 ou 2 trucs qui empechaient de démarrer que j'ai "corrigé" par exemple le point 9)

@mchev
Copy link
Owner

mchev commented Feb 26, 2023

Fixed by #45, Thanks @Ckrapock

@mchev mchev closed this as completed Feb 26, 2023
@Ckrapock
Copy link
Contributor Author

Victory!!! (and thanks for additional fixes behind it @mchev !)

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

No branches or pull requests

2 participants