-
Notifications
You must be signed in to change notification settings - Fork 1
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
Figure out how to structure project #28
Comments
My latest PR has a first-pass implementation of this structure. |
It's a good idea for enterprise apps with a ton of components, but for us, it'll be overcomplication.
Still, we can borrow some ideas from the architecture |
So far I've found it easier working this way but if you guys are used to the traditional methods I'm fine with that. |
It's easier while we haven't added redux. This post describes a few bad practices and very complicated nested structure. |
OK let's refactor on top of the code I checked in then.
…On Sat, Jun 3, 2017 at 11:13 PM Tim Reznich ***@***.***> wrote:
It's easier while we haven't added redux. This post
***@***.***/how-to-use-redux-on-highly-scalable-javascript-applications-4e4b8cb5ef38>
describes a few bad practises and very compicated nested structure.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#28 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AK-r0w3lwNM3vcbnpxLRfj6Lc3iPEmzcks5sAksdgaJpZM4NvPhR>
.
|
I think we should nest components by functionality and how they will be used, similar to what this article describes: https://medium.com/@alexmngn/how-to-better-organize-your-react-applications-2fd3ea1920f1
We can have top level components and nest components within our "scenes". What do you guys think?
The text was updated successfully, but these errors were encountered: