-
Notifications
You must be signed in to change notification settings - Fork 7
Conversation
@@ -21,4 +21,5 @@ npm-debug.log* | |||
yarn-debug.log* | |||
yarn-error.log* | |||
|
|||
# sass to css |
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.
just curious if we want to clear out the defaults in the .gitignore that aren't relevent to our use case
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.
seems good
@@ -27,6 +28,7 @@ ENV CMS_API=${CMS_API:-https://joplin.herokuapp.com/api/graphql/} | |||
ARG CMS_MEDIA | |||
ENV CMS_MEDIA=${CMS_MEDIA:-https://joplin.herokuapp.com/media/} | |||
|
|||
COPY public /app/public |
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.
saw this was in Dockerfile but not in Dockerfile.build -- just I added it. Not sure it's necessary at this moment tho.
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.
🐐 🌐
cdb0585
to
de14615
Compare
e6a02d8
to
739d065
Compare
26bd663
to
23520d3
Compare
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.
@@ -21,4 +21,5 @@ npm-debug.log* | |||
yarn-debug.log* | |||
yarn-error.log* | |||
|
|||
# sass to css |
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.
seems good
@@ -12,6 +12,7 @@ COPY yarn.lock /app/yarn.lock | |||
COPY package.json /app/package.json | |||
RUN yarn | |||
|
|||
COPY intl.buildlangs.js /app/intl.buildlangs.js |
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'll need to add this to the Docker.storybooks too huh.
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.
nah, the build langs is only to generate the base english json -- so it's only needed in a dev environment
To re-generate default.json to update the static content to be translated, the following command can be run: | ||
|
||
``` | ||
docker exec --interactive --tty janis yarn run build-langs |
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.
🔥
Janis static content language translation