-
Notifications
You must be signed in to change notification settings - Fork 2
Fixation of architecture and its documentation #6
Comments
Thanks for the sketches, they look good already. But i dont understand, why we need the Middleware API? The container is already subscribed on the redux store and therefore gets called, if anything changes. This kind of asynchronization is already there, why the Middleware? |
In the first sketches the asynchronous call to the rdfstore was made in the action API. But this was wrong. The actions are mainly for synchronous calls to the Redux store. |
The middleware is basically the handler of asynchron request/responses and delivers. Therefore it makes sense. So the container doesn't wait for something, just fire and forget. But gets called, if an update arrived. This way? |
Yep, thats the way. |
Ok, sounds good. What are your next steps? |
Which bullet points can be already marked solved in the list above? |
With the updated Readme and source we can check the first three points. The new test concept is still on progress. |
Very interesting article how to structure a React application: https://medium.com/@alexmngn/how-to-better-organize-your-react-applications-2fd3ea1920f1#.a0y594u1s |
Yep, had this (or a similar) article in mind when creating the architecture. But we decided against it because the structure overhead.
But with current folder and file structure I would see some advantages in using more filigran separation.
Am 17. Januar 2017 10:47:07 MEZ schrieb Konrad Abicht <notifications@github.com>:
…Very interesting article how to structure a React application:
***@***.***/how-to-better-organize-your-react-applications-2fd3ea1920f1#.a0y594u1s
--
You are receiving this because you were assigned.
Reply to this email directly or view it on GitHub:
#6 (comment)
--
Diese Nachricht wurde von meinem CyanogenMod Android-Mobiltelefon mit K-9 Mail gesendet.
|
All the information you write will result in the future README.md.
The text was updated successfully, but these errors were encountered: