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

Translate src/vector #4119

Merged
merged 4 commits into from
May 31, 2017
Merged

Translate src/vector #4119

merged 4 commits into from
May 31, 2017

Conversation

kegsay
Copy link
Contributor

@kegsay kegsay commented May 31, 2017

Including src/vector/platform.

@kegsay kegsay assigned ara4n and richvdh and unassigned ara4n May 31, 2017
@kegsay kegsay requested a review from richvdh May 31, 2017 15:05
@kegsay
Copy link
Contributor Author

kegsay commented May 31, 2017

Integration tests failed when GH was down:

The command "scripts/fetch-develop.deps.sh" failed and exited with 1 during .

@@ -19,6 +19,7 @@ limitations under the License.

import VectorBasePlatform from './VectorBasePlatform';
import dis from 'matrix-react-sdk/lib/dispatcher';
import _t from 'matrix-react-sdk/lib/languageHandler';
Copy link
Member

Choose a reason for hiding this comment

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

Should it not be import { _t } from 'matrix-react-sdk/lib/languageHandler'; ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yep, typically I had only tested this on WebPlatform which was the only one to import correctly. Thanks.

Copy link
Member

Choose a reason for hiding this comment

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

np :)

@@ -18,6 +18,8 @@ limitations under the License.
*/

import BasePlatform from 'matrix-react-sdk/lib/BasePlatform';
import _t from 'matrix-react-sdk/lib/languageHandler';
Copy link
Member

Choose a reason for hiding this comment

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

same Q as above

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yep, typically I had only tested this on WebPlatform which was the only one to import correctly. Thanks.

@t3chguy
Copy link
Member

t3chguy commented May 31, 2017

Apart from Q, LGTM

@ara4n ara4n merged commit bac5ae8 into develop May 31, 2017
@t3chguy t3chguy deleted the kegan/translation-chimp branch May 12, 2022 08:58
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