Refactoring/user model

[xxfeel]

Closes #52

Description

1. Убрана роль 'пользователя'.
2. Переработана модель User согласно ER-диаграмме, добавлены методы для удобного отображения имён.
3. В phone добавлена валидация телефонного номера, для российского формата. Протестировано пока только в django-shell.

>>> admin = User.objects.get(pk=1)
>>> admin.phone
''
>>> admin.phone = '8800'
>>> admin.phone
InvalidPhoneNumber(raw_input=8800)
>>> admin.phone.is_valid()
False
>>> admin.phone = '+79123456789'
>>> admin.phone
PhoneNumber(country_code=7, national_number=9123456789, extension=None, italian_leading_zero=None, number_of_leading_zeros=None, country_code_source=1, preferred_domestic_carrier_code=None)
>>> admin.phone.is_valid()
True

4. В Makefile добавлена команда make shell, которая запускает django-shell
5. При активации создания суперюзера manage.py createsuperuser, хотелось добиться того что бы роль 'admin' устанавливалась по умолчанию и без дополнительного запроса. Нужно будет переписывать в менеджере команду 'createsuperuser'. На самом деле пока что стоит костыль, при создании суперпользователя какую роль мы бы не поставили, в базу запишется админ. Если сейчас это не приоритетно, можно занятся этим чуть позже.
6. pre-commit не пропускает пакет django-phonenumber-field, из-за отсутсвия в нём типизации. Mypy и CI/CD плюсом не пропускают типизацию от django.

[KonstantinRaikhert]

По mypy:
Вот так нужно в pyproject добавить. А остальные варнинги придется решить исправлением кода)

[KonstantinRaikhert]

давай тогда сразу добавим shell +
https://django-extensions.readthedocs.io/en/latest/shell_plus.html

Обрати внимание что эти пакеты в dev)

[xxfeel]

Обрати внимание что эти пакеты в dev)

Извини, не понял какие пакеты ты имеешь ввиду

[KonstantinRaikhert]

имеется ввиду django-extensions, так есть более удобный инструмент shell_plus.
Ну чтобы они не уходили эти пакеты в прод, а остались в дев)
Может конечно очевидно, но решил напомнить

[xxfeel]

Нет, всё в порядке) показалось что открылась документация

[KonstantinRaikhert]

@IvanFilippov74 Посмотри пожалуйста, что я изменил в твоем пулреквесте. В основном это касается mypy flake8, заодно поправил как должны лежать настройки)

[xxfeel]

По flake8 не совсем понятно, ошибка указывала на импроты в models.py, после её устранения импорты выглядят так:

from core.constants import (
    EMAIL_MAX_LENGTH,
    NAME_MAX_LENGTH,
    QUERY_SET_LENGTH,
    ROLE_ADMIN,
    ROLE_AGENT,
    ROLE_MODERATOR,
    ROLES_CHOICES,
)
from django.contrib.auth.models import AbstractBaseUser, PermissionsMixin
from django.core.mail import send_mail
from django.db import models
from django.utils import timezone
from django.utils.translation import gettext_lazy as _
from phonenumber_field.modelfields import PhoneNumberField
from phonenumber_field.validators import validate_international_phonenumber
from users.managers import CustomUserManager

Но ведь по код стайлу мы импортируем в таком порядке:

стандартные библиотеки

модули джанго

локальные импорты

Или приложение core приравнивается к модулю фреймворка, а не к локальному импорту?