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

ACSF Cloud Hooks / Factory Hooks Enhancements #2682

Closed
wants to merge 5 commits into from

Conversation

lcatlett
Copy link
Contributor

Fixes #2673 and #2676

This is a mix of backports and enhancements.

Changes proposed:

  • Support additional environment stages and Stacks following the standard xxdev / xxtest / xxlive syntax
  • Excludes non-standard update environments / Stacks and production environment from Cloud Hook scripts
  • Disables post-code-update Cloud Hook execution on ACSF
  • Adds post-install, db-update, and post-theme-refresh Factory Hooks
  • Updates post-settings-php Factory Hook

@vyasamit2007
Copy link

I've applied this patch on my BLT version - 8.9.14 and deployed it on the environment 01test2, now it's giving me below error:
/var/www/html/SUBSCRIPTION.01test2/vendor/acquia/blt/scripts/cloud-hooks/functions.sh: line 11: syntax error near unexpected token `('

@grasmash grasmash added the Enhancement A feature or feature request label Mar 27, 2018
@grasmash
Copy link
Contributor

grasmash commented Apr 2, 2018

Replaced by #2685

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement A feature or feature request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants