Skip to content
  • Goerge Egiazaryan's avatar
    83054bc7
    исправлены замечания после 3-го ревью; (#83) · 83054bc7
    Goerge Egiazaryan authored
    
    
    * add review and api apps
    
    * Сделал модель для произведений
    
    * Исправил ошибку с максимальной длинной рейтинга
    
    * Добавил модель для категорий
    
    * переместил приложения reviews и api на правильный уровень
    
    * подключены приложения reviews, api. Выполнена базовая конфигурация проекта.
    
    * Добавил файл сериализатора
    
    * Добавил докстринги, сделал роутер для апи, зарегестрировал эндпоинт для произведений
    
    * Добавил модель для жанров
    
    * добавил приписку v1 в urls api
    
    * Добавил ViewSet для категорий
    
    * Добавил эндпойнты для категорий
    
    * Связал модель произведений с жанрами и категориями
    
    * Переделал связанное удаление
    
    * + добавлена модель Пользователя и подключена в settings.py
    + исправлена ошибка с max_length для SmallIntegerField
    
    * Связал модель произведений с пользователем
    
    * Переделал связть в Title
    
    * Переделал связь в Title
    
    * Добавил эндпоинт для жанра
    
    * Добавил модели для отзывов и коментариев так же сделал эндпоинты
    
    * Переделал эндпоинты и связь в модели Comment
    
    * написан(частично) сериализатор для пользователей
    добавлены 2 константы в модель пользователя.
    
    * Провёл рефактор
    
    * добавлено: emailbackend, simple_JWT setting. Подключен DRF
    
    * Добавил view и сериализатор для Categories
    
    * Добавил views и сериализатор для Genres
    
    * Исправил dockstring
    
    * сделал серрилизатор и view для произведений
    
    * Добавил базовую структуру обзоров
    
    * Исправил импорт модели с Genre на Genres для сериализатора API
    
    * исправил баг в серрилизаторе произведений
    
    * Фикс urls и serializers для Reviews
    
    * Добавил сиррилизатор для комментариев
    
    * Исправил докстринги
    
    * Добавил валидацию оценки Reviews
    
    * реализована работа эндпойнтов signup и token:
     'signup' - регистрация/отправка письма с confirmation_code
     'token' - проверка username и conf_code и возврат JWT токена.
    
    * подключены пакеты: 'rest_framework_simplejwt' и 'django_filters'
    Добавлены настройки REST_FRAMEWORK
    
    * убрал отладочный принт; вернул рандомную генерацию SECRET_KEY
    
    * исправления
    
    * удалили ненужные файлы; поправили README.md для прохождения тестов.
    
    * удалили ненужные файлы;
    
    * Добавил пагинацию и отрефакторил settings
    
    * Добавил фильтры и поиск ко всем эндпоинтам кроме User
    
    * вернул test_05_review.py в изначальное состояние!
    
    * Исправил поля при GET и POST запросах к Titles
    
    * рефактор регистрации через endpoint signup
    Исправил ошибку в сериализаторе!
    
    * рефактор модели User:
    переопределено поле username и добавлен валидатор регулярного выражения;
    добавлено сортировка по умолчанию(id);
    
    * рефактор сериализатора UserSerializer:
     - проверка поля на соответствие регулярному выражению;
     добавлен сериализатор для endpoint'а me.
    
    * рефактор UserSignupView:
     вернул рабочий вариант
    
    добавлены UserViewSet и UserMeView для endpoint'ов 'users/' и 'users/me/'
    
    зарегистрированы endpoint'ы
    
    * Пофиксил ошибку в валидации года записи
    
    * Добавил подсчет рейтинга произведения
    
    * фикс views.py забыл добавить generics.
    
    * Добавил пермишн
    
    * Исправил связь между views и сериализатором комментариев
    
    * закончили с юзерами (судя по тестам)
    
    * статический секрет для упрощения работы
    
    * Merge pull request #54
    
    * Фикс проблемы с повторными обращениями.
    
    * Фикс проблемы с повторными обращениями(снова!)
    
    * Пофиксил категории. Тесты проходят...
    
    * Ограничение PUT запроса для произведений
    
    * Сделал кастомный фильтр для Произведений
    
    * Исправил ошибку при удалении жанра
    
    * Сделал миксин проверки автора
    
    * Добаил пермишн автор-или-чтение
    
    * Исправил ошибку с полем author
    
    * Добавил пермишн на автора-или-чтение
    
    * README.md заполнен.
    
    * Исправил пермишн IsAuthorOrReadOnly
    
    * Переименовал модели Titles, Reviews, Comment
    
    * Удалил лишний код
    
    * Добавил ограничение уникальных обзоров для Review
    
    * Работает все, кроме комментариев. (74 из 80!)
    
    * Merge pull request #61
    
    * Комментарии работают (80 из 80!)
    
    * Уборка. Убрал лищние,на мой взгляд, докстринги.
    -Постарался убрать одноразовые переменные.
    -Добавил сортировки по умолчанию
    -Поправил описание полей в некоторых моделях
    - оптимизировал импорты(чтоб было компактнее)
    
    * корректировки по PEP8
    
    * Работающие миграции.
    
    * Merge pull request #68
    
    * Сделал константы
    
    * Merge pull request #67
    
    * Переписали модели. Почти ничего не сломали.
    
    * Merge pull request #69
    
    * добавлено требование авторизации по умолчанию для проекта!
    
    * зарегистрировал модели в админке;
    
    * вынес константы в settings.py (как изначально говорил Никита!)
    
    * константы берутся из settings
    
    * переработаны эндпойнты в соответствии с ревью
    
    * исправлена ошибка наследования для Meta;
    
    * добавлены методы проверки прав;
    
    * промежуточное сохранение сделаных permissions.py
    
    * Сделал все Permissions
    
    * Поправил базовые классы(забыли указать abstract=True!!!!)
    Прописал constraint для модели review
    
    * более технологично.
    
    * Merge pull request #63
    
    * Добавил managment-команду для импорта в БД из CSV
    
    * Merge branch 'pre_release' into feature/csv_parsing
    
    * Merge branch 'pre_release' into feature/csv_parsing
    
    * Реализовал функцию импорта в БД из CSV
    
    * Улучшил код импорта
    
    * Merge branch 'pre_release' into feature/csv_parsing
    
    * Обновил миграции
    
    * fix теста с юзерами
    
    * Реализовал валидацию score через валидатор поля модели Review
    
    * Все тесты проходят. Код выглядит приемлемо. (#72)
    
    * убрано то, что было велено убрать...
    
    * криво, но похоже на правду.
    
    * переместил приложения чтобы не использовать строки.
    
    * устранена часть замечаний по README.md
    
    * Переделал расчет рейтинга и удалил неиспользуемые импорты
    
    * рефактор моделей.
    
    * второй вариант импорта. Работает, но лучше написать свой.
    
    * Рефакторинг по PEP8
    
    * Merge pull request #73
    
    * Добавил managment-команду для импорта в БД из CSV
    
    * Merge branch 'pre_release' into feature/csv_parsing
    
    * Merge branch 'pre_release' into feature/csv_parsing
    
    * Реализовал функцию импорта в БД из CSV
    
    * Улучшил код импорта
    
    * Merge branch 'pre_release' into feature/csv_parsing
    
    * Merge branch 'pre_release' into feature/csv_parsing
    
    * Merge branch 'pre_release' into feature/csv_parsing
    
    * Переписал команду импорта CSV в БД
    
    * убрали ненужные принты.
    
    * правки перед ревью;
    
    * вынесли стэк вверх;
    
    * fix ошибки из ревью - убрал ненужный параметр;
    
    * Зарефакторили сериализатор TitleSerializer согласно ревью
    
    * fix formating
    
    * Переделал permissions.py
    
    * переделал сирилизаторы (#77)
    
    * Переделал serializers.py
    
    * Исправил недочет
    
    * Исправил urls.py
    
    * Переименовал модели
    
    * поправлены все замечания в части пользователей (#78)
    
    * * вынесены генерация и проверка pincode в отдельный модуль(utils);
    * pincode одноразовый. При попытке авторизации, вне зависимости от результата - пинкод меняется;
    * скорректирован пермишн AdminOnly;
    flake8=ok, migrations=ok, tests=ok;
    
    * исправлены замечания;
    
    * исправлен запрос к бд;
    
    * исправил несколько ошибок
    
    * Исправил models.py
    
    * Исправили views
    
    * приделал лютый костыль!
    
    * приделал лютый костыль! и поправил flake8
    
    * исправил ошибку в серилизаторе
    
    * исправил ошибку в urls
    
    * исправил ошбики в models
    
    * исправил ошибку в validators.py
    
    * Merge pull request #82
    
    * фикс проблемы с проверкой во вьюхе;
    
    * добавил контроль длины пинкода;
    
    * поправил дублирование; Осталось 2 TODO
    
    * исправлено падение на версии python3.8;
    
    * исправлено замечание с related_name;
    
    * cleanup;
    
    * добавили валидацию username;
    
    * решен вопрос с регуляркой;
    
    * flake8 fixes;
    
    ---------
    
    Co-authored-by: default avatarARLIKIN <nik5552000@mail.ru>
    Co-authored-by: default avatarSergey Kulbida <kulbida.sergey@outlook.com>
    Co-authored-by: default avatarSergey Kulbida <101509005+SergeyKDEV@users.noreply.github.com>
    Co-authored-by: default avatarARLIKIN <38999833+ARLIKIN@users.noreply.github.com>
    83054bc7
    исправлены замечания после 3-го ревью; (#83)
    Goerge Egiazaryan authored
    
    
    * add review and api apps
    
    * Сделал модель для произведений
    
    * Исправил ошибку с максимальной длинной рейтинга
    
    * Добавил модель для категорий
    
    * переместил приложения reviews и api на правильный уровень
    
    * подключены приложения reviews, api. Выполнена базовая конфигурация проекта.
    
    * Добавил файл сериализатора
    
    * Добавил докстринги, сделал роутер для апи, зарегестрировал эндпоинт для произведений
    
    * Добавил модель для жанров
    
    * добавил приписку v1 в urls api
    
    * Добавил ViewSet для категорий
    
    * Добавил эндпойнты для категорий
    
    * Связал модель произведений с жанрами и категориями
    
    * Переделал связанное удаление
    
    * + добавлена модель Пользователя и подключена в settings.py
    + исправлена ошибка с max_length для SmallIntegerField
    
    * Связал модель произведений с пользователем
    
    * Переделал связть в Title
    
    * Переделал связь в Title
    
    * Добавил эндпоинт для жанра
    
    * Добавил модели для отзывов и коментариев так же сделал эндпоинты
    
    * Переделал эндпоинты и связь в модели Comment
    
    * написан(частично) сериализатор для пользователей
    добавлены 2 константы в модель пользователя.
    
    * Провёл рефактор
    
    * добавлено: emailbackend, simple_JWT setting. Подключен DRF
    
    * Добавил view и сериализатор для Categories
    
    * Добавил views и сериализатор для Genres
    
    * Исправил dockstring
    
    * сделал серрилизатор и view для произведений
    
    * Добавил базовую структуру обзоров
    
    * Исправил импорт модели с Genre на Genres для сериализатора API
    
    * исправил баг в серрилизаторе произведений
    
    * Фикс urls и serializers для Reviews
    
    * Добавил сиррилизатор для комментариев
    
    * Исправил докстринги
    
    * Добавил валидацию оценки Reviews
    
    * реализована работа эндпойнтов signup и token:
     'signup' - регистрация/отправка письма с confirmation_code
     'token' - проверка username и conf_code и возврат JWT токена.
    
    * подключены пакеты: 'rest_framework_simplejwt' и 'django_filters'
    Добавлены настройки REST_FRAMEWORK
    
    * убрал отладочный принт; вернул рандомную генерацию SECRET_KEY
    
    * исправления
    
    * удалили ненужные файлы; поправили README.md для прохождения тестов.
    
    * удалили ненужные файлы;
    
    * Добавил пагинацию и отрефакторил settings
    
    * Добавил фильтры и поиск ко всем эндпоинтам кроме User
    
    * вернул test_05_review.py в изначальное состояние!
    
    * Исправил поля при GET и POST запросах к Titles
    
    * рефактор регистрации через endpoint signup
    Исправил ошибку в сериализаторе!
    
    * рефактор модели User:
    переопределено поле username и добавлен валидатор регулярного выражения;
    добавлено сортировка по умолчанию(id);
    
    * рефактор сериализатора UserSerializer:
     - проверка поля на соответствие регулярному выражению;
     добавлен сериализатор для endpoint'а me.
    
    * рефактор UserSignupView:
     вернул рабочий вариант
    
    добавлены UserViewSet и UserMeView для endpoint'ов 'users/' и 'users/me/'
    
    зарегистрированы endpoint'ы
    
    * Пофиксил ошибку в валидации года записи
    
    * Добавил подсчет рейтинга произведения
    
    * фикс views.py забыл добавить generics.
    
    * Добавил пермишн
    
    * Исправил связь между views и сериализатором комментариев
    
    * закончили с юзерами (судя по тестам)
    
    * статический секрет для упрощения работы
    
    * Merge pull request #54
    
    * Фикс проблемы с повторными обращениями.
    
    * Фикс проблемы с повторными обращениями(снова!)
    
    * Пофиксил категории. Тесты проходят...
    
    * Ограничение PUT запроса для произведений
    
    * Сделал кастомный фильтр для Произведений
    
    * Исправил ошибку при удалении жанра
    
    * Сделал миксин проверки автора
    
    * Добаил пермишн автор-или-чтение
    
    * Исправил ошибку с полем author
    
    * Добавил пермишн на автора-или-чтение
    
    * README.md заполнен.
    
    * Исправил пермишн IsAuthorOrReadOnly
    
    * Переименовал модели Titles, Reviews, Comment
    
    * Удалил лишний код
    
    * Добавил ограничение уникальных обзоров для Review
    
    * Работает все, кроме комментариев. (74 из 80!)
    
    * Merge pull request #61
    
    * Комментарии работают (80 из 80!)
    
    * Уборка. Убрал лищние,на мой взгляд, докстринги.
    -Постарался убрать одноразовые переменные.
    -Добавил сортировки по умолчанию
    -Поправил описание полей в некоторых моделях
    - оптимизировал импорты(чтоб было компактнее)
    
    * корректировки по PEP8
    
    * Работающие миграции.
    
    * Merge pull request #68
    
    * Сделал константы
    
    * Merge pull request #67
    
    * Переписали модели. Почти ничего не сломали.
    
    * Merge pull request #69
    
    * добавлено требование авторизации по умолчанию для проекта!
    
    * зарегистрировал модели в админке;
    
    * вынес константы в settings.py (как изначально говорил Никита!)
    
    * константы берутся из settings
    
    * переработаны эндпойнты в соответствии с ревью
    
    * исправлена ошибка наследования для Meta;
    
    * добавлены методы проверки прав;
    
    * промежуточное сохранение сделаных permissions.py
    
    * Сделал все Permissions
    
    * Поправил базовые классы(забыли указать abstract=True!!!!)
    Прописал constraint для модели review
    
    * более технологично.
    
    * Merge pull request #63
    
    * Добавил managment-команду для импорта в БД из CSV
    
    * Merge branch 'pre_release' into feature/csv_parsing
    
    * Merge branch 'pre_release' into feature/csv_parsing
    
    * Реализовал функцию импорта в БД из CSV
    
    * Улучшил код импорта
    
    * Merge branch 'pre_release' into feature/csv_parsing
    
    * Обновил миграции
    
    * fix теста с юзерами
    
    * Реализовал валидацию score через валидатор поля модели Review
    
    * Все тесты проходят. Код выглядит приемлемо. (#72)
    
    * убрано то, что было велено убрать...
    
    * криво, но похоже на правду.
    
    * переместил приложения чтобы не использовать строки.
    
    * устранена часть замечаний по README.md
    
    * Переделал расчет рейтинга и удалил неиспользуемые импорты
    
    * рефактор моделей.
    
    * второй вариант импорта. Работает, но лучше написать свой.
    
    * Рефакторинг по PEP8
    
    * Merge pull request #73
    
    * Добавил managment-команду для импорта в БД из CSV
    
    * Merge branch 'pre_release' into feature/csv_parsing
    
    * Merge branch 'pre_release' into feature/csv_parsing
    
    * Реализовал функцию импорта в БД из CSV
    
    * Улучшил код импорта
    
    * Merge branch 'pre_release' into feature/csv_parsing
    
    * Merge branch 'pre_release' into feature/csv_parsing
    
    * Merge branch 'pre_release' into feature/csv_parsing
    
    * Переписал команду импорта CSV в БД
    
    * убрали ненужные принты.
    
    * правки перед ревью;
    
    * вынесли стэк вверх;
    
    * fix ошибки из ревью - убрал ненужный параметр;
    
    * Зарефакторили сериализатор TitleSerializer согласно ревью
    
    * fix formating
    
    * Переделал permissions.py
    
    * переделал сирилизаторы (#77)
    
    * Переделал serializers.py
    
    * Исправил недочет
    
    * Исправил urls.py
    
    * Переименовал модели
    
    * поправлены все замечания в части пользователей (#78)
    
    * * вынесены генерация и проверка pincode в отдельный модуль(utils);
    * pincode одноразовый. При попытке авторизации, вне зависимости от результата - пинкод меняется;
    * скорректирован пермишн AdminOnly;
    flake8=ok, migrations=ok, tests=ok;
    
    * исправлены замечания;
    
    * исправлен запрос к бд;
    
    * исправил несколько ошибок
    
    * Исправил models.py
    
    * Исправили views
    
    * приделал лютый костыль!
    
    * приделал лютый костыль! и поправил flake8
    
    * исправил ошибку в серилизаторе
    
    * исправил ошибку в urls
    
    * исправил ошбики в models
    
    * исправил ошибку в validators.py
    
    * Merge pull request #82
    
    * фикс проблемы с проверкой во вьюхе;
    
    * добавил контроль длины пинкода;
    
    * поправил дублирование; Осталось 2 TODO
    
    * исправлено падение на версии python3.8;
    
    * исправлено замечание с related_name;
    
    * cleanup;
    
    * добавили валидацию username;
    
    * решен вопрос с регуляркой;
    
    * flake8 fixes;
    
    ---------
    
    Co-authored-by: default avatarARLIKIN <nik5552000@mail.ru>
    Co-authored-by: default avatarSergey Kulbida <kulbida.sergey@outlook.com>
    Co-authored-by: default avatarSergey Kulbida <101509005+SergeyKDEV@users.noreply.github.com>
    Co-authored-by: default avatarARLIKIN <38999833+ARLIKIN@users.noreply.github.com>
To learn more about this project, read the wiki.
Loading