[PR #1] refactor: runtime config #6

New Issue

arthur · 2025-12-01T17:04:27+01:00

arthur commented

2025-12-01 17:04:27 +01:00

Original Pull Request: https://github.com/ArthurDanjou/artsite/pull/1

State: closed
Merged: Yes

Because of some typos, you will have to update your environment variables in your deployment like the USER_UD instead of the USER_ID

Hello 👋,

I refactor the usage of the runtime config. First, there was some typos between the name of the environment variable and the name of the runtime config. Then, I remove the usage of process.env since Nuxt is already able to overwrite the runtime config with the environment variables.

You can learn more with this video: https://youtu.be/_FYV5WfiWvs?si=XWUumfMOBC90TivF

I think the runtime config discord.id and discord.token is not used and could be removed.

**Original Pull Request:** https://github.com/ArthurDanjou/artsite/pull/1 **State:** closed **Merged:** Yes --- **Because of some typos, you will have to update your environment variables in your deployment like the USER_UD instead of the USER_ID** Hello 👋, I refactor the usage of the runtime config. First, there was some typos between the name of the environment variable and the name of the runtime config. Then, I remove the usage of `process.env` since Nuxt is already able to overwrite the runtime config with the environment variables. You can learn more with this video: https://youtu.be/_FYV5WfiWvs?si=XWUumfMOBC90TivF I think the runtime config `discord.id` and `discord.token` is not used and could be removed.

arthur added the pull-request label 2025-12-01 17:04:27 +01:00

arthur closed this issue

2025-12-01 17:04:27 +01:00

arthur referenced this issue

2025-12-01 17:04:28 +01:00

[PR #6] [MERGED] Seo bug #11

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: arthur/artsite#6