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

ACA-3221: Added content-app to community docker compose file #704

Merged

Conversation

mstrankowski
Copy link
Contributor

@mstrankowski mstrankowski commented May 19, 2022

partially changed README.md

Ref. ACA-3221

Marcin Strankowski added 2 commits May 20, 2022 14:53
And updating community-docker-compose.yml to better reflect the content-app approach.
This just needs a new acs-ingress version with added changes -> upping version used in all docker-compose files
@mstrankowski mstrankowski changed the title ACA-3221: Added content-app using structures dedicated for ADW ACA-3221: Added content-app to community docker compose file May 24, 2022
Copy link
Contributor

@alandavis alandavis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - Both community and enterprise have been tested locally.

@mstrankowski mstrankowski merged commit e69ad20 into master May 24, 2022
@mstrankowski mstrankowski deleted the feature/ACA-3221_include_aca_in_the_acs_community_edition branch May 24, 2022 15:22
@mteodori
Copy link
Member

@mstrankowski I just noticed the alfresco-content-app is pulled from quay rather than dockerhub so it requires auth, shall we change this?

alxgomz pushed a commit that referenced this pull request Jan 23, 2024
* ACA-3221: 
Alfresco Content App (ACA) to community-docker-compose file. 
Updated acs-ingress (proxy, image: alfresco-acs-nginx) to correctly route the application to "{host:address:port}/aca" address. 
Updated readme files and docker compose UML diagram of community docker compose file
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants