search

shri-2015-org / 666

666 team's chat =>
http://anonym.club
4 stars 3 forks source link

add mongoose models #116

Closed deemidroll closed 8 years ago

petrukhin commented 8 years ago

Организация файлов получается плохая, сейчас в папке models лежат в перемешку и модели, и тесты, и утилиты. Лучше отделить сущности друг от друга.

deemidroll commented 8 years ago

Сделано для удобства удаления файлов на продакшене. Стоит завести отдельную директорию тут же в моделях? Или вообще единую со всеми тестами?

pofigizm commented 8 years ago

Я считаю, что файл тестов нужно хранить вместе с тестируемым файлом. Это наглядно показывает что уже покрыто тестами и не нужно городить сложные импорты.

tempname11 commented 8 years ago

Поддерживаю @Pofigizm про файлы тестов.

deemidroll commented 8 years ago

@Pofigizm, @tempname11 посмотрите плиз, пока не реализовал правилный joinRoom, остальное сделал в соответствии с комментами и вчерашними обсуждениями

tempname11 commented 8 years ago

Вроде всё неплохо, жду продолжения:)

pofigizm commented 8 years ago

Вроде норм.

deemidroll commented 8 years ago

Сейчас создается стандартная комната, без нее клиенсткий код не обрабатывает исключения, что сервер не может предоставить списко комнат для топа или для поиска. Стоит ли слать пустой массив комнат? В общем, призываю к обсуждению :)

tempname11 commented 8 years ago

@deemidroll конечно. Если на сервере нет комнат, это же не значит, что "список комнат" не существует. Это значит, что он именно пуст.

pofigizm commented 8 years ago

@deemidroll ты закончил? готов это вмержить? или что-то еще делаешь?

deemidroll commented 8 years ago

Можно вмержить, но там надо насчёт пустых комнат поправить. Плюс местами мне не нравится качество кода. Предлагаю вам попробовать ветку у себя с установленной монгой и поставить +1, чтобы вмержить. Я пока поправлю ответ, если комнат нет

deemidroll commented 8 years ago

Кстати, комнаты пока не удаляются, как и юзеры по дисконнекту

tempname11 commented 8 years ago

Сегодня (чуть позже) попробую обязательно.

deemidroll commented 8 years ago

Вопросы:

  1. Какое время должна жить комната после того, как ее покинут все пользователи, прежеде чем она будет удалена?
  2. Какое время долеж жить юзер после последнего дисконнекта? То есть, сколько времени мы будем позволять его реконнектить?
tempname11 commented 8 years ago

Мои ответы (личное мнение)

  1. Сразу удаляться.
  2. Для тестирования удобно поставить маленькое время, а для продакшна, ну скажем 24 часа для ровного счету.
tempname11 commented 8 years ago

С небольшими странными проблемами работает, давайте смержим, и дальше проблемы будем отлаживать в девелопе.

+1

deemidroll commented 8 years ago

перечисли проблемы, плиз

exah commented 8 years ago

У меня вроде ок :+1:

pofigizm commented 8 years ago

Не мержи пока не вмержим дев в мастер

exah commented 8 years ago

В деве будет монга, а в мастере будет пока без неё?

pofigizm commented 8 years ago

да. когда все отладим - выкатим с ней.

deemidroll commented 8 years ago

ок

deemidroll commented 8 years ago

@Pofigizm вмержу после твоего +1

pofigizm commented 8 years ago

Нужен второй +1 на том ПР

pofigizm commented 8 years ago

:+1:

tempname11 commented 8 years ago

@deemidroll я сам пока не опознал их в точности. То ли я запускал криво, то ли реально есть баги. Буду еще разбираться.

pofigizm commented 8 years ago

Мы что решили? Мержим?

deemidroll commented 8 years ago

+1 :)

deemidroll commented 8 years ago

Мержу?

deemidroll commented 8 years ago

тут уже достаточно плюсов

tempname11 commented 8 years ago

Ура! наконец-то:))