basics_shlyahtin_2 #5
Conversation
playground/server/docker-compose.yml
Outdated
| image: mongo:latest | ||
| container_name: mongodb | ||
| ports: | ||
| - "27017:27017" |
There was a problem hiding this comment.
Это dev версия. В prod версии такого быть не должно на будущее.
playground/server/docker-compose.yml
Outdated
| - MONGO_URI=mongodb://mongodb:27017/testdb | ||
|
|
||
| mongodb: | ||
| image: mongo:latest |
There was a problem hiding this comment.
Зафиксировать версию образа для избежания потенциальных ошибок, вызванных новыми версиями.
playground/server/docker-compose.yml
Outdated
| depends_on: | ||
| - mongodb | ||
| environment: | ||
| - MONGO_URI=mongodb://mongodb:27017/testdb |
There was a problem hiding this comment.
На будущее не хардкодить, а использовать .env
playground/server/app.py
Outdated
| def insert_data(): | ||
| """Тестовая вставка данных в коллекцию 'users'""" | ||
| users = mongo.db.users | ||
| user_id = users.insert_one({"name": "Test user", "age": 25}).inserted_id |
There was a problem hiding this comment.
С базой данных нужно работать осторожно, поэтому нужно использовать try-except
playground/client/popup.js
Outdated
There was a problem hiding this comment.
Выглядит хорошо. Обработку ошибок, системные сообщения будем уже в main дорабатывать. Позже посмотрю с практической точки зрения.
There was a problem hiding this comment.
Хороший прототип с всплывающим типом. Улучшения, доработки будут вестись уже в основной программе.
proitshnik
added
In Progress
proitshnik
added
In Review
| container_name: mongodb | ||
| restart: always | ||
| env_file: | ||
| - .env # Переменные теперь загружаются из .env |
There was a problem hiding this comment.
Необходимо подгрузить его на GitHub
There was a problem hiding this comment.
А заливать именно .env или сделать отдельно файлик .env.sample? Я немного про это почитал, и некоторые рекомендуют подобную практику
There was a problem hiding this comment.
Поскольку мы работаем в playground, как в песочнице, то .env (.env.sample шаблон для него, чтобы вписывать чувствительные данные на месте).
Fixes #2