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

Removed the video_api module and everything related to it #552

Merged
merged 1 commit into from
Oct 1, 2024

Conversation

Toksi86
Copy link
Contributor

@Toksi86 Toksi86 commented Sep 27, 2024

Description

Удалил модуль video_api и связанный с ним код.

Type of change

  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

How Has This Been Tested?

  • Произвёл тестирование проекта при помощи Pytest;
  • Запустил проект на локальной машине.

Checklist:

  • Мой код соответствует code-style данного проекта
  • Я провел самоанализ собственного кода

Copy link
Member

@KonstantinRaikhert KonstantinRaikhert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Пробовал тестировать? Создавать обьекты, ошибки не ловишь?

@Toksi86
Copy link
Contributor Author

Toksi86 commented Sep 29, 2024

Пробовал тестировать? Создавать обьекты, ошибки не ловишь?

Запускал только те тесты, которые имеются в проекте. Сам руками не пробовал создавать объекты, но намёк ясен, попробую.

Copy link
Member

@KonstantinRaikhert KonstantinRaikhert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Вроде все работает. Но на всякий случай посмотри сам, если ок. То мержи)

@Toksi86
Copy link
Contributor Author

Toksi86 commented Oct 1, 2024

Вроде все работает. Но на всякий случай посмотри сам, если ок. То мержи)

Проверил в ручную CRUD функции из разделов: "Игроки; Пользователи; Команды; Персонал команд; Соревнования; Аналитика; Выгрузки; Игры" и выгрузку информации. Все работает. Сейчас попробую сделать merge

@Toksi86 Toksi86 merged commit 2ff17bf into dev Oct 1, 2024
3 checks passed
@KonstantinRaikhert KonstantinRaikhert deleted the bugfix/remove-video-api-module branch October 11, 2024 07:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Демонтировать приложение video api и все с ним связанное.
2 participants