-
Notifications
You must be signed in to change notification settings - Fork 6
Резюме зустрічі 01.09.15
vaa25 edited this page Sep 14, 2015
·
4 revisions
На даній зустрічі було розглянуто поточний стан реалізації user stories.
🕒 [03:11 - 5:45]
🕒 [08:20 - 12:30] - Замечания на основании выходных данных
- Определять бесплатность по event.price, а не по event.free, колонка free в таблице event не нужна;
- Не выводить event.city.location;
- Добавить фильтр по времени для вывода БУДУЩИХ событий;
- Согласовать event.location и event.city.location в тестовых данных
🕒 [23:10 - 45:50] - Замечания по коду
- Почистить комментарии
- Реализовать динамический SQL через Builder, а не в аннотациях, и покрыть его тестами
- Добавить запись в логгер при Exception
- При REST запросе выполнять все обращения к БД в одной транзакции, то есть в одном методе сервиса
- Передавать параметры REST запроса с помощью объекта
🕒 [46:05 - 49:53] - Переименование сервисов