-
-
Notifications
You must be signed in to change notification settings - Fork 778
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
DO NOT MERGE: docker-compose.yml has been updated to pull the test image from dockerhub #7359
base: gh-pages
Are you sure you want to change the base?
DO NOT MERGE: docker-compose.yml has been updated to pull the test image from dockerhub #7359
Conversation
Want to review this pull request? Take a look at this documentation for a step by step guide! From your project repository, check out a new branch and test the changes.
|
Review ETA: 9/1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The code changes align with the ticket description, and the images on the website are loading fine in the local testing environment.
Thank you for working on this ticket.
Review ETA: 13 PM 9/22/2024 |
Putting review on hold as discussed in Sunday meeting |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@roslynwythe everything appears to work when I run the container locally, the site is up and pages display as they should.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @roslynwythe, the first time I wanted to review I had an issue running docker with the changes. There was an error when pulling the image, but now it's working. I commented that on Sunday.
The changes to the source code are correct, and the site is up and running.
Thanks for working on this issue
Fixes #7358
What changes did you make?
docker-compose.yml
to referencehackforlaops/ghpages:testing
Why did you make the changes (we will use this info to test)?
docker-compose up
runs it will pull down the test image, recreate the container, build and server the site so that we can testInstructions for Reviewers/Merge Team
docker-compose up
and confirm that the site at http://localhost:4000 is workingScreenshots of Proposed Changes To The Website (if any, please do not include screenshots of code changes)
No visual changes to the website