Skip to content

Conversation

@justanothercatgirl
Copy link
Contributor

@justanothercatgirl justanothercatgirl commented Nov 3, 2025

Важно

Пока что не мёржить! надо на синке обсудить, что делать с файлом миграции, который я добавил (чё там с типами данных), и мне нужно дописать definitions перед тем, как я смогу раскомменить одну строчку)

Так что пока что это premature pull request (ppr)

Изменения

Слишком много писать. Вкратце - откопал в исходнике sqlalchemy функцию, которая позволился избавиться от SensitiveBase и реализовать модицикацию __table_args__.info через декораторы.

Check-List

  • Вы проверили свой код перед отправкой запроса?
  • Вы написали тесты к реализованным функциям?
  • Вы не забыли применить форматирование black и isort для Back-End или Prettier для Front-End?
  • не помню...

@github-actions
Copy link

github-actions bot commented Nov 3, 2025

💩 Code linting failed, use black and isort to fix it.

@github-actions
Copy link

github-actions bot commented Nov 3, 2025

Summary

Tests Skipped Failures Errors Time
1 0 💤 0 ❌ 0 🔥 22.869s ⏱️

@justanothercatgirl
Copy link
Contributor Author

Так, по порядку:

Добавил два файла миграции:

  1. Догоняет то, что сильно отстало от объявлений табличек dwh-definitions (в основном изменение типа колонок).
  2. Создаёт группы и выдаёт права для тех схем, про которые мы в прошлых миграциях забывали

Применил новое форматирование ко всему репу

В base.py добавил дектораторы @encrypted, @sensitive

В целом, это все изменения)

@justanothercatgirl justanothercatgirl changed the title Добавил декораторы для @sensitive и @encrypted Добавил декораторы для @sensitive и @encrypted + фикс отстающих миграций Nov 14, 2025
@justanothercatgirl
Copy link
Contributor Author

Пожалуйста, проверьте миграцию multicommit_fix_state на то, что происходит в БД реально. Потому что там слишком много всего, к чему у меня нет доступа.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Убрал EncryptedInfo т.к. в скором времени будет не нужна (наверное, нужно в pipelines даг тоже убрать, но он всё равно не включён у нас)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

В целом, в доке всё написано

Убрал SensitiveBase, вместо этого использовал декораторы, которые вызывают sqlalchemy.Table._init_existing чтобы изменить info на табличке и добавить туда нужные атрибуты

@justanothercatgirl
Copy link
Contributor Author

Блочится #187

@justanothercatgirl justanothercatgirl force-pushed the sotov branch 2 times, most recently from b7334cf to e2bd6e9 Compare December 2, 2025 18:31
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.

3 participants