search

eshcherbin / save-the-moment

SPbAU educational Android project
0 stars 0 forks source link

Implemented basic DB functionality in MomentManager #7

Closed eshcherbin closed 7 years ago

eshcherbin commented 7 years ago

Реализована база данных хранения моментов.

eshcherbin commented 7 years ago

@krinkinmu, можете проверить фиксы? Коммит был сделан два часа назад, а после этого вы ещё отвечали на другой Issue, следовательно, возможно, что этот PR остался незамеченным.

krinkinmu commented 7 years ago

Он не остался незамеченным, вы просто не все замечания поправили.

eshcherbin commented 7 years ago

Вы про то, что замечание "после исправления с foreign key достаточно будет удалить только момент" автоматически не закрылось? Я его исправил, удаляю только момент, foreign key delete on cascade добавил. Или я что-то другое не исправил?

krinkinmu commented 7 years ago

Вам на странице должны быть показаны все замечания, которые я написал - что трудного в том, чтобы посмотреть на них? Их же не 10^9 и даже не 10^6. Я имею ввиду замечание про использование транзакций.

krinkinmu commented 7 years ago

Approved