Грустное мультиисью

[danilaeremin]

1. почему тесты не запускаются в CI/CD? только flake
2. почему при развертывании просят сгенерить миграции? они уже должны быть
3. при обработке форм зачем-то переопределяется метод post, хотя всё решается просто валидностью или нет формы
4. страх и ужас с отступами в питоне
5. ошибки валидации HTML
6. зачем мы храним закомментированное? ни в коде, ни в шаблонах нет смысла это держать
7. отступы в шаблонах:(
8. зачем тут Q?
   
   можно же просто через фильтр
9. смешались в кучу кони, люди... полный бардак с неймингом полей в базе, где-то большая буква, где-то нет, где-то русский, где-то нет
   
10. зачем кэш лежит в репозитории?
11. except Exception - вам же всё должно кричать, что слишком широкое исключение!
12. почему как пользователя меня заставляют думать про какие-то автономные округа, про тип субъекта? что москва это москва в москве в городе федерального значения? тут поседел окончательно
13. подсчет доставки... стоило сделать базовый класс с интерфейсом и для каждого типа доставки сделать свой дочерний, чтобы было красивее/разделимей и понятней
14. я вряд ли захочу вбивать столько полей... это же боль... вы же всё равно геокодируете это, значит вы получаете всю эту инфу
    
15. я не особо, что могу понять, на странице сравнения доставок, никакой удобной таблички
    
16. почему в форме обратной связи предзаполнено поле почты? это же не placeholder, а прям текст!
    
17. ключ для геокодирования утек
    'apikey': '40d1649f-0493-4b70-98ba-98533de7710b',

[Prrromanssss]

Все слишком плохо?🙁
А мы сможем еще исправить, или проект уже не трогать?

[Prrromanssss]

9. пункт
   а какой там бардак?
   'Delivery' это название другой модели, которая через o2m связана

[danilaeremin]

Вы можете доделывать проект до презентации, чтобы там всё было хорошо

[Prrromanssss]

все исправим

[danilaeremin]

Окей, сошлюсь на другое место:)

В целом просто сбивает, что где-то verbose_name через порядковый аргумент, а в случае с вторичным ключом он именованный в 9-ом пункте тогда. и почему это просто строкой, а не через явное указание модели?

[danilaeremin]

Ещё забыл про импорты добавить:)

[Prrromanssss]

да, я увидел, что там вырви глаз код😨

[Prrromanssss]

а что по импортам, у меня isort стоял на автосохранении, я ему и доверял

[Prrromanssss]

а баллы у нас уже неизменны, или мы можем доработать и что-то изменится ?

[danilaeremin]

а что по импортам, у меня isort стоял на автосохранении, я ему и доверял

разве можно доверять тому, что написали люди?:)

[danilaeremin]

а баллы у нас уже неизменны, или мы можем доработать и что-то изменится ?

как я понимаю, это уже окончательно

[Prrromanssss]

а когда вы их скажете?😥

[Prrromanssss]

Данила, а сейчас получше👉🏻👈🏻