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] /rest_filters updates #331

Merged

Conversation

thedoubl3j
Copy link
Member

Updates to rest_filters dir for translation. thanks to @AlanCoding for calling out during the work on this PR for making sure we don't translate when necessary (and also syntax checking me to make sure we are actually translating all that is needed).

Testing Steps

  1. pull down PR branch and start dev virtual env
  2. install django in virtenv if it is not already present (I believe it is but just in case)
    pip freeze | grep Django
    pip install Django
  3. make a locale directory (makemessages looks for one and we don't have one configured yet so just create your own) mkdir locale
  4. run the makemessages command check the output to see if strings were extracted correctly.
    django-admin makemessages -l es
    vim locale/es/LC_MESSAGES/django.po

@thedoubl3j
Copy link
Member Author

Technically a WIP, spoke more @AlanCoding and need to change a few more things (but everything here is good for review)

@john-westcott-iv john-westcott-iv added the comments left This PR was reviewed with comments label Apr 25, 2024
@john-westcott-iv john-westcott-iv added Ready To Merge and removed comments left This PR was reviewed with comments labels May 2, 2024
Copy link

sonarcloud bot commented May 2, 2024

@AlanCoding
Copy link
Member

Ignore the DAB Consumers failure, I've confirmed other PRs are seeing the same errors.

@john-westcott-iv john-westcott-iv merged commit c4596a5 into ansible:devel May 2, 2024
7 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants