diff options
| author | eugenijm <eugenijm@protonmail.com> | 2019-02-02 17:04:07 +0300 |
|---|---|---|
| committer | eugenijm <eugenijm@protonmail.com> | 2019-02-02 17:04:09 +0300 |
| commit | 0ab2f179918d6d1c37c80f73cab25e9865a6e796 (patch) | |
| tree | d5da19fd0d7bda10a5a31622bbc2832fa6baa90d | |
| parent | fbe7af3d56e4e6e168208a511bd7cd4e4e4e7a40 (diff) | |
Explicitly cast notification ids to integers when using them for setting maxId
This fixes the bug where BE receives incorrect `since_id` when marking
notifications as read.
| -rw-r--r-- | src/modules/statuses.js | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/src/modules/statuses.js b/src/modules/statuses.js index 56619455..b58f8404 100644 --- a/src/modules/statuses.js +++ b/src/modules/statuses.js @@ -273,10 +273,11 @@ const addNewNotifications = (state, { dispatch, notifications, older, visibleNot // Only add a new notification if we don't have one for the same action if (!state.notifications.idStore.hasOwnProperty(notification.id)) { - state.notifications.maxId = notification.id > state.notifications.maxId + const notificationId = parseInt(notification.id, 10) + state.notifications.maxId = notificationId > parseInt(state.notifications.maxId, 10) ? notification.id : state.notifications.maxId - state.notifications.minId = notification.id < state.notifications.minId + state.notifications.minId = notificationId < parseInt(state.notifications.minId, 10) ? notification.id : state.notifications.minId |