Add landing page service #325
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
nitrosx
approved these changes
Oct 2, 2024
minottic
force-pushed
the
lp
branch
2 times, most recently
from
74aea6c to
c9e77c4
Compare
Add config Add README Add documentation to main README Add landingpage to main compose.yaml includes
Update README with HTTPS changes Use newer image and fix config
Add DEV instructions to landingpage README Use newer image and fix config
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR adds the landing page SciCat service. It is split into commits for readability:
beandfeURLs whenever the user setsBACKEND_HTTPS_URLandFRONTEND_HTTPS_URL. This is required as the landingpage, being a SPA, cannot use the docker internal network and uses the host network from the browser. It also includes README required changesREADMEcloses: #29