-
Notifications
You must be signed in to change notification settings - Fork 851
Open
Description
The request handler for receiving notifications does not process the "since" parameter:
redux-essentials-example-app/src/api/server.js
Lines 207 to 217 in 49fd0ea
rest.get('/fakeApi/notifications', (req, res, ctx) => { | |
const numNotifications = getRandomInt(1, 5) | |
let notifications = generateRandomNotifications( | |
undefined, | |
numNotifications, | |
db | |
) | |
return res(ctx.delay(ARTIFICIAL_DELAY_MS), ctx.json(notifications)) | |
}), |
Attention to line 211:
undefined, |
As a result, in Redux Essentials, Part 6 notifications with an incorrect creation time are received.
New notifications appear lower in the list:
Maybe I didn't understand something, but it seems to me that it should not be so.
I fixed it with the following code:
ashishmalg
Metadata
Metadata
Assignees
Labels
No labels