Skip to content

Conversation

@petrCher
Copy link
Member

@petrCher petrCher commented Nov 27, 2025

Изменения

Раньше в additional_data могла поступить строка, что не является корректным, так как сервисы в marketing api передают данные формата json в additional_data.

Детали реализации

Теперь есть проверка на формат json, все протестировано и корректно работает. Также поменял в models поля на mapped_column так как это в sqlalchemy более современно и правильно.

Check-List

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

@github-actions
Copy link

github-actions bot commented Nov 27, 2025

Code Coverage

Coverage Report
FileStmtsMissCoverMissing
marketing_api
   __main__.py440%1–7
marketing_api/models
   db.py31197%59
marketing_api/routes
   base.py61493%77, 81, 88, 93
TOTAL140994% 

Summary

Tests Skipped Failures Errors Time
5 0 💤 0 ❌ 0 🔥 0.459s ⏱️

@petrCher
Copy link
Member Author

additional_data: dict | list[dict] | None = None все-таки это удобнее у быстрее варианта использования json из pydantic, тем более что в нем придется все равно задавать dict | list[dict]

@Zimovchik Zimovchik merged commit 43e666c into main Dec 2, 2025
2 checks passed
@Zimovchik Zimovchik deleted the json_correct branch December 2, 2025 09:07
@Zimovchik Zimovchik restored the json_correct branch December 2, 2025 09:08
@Zimovchik Zimovchik deleted the json_correct branch December 2, 2025 09:08
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.

Часть записей в marketing api отдает не jsonы в additional data

3 participants