-
Notifications
You must be signed in to change notification settings - Fork 19
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Separating presentational and container components #225
Conversation
401318e
to
4254417
Compare
Is it definitely necessary to have everything in its own directory? It's hard for me to see the advantage of that. |
@bnaul I think the idea is to eventually put all of the redux code
(actions, reducers, etc) related to each component into that component's respective
directory.
…On Monday, August 7, 2017, Brett Naul ***@***.***> wrote:
Is it definitely necessary to have everything in its own directory? It's
hard for me to see the advantage of that.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#225 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AG5TTRbWUiuGoMnLo-4NWKdsts3QrvQVks5sVypagaJpZM4OuLNq>
.
|
376486f
to
fab7aad
Compare
afacff9
to
1640cac
Compare
b843883
to
c8bdbe2
Compare
Closes #255 |
@acrellin I haven't thought about this for a while, but Andy mentioned https://github.com/react-boilerplate/react-boilerplate to me in cesium-ml/baselayer#41 It may be worth to just double check how they do things (for all I know it's the same as in this refactor). |
@stefanv Looks like we're using the same structure |
… separate directories
…ntainer) component files
72c15d6
to
bd341c0
Compare
@stefanv this is ready for review |
36a6dcc
to
e4eb913
Compare
We've now moved to functional components + redux logic in |
This is just the
Projects.jsx
refactor.