-
Notifications
You must be signed in to change notification settings - Fork 219
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
Migration to Nuxt 3 #3646
Migration to Nuxt 3 #3646
Changes from all commits
85738fc
c836166
afcc564
2ef58b8
d0f54b8
314e557
684f7f5
12f0e11
6f1fd2e
8892139
d5dfb62
2a31ec0
066bb6a
03b7a2d
b2773f2
4059e6e
cf392f2
d063af8
d14f96a
0a7f47f
645ef09
6505c4d
c064ac4
f5d1809
7edfb39
3b1259b
1408deb
c16ecf4
5b3edae
db27d7a
3481d88
3a29226
c5a38fa
25bab10
e248b99
e012058
f9f7b0b
59d228c
a49facd
32bfc93
f4fe4dc
2cc6390
3e7b31b
595b955
bee8863
72e80ac
9ba5e8f
7edd0e5
3a80bf0
3752da1
87ff226
f2cc4c9
f5483ea
4415bf6
3b18b1e
4ed3b83
d29464c
d85aa61
716fa6f
43d82ae
b1bb461
6354259
5b67f82
c820e22
f9e4d49
d3bcfd4
2cd94a6
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. These will need to be added to the ECS template for the frontend before we deploy, right? Or some subset of them? |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,3 +1,13 @@ | ||
#API_URL=http://127.17.0.1:8000/ | ||
#API_CLIENT_ID="" | ||
#API_CLIENT_SECRET="" | ||
#NUXT_PUBLIC_API_URL=http://127.17.0.1:8000/ # local dev API | ||
#NUXT_PUBLIC_API_URL=http://localhost:49153/ # talkback proxy | ||
#NUXT_PUBLIC_API_URL=https://api.openverse.engineering/ # prod API | ||
#NUXT_API_CLIENT_ID="" | ||
#NUXT_API_CLIENT_SECRET="" | ||
NUXT_PUBLIC_SAVED_SEARCH_COUNT=4 | ||
NUXT_PUBLIC_PROVIDER_UPDATE_FREQUENCY=3600000# 1 hour | ||
NUXT_PUBLIC_PLAUSIBLE_DOMAIN="openverse.org" #localhost | ||
NUXT_PUBLIC_PLAUSIBLE_API_HOST="https://openverse.org" #http://localhost:50290 | ||
NUXT_PUBLIC_SENTRY_DSN="https://[email protected]/5799642" | ||
NUXT_PUBLIC_SENTRY_ENVIRONMENT="local" | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Safe to expose/use publicly as the default? |
||
NUXT_PUBLIC_SENTRY_RELEASE="" | ||
PORT=8443 |
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
This file was deleted.
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -22,7 +22,7 @@ WORKDIR /home/node/ | |
|
||
# Copy monorepo mocking files into `/home/node`, which pretends to be the monorepo root. | ||
# Note: these files must be manually un-ignored in the root .dockerignore | ||
COPY --from=repo_root --chown=node:node .npmrc .pnpmfile.cjs pnpm-lock.yaml tsconfig.base.json ./ | ||
COPY --from=repo_root --chown=node:node .npmrc .pnpmfile.cjs pnpm-lock.yaml ./ | ||
RUN echo '{"packages":["frontend/"]}' > pnpm-workspace.yaml | ||
|
||
# Copy the `frontend/` directory into `/home/node/frontend`, as a package in the monorepo. | ||
|
@@ -41,9 +41,10 @@ RUN pnpm install | |
# disable telemetry when building the app | ||
ENV NUXT_TELEMETRY_DISABLED=1 | ||
ENV NODE_ENV=production | ||
ENV SENTRY_DSN=https://[email protected]/5799642 | ||
ENV NUXT_PUBLIC_SENTRY_DSN=https://[email protected]/5799642 | ||
|
||
ARG API_URL | ||
ARG API_URL=https://api.openverse.engineering/ | ||
ENV NUXT_PUBLIC_API_URL=$API_URL | ||
|
||
RUN pnpm build:only | ||
|
||
|
@@ -68,7 +69,9 @@ COPY --from=builder --chown=node:node /home/node/frontend ./frontend/ | |
WORKDIR /home/node/frontend/ | ||
|
||
ARG SEMANTIC_VERSION | ||
ENV SENTRY_RELEASE=$SEMANTIC_VERSION | ||
ARG DEPLOYMENT_ENV=production | ||
ENV NUXT_PUBLIC_SENTRY_RELEASE=$SEMANTIC_VERSION | ||
ENV NUXT_PUBLIC_SENTRY_ENVIRONMENT=$DEPLOYMENT_ENV | ||
|
||
# set app serving to permissive / assigned | ||
ENV NUXT_HOST=0.0.0.0 | ||
|
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.
Should we just comment these out if they're going to come back, or are they gone for good?
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.
#3664 is open to re-add Storybook so I suppose this is coming back. Also applies to similar steps related to Storybook in the CI + CD workflow.