Skip to content

Conversation

@Pest12
Copy link
Contributor

@Pest12 Pest12 commented Jul 21, 2025

Удалил файл main.py.
Файл requirements.txt пока не удалял, потому что Николай писал, что у него без него не деплоился проект, так что с этим нужно будет разобраться.
Создал модуль config.py и перенес туда переменные окружения, надо проверить, правильно ли я это оформил. И пока их не подставлял в settings и остальные части проекта.
Так же пока не объединял в один словарь подключение к двум базам данных в settings, нужно будет обсудить это с наставником

@Pest12 Pest12 changed the title Правки после первого созвона с наставником Fixes after the first meeting the mentor Jul 30, 2025
@Pest12 Pest12 changed the title Fixes after the first meeting the mentor Fixes after the first meeting the mentor Aug 1, 2025
@Pest12 Pest12 changed the title Fixes after the first meeting the mentor Fixes after the first meeting the mentor. Aug 1, 2025
@Korjick
Copy link

Korjick commented Oct 3, 2025

Нужно исправить

  1. Не понял, почему в config по умолчанию указан sqlite3, а логин, пароль и хост как от postgres. Тут либо одно либо другое (советую оставить sqlite3).
  2. Также в качестве хоста по умолчанию обычно ставят название БД, а не 127.0.0.1.
  3. Кроме того, наверное имелось ввиду DATABASE_PASSWORD в строке 'POSTGRES_PASSWORD': os.getenv('POSTGRES_PASSWORD', 'password').
  4. Стоит указать и параметр ENGINE в .env файле и по умолчанию сделать его 'django.db.backends.sqlite3'

@Pest12
Copy link
Contributor Author

Pest12 commented Oct 7, 2025

Внес правки.
И хочу еще спросить по поводу БД в settings. Какое подключение к ней лучше оставить? В данный момент есть только локальная и для нее 2 закомиченных подключения и вроде бы в каком то из непринятых ПР я видел еще одно подключение добавилось.

@arthur810629
Copy link
Collaborator

Какой статус?

@Pest12
Copy link
Contributor Author

Pest12 commented Oct 24, 2025

Нужно дождаться, когда наставник разберется и ответит по поводу базы данных

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants