Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature 45 #62

Merged
merged 4 commits into from
Mar 24, 2016
Merged

Feature 45 #62

merged 4 commits into from
Mar 24, 2016

Conversation

Ananaskelly
Copy link
Contributor

Изменения по #45 задаче

@Ananaskelly
Copy link
Contributor Author

Мне почему-то казалось, что я не смогу проверить локально, поэтому я не проверила изначально, но у меня все равно не вышло решить проблему.

@malinink
Copy link
Owner

@Ananaskelly
Ответ на самом деле довольно прост, держите подсказку

@Ananaskelly
Copy link
Contributor Author

поняла, спасибо
всё попробовала, но не заметила, что там был tinyInteger
теперь вроде всё

@Ananaskelly
Copy link
Contributor Author

Я вот сейчас выяснила, что модель при регистрации не создается и ругается на foreign key. Решилось добавлением nullable() к defaul_game_type_id. Стоит ли так исправить?

@malinink malinink merged commit 8c6e433 into malinink:dev Mar 24, 2016
@malinink
Copy link
Owner

@Ananaskelly
nullable да, неплохо - напишите этот комент в задаче по тестированию этой модели, путь в рамках её решения подправят.
Но надо понимать что это не для всех полей модели должно работать именно так.
В данном случае null означает что пользователь не определил тип игры по-умолчанию, и для поиска надо у него спросить - что будем искать.
Наверняка будут можели где связь не может быть null.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants