basics_Zazulya_2 #3
Conversation
playground/server/docker-compose.yml
Outdated
|
|
||
| # Сервис для MongoDB | ||
| mongo-db: | ||
| image: mongo:latest |
There was a problem hiding this comment.
Зафиксировать версию образа для избежания потенциальных ошибок, вызванных новыми версиями.
| image: mongo:latest | ||
| container_name: mongo-db | ||
| ports: | ||
| - "27017:27017" |
There was a problem hiding this comment.
Это dev версия. В prod версии такого быть не должно на будущее.
There was a problem hiding this comment.
Выглядит хорошо. Позже посмотрю с практической точки зрения.
proitshnik
added
In Progress
playground/Zazulya/client/index.js
Outdated
| recorder = new MediaRecorder(combinedStream, {mimeType: "video/webm"}); | ||
| console.log(recorder); | ||
| // Получаем путь для сохранения файла | ||
| fileHandler = await directoryHandle.getFileHandle("testreco.webm", {create: true}); |
There was a problem hiding this comment.
Получается, что файл локально будет постоянно перезаписываться. Для начала думаю пойдет, но пользователь ожидает, что если он 3 раза сделает скринкаст, то и записей будет 3. Подумай, как можно создавать несколько записей. Скорее всего помимо предмета, группы и ФИО придется передавать еще время начала записи.
There was a problem hiding this comment.
Для нескольких записей добавил время к названию файла. Время сделал без ':', чтобы сохранялось на всех системах. И в базу теперь заносится начало и конец записи, можно как-нибудь по-другому их сохранять.
Ascens1ve
dismissed stale reviews from PetrBodunov and proitshnik
via
e08387d
| if (directoryHandle === null) { | ||
| uploadInfo.textContent = "Выберите место сохранения"; | ||
| return; | ||
| } |
There was a problem hiding this comment.
Не понял для чего нужно дублирование if (выше в коде - 25 строка):
if (!directoryHandle) {
uploadInfo.textContent = "Выберите место сохранения";
return;
}
No description provided.