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

Install and enable intl in docker #45

Merged
merged 2 commits into from
Feb 22, 2025

Conversation

marius-hi
Copy link
Contributor

@marius-hi marius-hi commented Feb 22, 2025

Added intl package and its dependency as it is missing when running via a docker container

@matthiasstraka matthiasstraka merged commit 0f41981 into matthiasstraka:main Feb 22, 2025
1 check passed
@matthiasstraka
Copy link
Owner

Thanks, just let me ask you: where did it show up missing?

@marius-hi
Copy link
Contributor Author

marius-hi commented Feb 22, 2025

Thanks, just let me ask you: where did it show up missing?

It show up in templates/instrument/terms.html.twig (line 201)

@matthiasstraka
Copy link
Owner

Thanks for testing that out. I hardly every use the Docker version myself, but as an easy way to test the app, it is important that it works properly. Which also reminds me that I should start working on #37 soon :)

@marius-hi
Copy link
Contributor Author

You're welcome. I personally prefer to run all applications in Docker containers. So far it works properly in the container.

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.

2 participants