Skip to content

feat: add translations#60

Draft
MyuddinKhatri wants to merge 6 commits into
agritheory:mainfrom
MyuddinKhatri:add-translations
Draft

feat: add translations#60
MyuddinKhatri wants to merge 6 commits into
agritheory:mainfrom
MyuddinKhatri:add-translations

Conversation

@MyuddinKhatri

Copy link
Copy Markdown
Collaborator

closes #23

@Alchez

Alchez commented Jul 2, 2024

Copy link
Copy Markdown
Collaborator

@MyuddinKhatri also add in docs

@MyuddinKhatri
MyuddinKhatri marked this pull request as draft July 2, 2024 05:57
@MyuddinKhatri

Copy link
Copy Markdown
Collaborator Author

@MyuddinKhatri also add in docs

@Alchez Yes, I converted this to draft for now as it needs frappe environment in pre-commit context to run, Will have to look into it.

Comment thread test_utils/pre_commit/add_translations.py Outdated


def add_translations(lang, app):
untranslated_file = "untranslated_strings"

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

@Alchez Suggests wrapping this entire function in a try/except that allows it to continue in the pre-commit context

@agritheory

Copy link
Copy Markdown
Owner

@IshwaryaM1030 Please add the is_frappe_environment function to this, then it should be ready to go

@agritheory

Copy link
Copy Markdown
Owner

@IshwaryaM1030 Please rebase and mark ready when appropriate

@IshwaryaM1030

Copy link
Copy Markdown
Collaborator

@agritheory The changes related to translations are already covered in #118 , So we can close this PR. Once you confirm I will close it.

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.

Pre-commit to automatically add to translation file(s)

4 participants