diff options
| author | Henry Jameson <me@hjkos.com> | 2019-01-17 19:16:45 +0300 |
|---|---|---|
| committer | Henry Jameson <me@hjkos.com> | 2019-01-17 19:16:45 +0300 |
| commit | 5251de317dcd67d123109d64a0c39037267e1705 (patch) | |
| tree | 9b780b8239063558a46430a4735076af963a021e /src/components/conversation/conversation.js | |
| parent | d7bd294666cba08b6f6a8d447fbdf4cd59e66b2b (diff) | |
| parent | 387bf794ffbfb202fc426a578f174e7d6e3681d6 (diff) | |
Merge branch 'switch-to-string-ids' into favorites
with some changes/merge conflicts resolution
* switch-to-string-ids:
fixx?????
fix notifications?
fix lint
fix tests, removed one unused function, fix real problem that tests helped to surface
added some more explicit to string conversion since BE seem to be sending numbers and it could cause an issue.
Remove all explicit and implicit conversions of statusId to number, changed explicit ones so that they convert them to string
Diffstat (limited to 'src/components/conversation/conversation.js')
| -rw-r--r-- | src/components/conversation/conversation.js | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/components/conversation/conversation.js b/src/components/conversation/conversation.js index 7bad14a5..237de7e5 100644 --- a/src/components/conversation/conversation.js +++ b/src/components/conversation/conversation.js @@ -33,7 +33,7 @@ const conversation = { replies () { let i = 1 return reduce(this.conversation, (result, {id, in_reply_to_status_id}) => { - const irid = Number(in_reply_to_status_id) + const irid = String(in_reply_to_status_id) if (irid) { result[irid] = result[irid] || [] result[irid].push({ @@ -70,7 +70,7 @@ const conversation = { } }, getReplies (id) { - id = Number(id) + id = String(id) return this.replies[id] || [] }, focused (id) { @@ -81,7 +81,7 @@ const conversation = { } }, setHighlight (id) { - this.highlight = Number(id) + this.highlight = String(id) } } } |
