maverick/flex-template-web
1
0
Fork 0
mirror of https://github.com/kingomarnajjar/flex-template-web.git synced 2026-07-26 06:47:17 +10:00
Code Issues Projects Releases Packages Wiki Activity Actions

Refetch first page if totalItems has changed between pagination calls.

Browse source
This commit is contained in:
Vesa Luusua 2017-12-05 14:45:25 +02:00
parent 01eff1ef3e
commit cf7497d820
2 changed files with 34 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

17
src/containers/OrderPage/OrderPage.duck.js
View file

@ -205,8 +205,25 @@ const fetchMessages = (txId, page) => (dispatch, getState, sdk) => {
const denormalizedMessages = denormalisedEntities(entities, 'message', messageIds);
const { totalItems, totalPages, page: fetchedPage } = response.data.meta;
const pagination = { totalItems, totalPages, page: fetchedPage };
const totalMessages = getState().OrderPage.totalMessages;
// Original fetchMessages call succeeded
dispatch(fetchMessagesSuccess(denormalizedMessages, pagination));
// Check if totalItems has changed between fetched pagination pages
// if totalItems has changed, fetch first page again to include new incoming messages.
// TODO if there're more than 100 incoming messages,
// this should loop through most recent pages instead of fetching just the first one.
if (totalItems > totalMessages && page > 1) {
dispatch(fetchMessages(txId, 1))
.then(() => {
// Original fetch was enough as a response for user action,
// this just includes new incoming messages
})
.catch(() => {
// Background update, no need to to do anything atm.
});
}
})
.catch(e => {
dispatch(fetchMessagesError(storableError(e)));

17
src/containers/SalePage/SalePage.duck.js
View file

@ -283,8 +283,25 @@ const fetchMessages = (txId, page) => (dispatch, getState, sdk) => {
const denormalizedMessages = denormalisedEntities(entities, 'message', messageIds);
const { totalItems, totalPages, page: fetchedPage } = response.data.meta;
const pagination = { totalItems, totalPages, page: fetchedPage };
const totalMessages = getState().OrderPage.totalMessages;
// Original fetchMessages call succeeded
dispatch(fetchMessagesSuccess(denormalizedMessages, pagination));
// Check if totalItems has changed between fetched pagination pages
// if totalItems has changed, fetch first page again to include new incoming messages.
// TODO if there're more than 100 incoming messages,
// this should loop through most recent pages instead of fetching just the first one.
if (totalItems > totalMessages && page > 1) {
dispatch(fetchMessages(txId, 1))
.then(() => {
// Original fetch was enough as a response for user action,
// this just includes new incoming messages
})
.catch(() => {
// Background update, no need to to do anything atm.
});
}
})
.catch(e => {
dispatch(fetchMessagesError(storableError(e)));