feat: Standardize Message model and update messages to match #1467
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1467 +/- ##
==========================================
+ Coverage 79.04% 79.16% +0.11%
==========================================
Files 172 172
Lines 5766 5764 -2
Branches 665 665
==========================================
+ Hits 4558 4563 +5
+ Misses 1054 1047 -7
Partials 154 154 ☔ View full report in Codecov by Sentry. |
|
codecov/patch is failing, but the new code doesn't add or meaningfully change the existing functions, it looks like backend/capellacollab/settings/modelsources/t4c/repositories/routes.py is just already missing some coverage. |
romeonicholas
force-pushed
the
standardize-messages
branch
from
6f53fce
to
6014155
Compare
|
MoritzWeber0
force-pushed
the
standardize-messages
branch
from
6014155
to
0f07c17
Compare
|
A Storybook preview is available for commit 005e7ab. |
|
The generated OpenAPI client is not up to date with the latest changes in the OpenAPI specification. |
MoritzWeber0
force-pushed
the
standardize-messages
branch
from
0f07c17
to
40488bc
Compare
|
The existing Message pydantic model used for warnings and errors was pretty loose and inconsistently used, this PR makes all fields required and updates all existing uses of the model.
I removed the techincal field because it was only used once or twice in places where exceptions were already being logged along with the warnings. err_code may also be unecessary, it's currently only being used in tests.