Skip to content
This repository has been archived by the owner on Dec 13, 2023. It is now read-only.

Updated serverAndUI Dockerfile to set the correct startup script #3292

Merged
merged 2 commits into from
Oct 17, 2022

Conversation

bmuluu
Copy link
Contributor

@bmuluu bmuluu commented Oct 12, 2022

Set the correct startup script so that the UI is served.

Pull Request type

  • Bugfix
  • Feature
  • Refactoring (no functional changes, no api changes)
  • Build related changes (Please run ./gradlew generateLock saveLock to refresh dependencies)
  • WHOSUSING.md
  • Other (please describe):

NOTE: Please remember to run ./gradlew spotlessApply to fix any format violations.

Changes in this PR

Updated the Dockerfile for the combined server and UI combined image.

Describe the new behavior from this PR, and why it's needed
The front end was not being served. The Dockerfile now uses the correct startup script.

Alternatives considered

Describe alternative implementation you have considered

Nginx was not running so the UI was not being served.
@jxu-nflx jxu-nflx merged commit 49a1076 into Netflix:main Oct 17, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants