Продолжение 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`.
Независимое ревью 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.
Финальное независимое ревью 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 готов к слиянию.
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
agent
Продолжение Issue #43: API и UI показывают историю оценок портфеля по датам импортированных отчётов для брокерских и ИИС-счетов.
Проверки:
npm run build(shared, backend, frontend),npm run test:portfolio:overview.Независимое ревью PR #48 проведено по полному diff и контексту.
Блокирующее замечание:
Проверено:
Финальное независимое ревью PR #48 проведено повторно по полному актуальному diff, включая коммит
e0723b3. Предыдущее блокирующее замечание устранено: в CHANGELOG.md добавлена запись Frontend 0.14.0 / Backend 0.13.0 / Shared 0.8.0 с описанием динамики портфеля.Повторно проверено:
npm run buildдля shared, backend и frontend проходит.npm run test:portfolio:overviewпроходит (overview/history mapper).Блокирующих замечаний нет. PR готов к слиянию.