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

feat: добавить динамику стоимости портфеля #48

Merged
admin merged 2 commits from feature/portfolio-history into main 2026-08-27 05:22:36 +00:00
Conversation 2 Commits 2 Files Changed 13 +133 -8
agent commented 2026-08-27 05:18:20 +00:00
Collaborator
Copy Link

Продолжение Issue #43: API и UI показывают историю оценок портфеля по датам импортированных отчётов для брокерских и ИИС-счетов.

Проверки: npm run build (shared, backend, frontend), npm run test:portfolio:overview.

Продолжение Issue #43: API и UI показывают историю оценок портфеля по датам импортированных отчётов для брокерских и ИИС-счетов. Проверки: `npm run build` (shared, backend, frontend), `npm run test:portfolio:overview`.
agent added 1 commit 2026-08-27 05:18:21 +00:00
feat: add portfolio valuation history 29ba82f4bf
agent reviewed 2026-08-27 05:19:40 +00:00
agent left a comment
Author
Collaborator
Copy Link

Независимое ревью PR #48 проведено по полному diff и контексту.

Блокирующее замечание:

  • В PR нет записи в CHANGELOG.md. По правилам репозитория каждый PR после прохождения проверок должен содержать пользовательски значимое изменение в changelog; для этой функциональности нужна отдельная запись с актуальными версиями.

Проверено:

  • SQL-агрегация суммирует valuation по каждому portfolio_reports и сортирует точки по account/date/report timestamps/id; выборка ограничена brokerage/iis и защищена общим requireAuth.
  • UI корректно показывает график только при двух и более точках, использует существующий Recharts и форматтеры; сборка shared/backend/frontend проходит.
  • Mapper-тест toPortfolioHistory проходит (npm run test:portfolio:overview).
  • Явных дефектов в дедупликации, утечек или регрессий не обнаружено. После добавления changelog необходимо повторить проверки и ревью полного PR.
Независимое ревью PR #48 проведено по полному diff и контексту. Блокирующее замечание: - В PR нет записи в CHANGELOG.md. По правилам репозитория каждый PR после прохождения проверок должен содержать пользовательски значимое изменение в changelog; для этой функциональности нужна отдельная запись с актуальными версиями. Проверено: - SQL-агрегация суммирует valuation по каждому portfolio_reports и сортирует точки по account/date/report timestamps/id; выборка ограничена brokerage/iis и защищена общим requireAuth. - UI корректно показывает график только при двух и более точках, использует существующий Recharts и форматтеры; сборка shared/backend/frontend проходит. - Mapper-тест toPortfolioHistory проходит (npm run test:portfolio:overview). - Явных дефектов в дедупликации, утечек или регрессий не обнаружено. После добавления changelog необходимо повторить проверки и ревью полного PR.
agent added 1 commit 2026-08-27 05:20:04 +00:00
docs: note portfolio history e0723b3ab6
agent reviewed 2026-08-27 05:21:23 +00:00
agent left a comment
Author
Collaborator
Copy Link

Финальное независимое ревью PR #48 проведено повторно по полному актуальному diff, включая коммит e0723b3. Предыдущее блокирующее замечание устранено: в CHANGELOG.md добавлена запись Frontend 0.14.0 / Backend 0.13.0 / Shared 0.8.0 с описанием динамики портфеля.

Повторно проверено:

  • SQL-выборка истории группирует valuation по каждому отчёту, детерминированно сортирует точки и ограничивает данные brokerage/iis; endpoint находится за requireAuth.
  • UI корректно отображает историю только при наличии нескольких точек и использует существующие форматтеры/компоненты.
  • npm run build для shared, backend и frontend проходит.
  • npm run test:portfolio:overview проходит (overview/history mapper).

Блокирующих замечаний нет. PR готов к слиянию.

Финальное независимое ревью PR #48 проведено повторно по полному актуальному diff, включая коммит e0723b3. Предыдущее блокирующее замечание устранено: в CHANGELOG.md добавлена запись Frontend 0.14.0 / Backend 0.13.0 / Shared 0.8.0 с описанием динамики портфеля. Повторно проверено: - SQL-выборка истории группирует valuation по каждому отчёту, детерминированно сортирует точки и ограничивает данные brokerage/iis; endpoint находится за requireAuth. - UI корректно отображает историю только при наличии нескольких точек и использует существующие форматтеры/компоненты. - `npm run build` для shared, backend и frontend проходит. - `npm run test:portfolio:overview` проходит (overview/history mapper). Блокирующих замечаний нет. PR готов к слиянию.
admin merged commit 4d9663a906 into main 2026-08-27 05:22:36 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
agent
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#48
Delete Branch "feature/portfolio-history"

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?