Skip to content

MCKIN-19254: fix js translations for answer block - #286

Merged
taniwha merged 1 commit into
open-craft:problem-builder-v3from
msaqib52:problem-builder-v3
Jun 17, 2020
Merged

MCKIN-19254: fix js translations for answer block#286
taniwha merged 1 commit into
open-craft:problem-builder-v3from
msaqib52:problem-builder-v3

Conversation

@msaqib52

Copy link
Copy Markdown
Contributor

Answer block is not loading js translations due to which following string is not getting translated, This PR includes js translations in answer block response.
pbt

@msaqib52

Copy link
Copy Markdown
Contributor Author

@xitij2000 please review this when you have sometime

@taniwha taniwha 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.

👍

  • I tested this: (describe what you tested)
  • 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.

@msaqib52

Copy link
Copy Markdown
Contributor Author

@taniwha thank you for the review, could you also merge it and create release tag?

@xitij2000

xitij2000 commented Jun 16, 2020

Copy link
Copy Markdown
Member

@msaqib52 Please also create an equivalent PR against the master branch. Unless a PR fixes a bug that is specific to v3 and not v4, all PRs should target both releases.

@msaqib52

Copy link
Copy Markdown
Contributor Author

@xitij2000 have created this PR for v4 #287

@taniwha

taniwha commented Jun 17, 2020

Copy link
Copy Markdown

@msaqib52 The merge and release tag (v3.4.19) for this PR were already done, but I'll do the master-PR too.

@taniwha

taniwha commented Jun 17, 2020

Copy link
Copy Markdown

or, I thought I had, I'll fix

@taniwha
taniwha merged commit 09442af into open-craft:problem-builder-v3 Jun 17, 2020
@taniwha

taniwha commented Jun 17, 2020

Copy link
Copy Markdown

@msaqib52 my appologies

@msaqib52

Copy link
Copy Markdown
Contributor Author

Thank you @taniwha

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.

3 participants