feat: 💥 Allow creating multiple instances of app containers #1788
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Pull request
Purpose
Allow creating multiple instances of built-in apps without using an override.
Approach
Replace all references of the app's name in templates in
.apps
to $(appname} and ${APPNAME}.On run, copy the folders of apps being used to
.instances
, using folder names similar to below.:The "appname" variables in the files in these folders will be replaced by the folder name.
We will then use the
.instances
folder insstead of the.apps
folders when creating the.env
file.Open Questions and Pre-Merge TODOs
Check all boxes as they are completed
Learning
Describe the research stage
Links to blog posts, patterns, libraries or addons used to solve this problem
Requirements
Check all boxes as they are completed