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

docs: update configuration guide for port changes affecting PDF gener… #140

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

Conversation

PeRobin
Copy link

@PeRobin PeRobin commented Mar 13, 2025

Description

Add note explaining that when changing the default port in Docker Compose setup, the HostName in appsettings.json must also be updated to include the new port number for PDF generation to work correctly.

This clarifies the connection between changing the Docker port configuration and updating the app's HostName setting to ensure PDF generation functions properly.

Related Issue(s)

  • No specific issue ticket, but addresses an aspect of local development setup

Verification

  • Documentation-only change

Documentation

  • This PR is a documentation update itself

@PeRobin PeRobin force-pushed the docs/update-port-config-for-pdf branch 3 times, most recently from ad9223f to fe23704 Compare March 13, 2025 11:29
…ation

Add note explaining that when changing the default port in Docker Compose setup,
the HostName in appsettings.json must also be updated to include the new port
number for PDF generation to work correctly.
@PeRobin PeRobin force-pushed the docs/update-port-config-for-pdf branch from fe23704 to f009b9f Compare March 13, 2025 11:49
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.

1 participant