admin/family_budget
1
0
Fork 1
Code Issues 1 Pull Requests Actions Packages Projects Releases Wiki Activity

Исправить валидацию фильтров аналитики #38

Merged
admin merged 1 commits from fix/analytics-query-validation into main 2026-08-24 19:09:49 +00:00
Conversation 2 Commits 1 Files Changed 3 +38 -1
agent commented 2026-08-24 19:06:48 +00:00
Collaborator
Copy Link

Некорректные accountId/categoryId теперь возвращают 400 до выполнения SQL. categoryId=0 остаётся корректным значением для фильтра «Без категории».

Проверки: npm run build -w backend; npm run test:analytics -w backend; npm run test:analytics:query -w backend.

Интеграционная SQL-проверка пока блокирована недоступностью PostgreSQL (тайм-аут подключения).

Некорректные accountId/categoryId теперь возвращают 400 до выполнения SQL. categoryId=0 остаётся корректным значением для фильтра «Без категории». Проверки: npm run build -w backend; npm run test:analytics -w backend; npm run test:analytics:query -w backend. Интеграционная SQL-проверка пока блокирована недоступностью PostgreSQL (тайм-аут подключения).
agent added 1 commit 2026-08-24 19:06:48 +00:00
fix: validate analytics filter ids 8939843462
agent commented 2026-08-24 19:08:08 +00:00
Author
Collaborator
Copy Link

Независимое ревью завершено: блокирующих замечаний нет. Валидация корректно отклоняет нечисловые, дробные и составные ID до SQL, сохраняя categoryId=0 для «Без категории». Проверены: сборка backend, test:analytics, test:analytics:query — успешно.

Независимое ревью завершено: блокирующих замечаний нет. Валидация корректно отклоняет нечисловые, дробные и составные ID до SQL, сохраняя `categoryId=0` для «Без категории». Проверены: сборка backend, `test:analytics`, `test:analytics:query` — успешно.
admin merged commit 29c4acd0a9 into main 2026-08-24 19:09:49 +00:00
agent commented 2026-08-24 19:15:12 +00:00
Author
Collaborator
Copy Link

Повторное независимое ревью полного PR после commit 69f2cc8 завершено: блокирующих замечаний нет. Исходная валидация не регрессировала, CHANGELOG корректно отражает Backend 0.10.2 и пользовательски значимое исправление. Повторно успешны: сборка backend, test:analytics, test:analytics:query.

Повторное независимое ревью полного PR после commit 69f2cc8 завершено: блокирующих замечаний нет. Исходная валидация не регрессировала, CHANGELOG корректно отражает Backend 0.10.2 и пользовательски значимое исправление. Повторно успешны: сборка backend, `test:analytics`, `test:analytics:query`.
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
agent admin
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: admin/family_budget#38
Delete Branch "fix/analytics-query-validation"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?