This repository has been archived by the owner on Sep 7, 2020. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 249
Boilerplate: Rename LayoutContainer to AppContainer #444
Labels
Comments
👍 I agree |
@revolunet @thangngoc89 any thoughts on this? |
This mightbe duplicated with react hot loader 3 if we go for that name. |
we will use rhl3 appcontainer so people don't have to ;) |
Ok. If so, that name is fine to me |
This was referenced Aug 31, 2016
MoOx
added a commit
that referenced
this issue
Sep 1, 2016
We encourage you to update to a similar structure if you were using the previous one. Main changes: - ``web_modules/layouts`` => ``src/layouts`` - ``web_modules/{Components}`` => ``src/components/*`` - ``web_modules/app/*`` => ``src/*`` - ``web_modules/LayoutContainer`` => ``src/AppContainer.js`` Closes #529 Closes #444
MoOx
added a commit
that referenced
this issue
Sep 1, 2016
We encourage you to update to a similar structure if you were using the previous one. Main changes: - ``web_modules/layouts`` => ``src/layouts`` - ``web_modules/{Components}`` => ``src/components/*`` - ``web_modules/app/*`` => ``src/*`` - ``web_modules/LayoutContainer`` => ``src/AppContainer.js`` Closes #529 Closes #444
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Following https://twitter.com/MoOx/status/728210032093990912
The text was updated successfully, but these errors were encountered: