-
-
Notifications
You must be signed in to change notification settings - Fork 832
Conversation
limitations under the License. | ||
*/ | ||
|
||
'use strict'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These can go away now, they are implicit.
|
||
'use strict'; | ||
|
||
var React = require('react'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we stick to import
in new code?
); | ||
} | ||
|
||
var url = ContentRepo.getHttpUriForMxc( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can't really tell in what situations one is supposed to use client.mxcUrlToHttp
vs. ContentRepo.getHttpUriForMxc
but we're using them both interchangeably here which seems wrong.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
one's the raw download content, the other's a thumbnail. i think it's just how the API is.
src/TextForEvent.js
Outdated
@@ -17,6 +17,7 @@ limitations under the License. | |||
var MatrixClientPeg = require("./MatrixClientPeg"); | |||
var CallHandler = require("./CallHandler"); | |||
import { _t } from './languageHandler'; | |||
import sdk from './index'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unintentional?
applied changes; going to assume there was an implicit lgtm O:-) |
No description provided.