Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -84,4 +84,4 @@ The local OAuth scope is `canvas:asset_library canvas:js_component` (the `member
scope is remote/Acquia-only and is invalid locally).

`ddev clone` reminds you to run it; because it cannot call a host command from
inside the nodejs container, it does not run bootstrap inline.
inside the web container, it does not run bootstrap inline.
2 changes: 1 addition & 1 deletion commands/nodejs/clone → commands/web/clone
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ shift # remaining args passed through to run.js

# NOTE: permission bootstrap runs as the HOST command `ddev canvas-bootstrap`
# (it needs drush — which lives in the web container — and writes the host
# storybook/.env). This `clone` command runs in the nodejs container and cannot
# storybook/.env). This `clone` command runs in the web container and cannot
# invoke a host command, so bootstrap is not called inline here. The reminder
# below points users at it; it is idempotent and safe to re-run.
echo "==> If local permissions are not bootstrapped yet, run from the host first: ddev canvas-bootstrap"
Expand Down
2 changes: 1 addition & 1 deletion install.yaml
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
name: canvas-migration
project_files:
- config.canvas-migration.yaml
- commands/nodejs/clone
- commands/web/clone
- commands/host/canvas-bootstrap
- web-build/Dockerfile.canvas-migration
- website-to-components
Expand Down