-
Notifications
You must be signed in to change notification settings - Fork 24
Add recommended project structure to examples #71
Comments
I'm curious about adding the storybook project to the |
@godanny86 It supports adding the storybook project to the Additionally, the changes in pull request 81 will simplify the config files. This comment also shows an example config where the storybook project is in the |
Ah nice! Thanks for the clarification @megazear7. Looks good! |
@CodeByAlex Do you think we should add an example project built with the AEM Project Archetype? It could show off both the |
@megazear7 Yeah I think that would be great. We should also ensure that the existing examples are updated to use the CLI correctly as well. |
I’ve been thinking about this one and I have mixed feelings. I’m thinking we probably shouldn’t have 3 examples right? Now that we know that the cli works with the non archetype should we just replace what’s in the kitchen sync with the archetype - or would that not make sense to some devs would it be better to have a variety |
I see three things that need examples:
It seems like the two examples we have now both demonstrate item 1 (correct me if I am wrong), and that an example project using the AEM Project Archetype could demonstrate both item 2 and item 3. So two examples projects? One example project that demonstrates item 1 from the above list combining the two example projects we have now, and a second example project that demonstrates items 2 and 3 from the above list. |
The examples should include a 'recommended' approach to using these tools, and should also be shown in the context of a 'normal' AEM project folder structure.
The text was updated successfully, but these errors were encountered: