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

Translations and linting #58

Merged
merged 4 commits into from
Oct 10, 2023
Merged

Translations and linting #58

merged 4 commits into from
Oct 10, 2023

Conversation

Lugrim
Copy link
Contributor

@Lugrim Lugrim commented Oct 7, 2023

Added some missing translations for the user module

Will also merge #57 if merged

@Lugrim Lugrim requested a review from Lymkwi October 7, 2023 22:35
@Lugrim Lugrim changed the title Translations Translations and linting Oct 7, 2023
@Lugrim Lugrim force-pushed the translations branch 3 times, most recently from 639ac60 to 2153b9e Compare October 9, 2023 20:54
@Lugrim Lugrim requested review from ShiroUsagi-san and Lymkwi and removed request for Lymkwi and ShiroUsagi-san October 10, 2023 06:54
Copy link
Collaborator

@Lymkwi Lymkwi left a comment

Choose a reason for hiding this comment

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

Voir les commentaires et nos remarques IRL quand j'ai fait la revue

insalan/user/models.py Show resolved Hide resolved
insalan/user/tests.py Outdated Show resolved Hide resolved
insalan/user/views.py Outdated Show resolved Hide resolved
msgid "Demande de ré-initialisation de mot de passe"
msgstr "Ask for a password reset"

#: insalan/user/models.py:147
Copy link
Collaborator

Choose a reason for hiding this comment

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

For the record: this translation entry is weird but that's fine because that block of text will eventually become a document/email template

msgid "UUID invalide"
msgstr "Invalid UUID"

#: tickets/views.py:32
#: insalan/tickets/tests.py:93 insalan/tickets/views.py:32
msgid "Utilisateur⋅ice non trouvé⋅e"
Copy link
Collaborator

Choose a reason for hiding this comment

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

there are still some translation strings to fix as shown here but I guess we can do that later, that's fine

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yup, I limited myself to the user module, that's a bottleneck for our frontend folks

@Lugrim Lugrim merged commit 1e14105 into dev Oct 10, 2023
5 checks passed
@Lugrim Lugrim deleted the translations branch October 14, 2023 21:06
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