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

571 restructuring db #585

Open
wants to merge 31 commits into
base: master
Choose a base branch
from
Open

571 restructuring db #585

wants to merge 31 commits into from

Conversation

necitboss
Copy link

@necitboss necitboss commented Jul 5, 2024

Основан на #580

Задача #571
Наверное, это самый веселый PR. Мне пришлось делать в той же ветке, что и первое задание по причине того, что не удалось решить проблему с тем, что проект из мастера не запускался в должном виде, По этой причине, в предыдущем PR у меня одна задача, а здесь сразу две.
Выполнено разделение db_methods и db_types на более мелкие файлы. Сейчас в папках db/types лежат файлы, содержащие по одному классу, которые были определены в db_types и один класс из db_methods. В db/methods же лежат функции из db_methods. Файлы логически разбиты на то, для чего они предназначаются, кроме одного. Из-за ошибки циклического импорта пришлось один файл вынести с функцией edit_user.
Также переименованы импорты этих функций и классов в тех файлах проекта, где они были импортированы.

@necitboss necitboss closed this Jul 5, 2024
@necitboss necitboss reopened this Jul 5, 2024
@necitboss necitboss linked an issue Jul 7, 2024 that may be closed by this pull request
@HadronCollider HadronCollider added the future_integration Интеграция/мерж в будущих релизах label Jul 22, 2024
@github-actions github-actions bot added the has conflicts if new merge has conflicts label Aug 26, 2024
@github-actions github-actions bot removed the has conflicts if new merge has conflicts label Aug 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
future_integration Интеграция/мерж в будущих релизах
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Реструктуризация модуля БД
2 participants