search

mkozhukharenko / ums-frontend

ums-frontend
MIT License
1 stars 1 forks source link

Added interrupts + reselect #26

Closed ormus2002 closed 8 years ago

ormus2002 commented 8 years ago
  1. Статистика посмотри плиз - почему то все время загружается, хотя interrupts я добавил.
  2. И еще - в редусеры добавил TIMEPERIODID_CHANGED, при смене настройки данные обнуляются, как мы и задумали. Но новые не грузятся - например если была открыта статистика, и юзер сменил настройку - при закрытии модального окна надо ему снова перезайти на эту же страницу. Я так думаю, что проще было б передавать timePeriod в компонент, и он сам будет себя обновлять. Разницы никакой - что в каждый редусер записываем его, что в каждый компонент. Но ты может придумаешь что то получше. Или оставим так.
mkozhukharenko commented 8 years ago

Я так думаю, что проще было б передавать timePeriod в компонент, и он сам будет себя обновлять. Разницы никакой - что в каждый редусер записываем его, что в каждый компонент.

@ormus2002, ты совершенно прав. В любом случае придется передавать timePeriodId в компонент и проверять поменялся он или нет (в componentWillReceiveProps). И если поменялся - то вызывать метод который запросит новые данные. Но тем не менее в редюсере statistics нужно будет чистить store когда timeperiodId поменялся.

у нас 2 API-шки которые нужно конфигурировать 1) departments/timeperiods/{{timePeriodId}}/specoffers 2) /stats/{{timePeriodId}}/…

в остальных запросах timeperiodID идет как query params: для  api/specoffers?timePeriodId=8

/api/enorlmnets та api/persons вообще на зависят от этого значения

_можно для этого написать Higher Order React Component которым оборачивать все компоненты с таким функционалом. Но это точно на потом, так как нужно понять как это будет работать. Почитать о HOC линк1 линк2 видео_

надо ему снова перезайти на эту же страницу. это вообще не вариант

mkozhukharenko commented 8 years ago

Статистика посмотри плиз - почему то все время загружается, хотя interrupts я добавил.

@ormus2002 все ок, работает. Пока проверял нашел еще один баг и пофиксил его сразу )

ormus2002 commented 8 years ago

"Статистика по областям" не грузится - пустой график Также "Статистика по областям (карта)" - просто висит Loading Я посмотрел стор - пусто в ALLOCATIONS_ADMINUNITS.data

И еще - там в мастере Славик уже добавил использование cache в экшенах, как быть теперь с нашими interrupts? Использовать их вместе?

mkozhukharenko commented 8 years ago

И еще - там в мастере Славик уже добавил использование cache в экшенах, как быть теперь с нашими interrupts? Использовать их вместе?

@ormus2002 не понял в чем проблема