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

Add deployment info for Divio Cloud #31

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

FinalAngel
Copy link

This merge request adds information on how to deploy the template to Divio Cloud according to a discussion between Adam and Meagen. The resulting template works out of the box within our environment ☺️

@vossisboss
Copy link
Contributor

Hey @FinalAngel! Thank you for submitting this. I was out for the Thanksgiving holiday for most of last week, so I'm just getting around to testing this. I went through your instructions to deploy a copy of this repository to Divio and unfortunately the deployment failed for me with the following error:

===== docker build =====

starting docker build
Operation Error: failed to parse Dockerfile: Syntax error - can't find = in "project_name". Must be of the form: name=value

Any ideas on what the issue might be?

I made a few small language suggestions as well. Please let me know if you have any questions.

@FinalAngel
Copy link
Author

@vossisboss, no worries, I also have been off for holidays 🙈 I made some amendments regarding the deployment to make it clear that you'd need a local setup following the "Getting Started" guide. You'll end up with a setup like this:
https://github.com/finalangel/wagtail-news-template-testing/

I sadly did not see your language suggestions in the PR, did you submit the review? 🙇‍♂️

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

Successfully merging this pull request may close these issues.

3 participants