Skip to content
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

Refactor: rename env vars to follow repo renaming #385

Merged

Conversation

josecelano
Copy link
Member

@josecelano josecelano commented Nov 15, 2023

  • Repo torrust-index-backend was renamed to torrust-index.
  • We are not using the IDX abbreviation for INDEX.
  • USER_ID was renamed when @da2ce7 did the container overhaul.
TORRUST_INDEX_BACK_CORS_PERMISSIVE -> TORRUST_INDEX_API_CORS_PERMISSIVE
TORRUST_IDX_BACK_CONFIG            -> TORRUST_INDEX_CONFIG
TORRUST_IDX_BACK_USER_UID          -> USER_ID
TORRUST_IDX_BACK_RUN_AS_USER       -> TORRUST_INDEX_RUN_AS_USER
TORRUST_IDX_BACK_MYSQL_DATABASE    -> TORRUST_INDEX_MYSQL_DATABASE

- Repo `torrust-index-backend` was renamed to `torrust-index`.
- We are not using the `IDX` abbreviation for `INDEX`.
- `USER_ID` was renamed when @da2ce7 did the container overhaul.

```
TORRUST_INDEX_BACK_CORS_PERMISSIVE -> TORRUST_INDEX_API_CORS_PERMISSIVE
TORRUST_IDX_BACK_CONFIG            -> TORRUST_INDEX_CONFIG
TORRUST_IDX_BACK_USER_UID          -> USER_ID
TORRUST_IDX_BACK_RUN_AS_USER       -> TORRUST_INDEX_RUN_AS_USER
TORRUST_IDX_BACK_MYSQL_DATABASE    -> TORRUST_INDEX_MYSQL_DATABASE
```
@josecelano josecelano force-pushed the issue-361-rename-env-variables branch from 1a8c066 to 5b1a3c1 Compare November 15, 2023 10:08
@josecelano josecelano requested a review from da2ce7 November 15, 2023 10:32
@josecelano
Copy link
Member Author

ACK 5b1a3c1

@josecelano josecelano merged commit fe73268 into torrust:develop Nov 15, 2023
13 checks passed
@josecelano josecelano linked an issue Nov 15, 2023 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Rename env variables to follow conventions
1 participant