heavyoranjs/somecompany-test-assignment

somecompany-test-assignment's Issues

somecompany ревью

owner-edit: убрано название компании, слишком легко гуглится и руинит тест

1. https://github.com/HeavyOranJS/somecompany-test-assignment/blob/master/assignment/views.py#L50
   Может упасть если username пустой. Есть смысл обернуть в try...except User.DoesNotExist.

2. https://github.com/HeavyOranJS/somecompany-test-assignment/blob/master/assignment/models.py
   К моделям и к полям моделей следует указывать verbose_name

3. https://github.com/HeavyOranJS/somecompany-test-assignment/blob/master/best_data_base.png
   А что такое family_name у employee?

4. https://github.com/HeavyOranJS/somecompany-test-assignment/blob/master/somecompany-testTest/settings.py#L44
   ALLOWED_HOST тоже нужно задавать в json файле

5. https://github.com/HeavyOranJS/somecompany-test-assignment/blob/overtime/assignment/forms.py#L81
   тут тоже не хватает обработки исключений. Для сетевых запросов советую https://github.com/not-kennethreitz/requests (ты только глянь на количество звезочек!)

6. https://github.com/HeavyOranJS/somecompany-test-assignment/blob/master/somecompany-testTest/dummy.config.json#L2
   =)

В целом неплохо, молодец.