Skip to content
This repository was archived by the owner on Oct 29, 2022. It is now read-only.

Dev Docker Environment #173

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Conversation

jordanlovato
Copy link

I setup the stack to run on docker, it seems to be working but I'm not 100% sure. I would be grateful if someone more experienced with the app could confirm that this branch provides a suitable dev environment. I did this as a warmup to get more familiar with the repo. I know that there's an admin panel and I have not figured out how to get a dev admin session working locally and have not yet confirmed that the admin panel is functional.

`1.5.x-dev` tags appears to no longer be getting picked up by composer.
Bumped tag in `composer.json` to `1.6.x-dev`.
Introduce docker: added redis, mysql, nginx, and fpm. Builds from
`Dockerfile`. I didn't want to adjust the cfg in `config/nginx/dgg.conf`,
So I created a new folder for nginx to mount `config/nginx/local`. I
adjusted the config and only incorporated the website. I intend to port
over the rest of the rewrites. Also updated the README with install instructions.
configure nginx to serve `/static` from `127.0.0.1:9880`, config stored
in `config/nginx/local/dgg_cdn.conf`. I've also removed all CSP headers
from `config/nginx/local`, not sure if CORS is important locally.
@cene-co-za cene-co-za marked this pull request as ready for review January 12, 2020 09:30
@cene-co-za
Copy link
Contributor

@jordanlovato are you ready for merge, I can deal with the conflicts?

@jordanlovato
Copy link
Author

@cene-co-za I'll confirm in a bit, do you know how I can get into the admin panel locally?

Rewrite rules were all wonk'd on cdn config. Fixed those, added a note
to README.md.
@jordanlovato
Copy link
Author

jordanlovato commented Jan 16, 2020

@cene-co-za I think I was confused, I looked through the routing system and it looks like everything is in links config. I think all the admin panel code I saw isn't being used.

I went through the PR and pushed up anything else that I could find. This is good to merge.

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

Successfully merging this pull request may close these issues.

2 participants