Skip to content
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

Managing Modules, Themes and AJAX Apps #25

Closed
p-a-s-c-a-l opened this issue Oct 5, 2018 · 3 comments
Closed

Managing Modules, Themes and AJAX Apps #25

p-a-s-c-a-l opened this issue Oct 5, 2018 · 3 comments
Assignees
Labels
BB: Infrastructure Container Engine / Cloud Infrastructure Building Block BB: UI Integration Platform UI Integration Platform Building Block help wanted Extra attention is needed question Further information is requested

Comments

@p-a-s-c-a-l
Copy link
Member

p-a-s-c-a-l commented Oct 5, 2018

At the moment the git repositories for modules and themes, e.g. csis-theme are referenced as git subtree in the general csis drupal repository docker-drupal (see #12 (comment) and #13 (comment))

Additionally, as described in React Integration in Drupal Page, the react apps and thus the react apps's git repositories currently reside in a subfolder of the csis-theme.

So we would end up with a subtree structure like:

So probably it would be better to switch to git submodules instead, but this approach has some caveats, too.

@fgeyer16 @patrickkaleta How are such issue addressed in other scm-controlled Drupal projects?

@p-a-s-c-a-l p-a-s-c-a-l added help wanted Extra attention is needed question Further information is requested labels Oct 5, 2018
@p-a-s-c-a-l
Copy link
Member Author

@p-a-s-c-a-l
Copy link
Member Author

Apps (like the Map Component) are now nested local git repositories in /drupal-data/web/apps and ignored in the "parent repositories" drupal-data and docker-drupal. This is just a temporary solution for development, apps should be distributed as Drupal modules!

Current directory structure with nested git repositories:

@p-a-s-c-a-l p-a-s-c-a-l added BB: UI Integration Platform UI Integration Platform Building Block BB: Infrastructure Container Engine / Cloud Infrastructure Building Block labels Jan 15, 2019
@p-a-s-c-a-l p-a-s-c-a-l added this to the CSIS Bootstrapping milestone Jan 15, 2019
@p-a-s-c-a-l
Copy link
Member Author

resolved

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BB: Infrastructure Container Engine / Cloud Infrastructure Building Block BB: UI Integration Platform UI Integration Platform Building Block help wanted Extra attention is needed question Further information is requested
Projects
None yet
Development

No branches or pull requests

5 participants