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

fix: make app's server messages translatable #12479

Closed
wants to merge 2 commits into from
Closed

fix: make app's server messages translatable #12479

wants to merge 2 commits into from

Conversation

szufisher
Copy link
Contributor

Please provide enough information so that others can review your pull request:

Explain the details for making this change. What existing problem does the pull request solve?

Screenshots/GIFs

Fix the error that attached file in comment with Chinese name does not work issue

refer to issue #12421

@szufisher szufisher requested review from a team and surajshetty3416 and removed request for a team February 24, 2021 14:45
@szufisher
Copy link
Contributor Author

@netchampfaris
Any further review findings? how to proceed?

Many thanks.

Fisher

@szufisher
Copy link
Contributor Author

Any further review findings? how to proceed?

@netchampfaris
Copy link
Contributor

@szufisher Your change is breaking get_dict

image

@szufisher
Copy link
Contributor Author

@szufisher Your change is breaking get_dict

image

seems the error you posted not generated from this PR's travis log, for me in my local development instance the same code runs just smoothly as expected.

@surajshetty3416 surajshetty3416 removed their request for review March 17, 2021 02:42
@netchampfaris
Copy link
Contributor

netchampfaris commented Mar 22, 2021

@szufisher Just tried it again. It breaks.

Try running these commands step by step.

  1. Stop dev server if already running to reset redis cache.
  2. bench start
  3. bench --site sitename console
  4. from frappe.translate import get_dict
    get_dict('boot')

image

@surajshetty3416
Copy link
Member

Closing due to inactivity. @szufisher Please re-open once the issue is fixed.

@thebachy1
Copy link
Contributor

Any updates?

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 1, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants