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 Dockerfile, support for default ADIF_URL #8

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

Conversation

robertlestak
Copy link

  • Adds a Dockerfile for containerization and easier deployment (if that's your prerogative)
  • Adds a scripts/build.sh script which consumes an ADIF_URL env var to update the index.html file with the default ADIF URL.
    • When set, this will always take precendee over the url query string param, enabling it to be used to solely display desired ADIF without allowing user to change URL param

@robertlestak robertlestak force-pushed the master branch 2 times, most recently from 7edbb94 to 4ecb4cb Compare October 27, 2024 18:34
@stephenhouser
Copy link
Owner

I know this is here, just have not decided if I want to merge or not.

Changes in this PR:

  • Setting a static ADIF URL to load which disables ability to load other URLs. (I see where this would be useful to folks just wanting to display their own maps)
  • Setting the Title/Brand on the page header.
  • Adding Dockerfile and k8s launch/configuration examples.

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