Skip to content

Revert "Prep to update the Tahoe languages" - #395

Merged
melvinsoft merged 1 commit into
appsembler/tahoe/masterfrom
revert-380-omar/update-langs
Jun 7, 2019
Merged

Revert "Prep to update the Tahoe languages"#395
melvinsoft merged 1 commit into
appsembler/tahoe/masterfrom
revert-380-omar/update-langs

Conversation

@melvinsoft

Copy link
Copy Markdown

Reverts #380

@OmarIthawi The previous PR broke Studio due to a version conflict, I'm reverting it until you get back and we can see it with more detail.

@melvinsoft
melvinsoft requested a review from thraxil June 7, 2019 12:57

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

from the exception, we could probably get away with just backing off edx-i18n-tools. but let's fix it first and investigate later.

@melvinsoft

Copy link
Copy Markdown
Author

Agreed, thanks @thraxil

@melvinsoft
melvinsoft merged commit 6fb98f2 into appsembler/tahoe/master Jun 7, 2019
@OmarIthawi

Copy link
Copy Markdown

😒 If you keep edx-i18n-tools, it's broken. If you fix it, it's also broken. I have to find a better creative way! Thanks @melvinsoft for fixing the Studio.

@melvinsoft

Copy link
Copy Markdown
Author

@OmarIthawi Agh, sorry to hear that. Is this a requirements in order to upgrade our translations?

@OmarIthawi

Copy link
Copy Markdown

Yes @melvinsoft, otherwise it's almost impossible to update the translations. I've addressed it in a better way on #393

@OmarIthawi
OmarIthawi deleted the revert-380-omar/update-langs branch May 6, 2020 22:28
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.

4 participants