-
Notifications
You must be signed in to change notification settings - Fork 0
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
Update Dockerfile and docker-compose for k8s #30
base: tre
Are you sure you want to change the base?
Conversation
Codecov Report
@@ Coverage Diff @@
## tre #30 +/- ##
=======================================
Coverage 73.08% 73.08%
=======================================
Files 278 278
Lines 6026 6026
Branches 1661 1661
=======================================
Hits 4404 4404
Misses 1437 1437
Partials 185 185 Continue to review full report at Codecov.
|
app/constants/AppConstants.js
Outdated
@@ -8,7 +8,7 @@ export default { | |||
'varaamotest-espoo.hel.ninja': 'ESPOO', | |||
'varaamo.vantaa.fi': 'VANTAA', | |||
'varaamotest-vantaa.hel.ninja': 'VANTAA', | |||
'varaamo.tampere.fi': 'TAMPERE', // PROD | |||
'localhost:3000': 'TAMPERE', // PROD |
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.
We don't want to change this. This would default to using Helsinki specific page for Varaamo.
|
||
COPY package.json package.json | ||
RUN yarn install |
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.
Add this before the COPY . /app/ step
No description provided.