Skip to content

MCKIN-24633 translation issue for portuguese - #301

Merged
shimulch merged 1 commit into
open-craft:problem-builder-v3from
mudassir-hafeez:MCKIN-24633-v3
Sep 22, 2020
Merged

MCKIN-24633 translation issue for portuguese#301
shimulch merged 1 commit into
open-craft:problem-builder-v3from
mudassir-hafeez:MCKIN-24633-v3

Conversation

@mudassir-hafeez

@mudassir-hafeez mudassir-hafeez commented Sep 21, 2020

Copy link
Copy Markdown
Contributor

when we load the instructor tool for portuguese, the following error message shows:
Screenshot 2020-09-21 at 5 56 16 PM

Actually, segundos reference is not define and seems like a translation in Portuguese which was used mistakenly. so updating it into an accurate reference.

 Committer: Mudassir Hafeez <mudasir5761@gmail.com>
@mudassir-hafeez

Copy link
Copy Markdown
Contributor Author

@xitij2000 failed test does not seem to be related to this PR. Can you please have a look?
Thanks,

@shimulch

Copy link
Copy Markdown

@mudassir-hafeez, Thanks for the PR. Re-running circle-ci jobs seem to pass this time.

@mudassir-hafeez

Copy link
Copy Markdown
Contributor Author

@shimulch can you please review and create a tag against this if code changes seem good?
Same for the PR #302 against master.

@shimulch shimulch left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM 👍

  • I tested this: ran tests locally
  • I read through the code
    - [ ] I checked for accessibility issues
    - [ ] Includes documentation
    - [ ] I made sure any change in configuration variables is reflected in the corresponding client's configuration-secure repository.

@shimulch
shimulch merged commit 32972a2 into open-craft:problem-builder-v3 Sep 22, 2020
@shimulch

Copy link
Copy Markdown

Released v3.5.2

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