Skip to content

Add reports endpoints for phase 2.#187

Merged
ggyurchev merged 8 commits into
mainlinefrom
reports-phase-two
Jul 9, 2026
Merged

Add reports endpoints for phase 2.#187
ggyurchev merged 8 commits into
mainlinefrom
reports-phase-two

Conversation

@ggoranov-smar

Copy link
Copy Markdown
Contributor

Pull Request

Summary

Implements 6 report API endpoints (phase 2):

  • GET /reports/{reportId}/scope — list report scope (TokenPaginationResponse<ReportScopeAsset>)
  • GET /reports/{reportId}/columns — list report columns (TokenPaginationResponse<ReportColumn>)
  • GET /reports/{reportId}/columns/{columnVirtualId} — get single report column
  • PUT /reports/{reportId}/columns/{columnVirtualId} — update report column (new UpdateReportColumnRequest model; BaseResponseStatus-wrapped result response)
  • DELETE /reports/{reportId}/columns/{columnVirtualId} — delete report column
  • GET /reports/{reportId}/definition — get report definition

Checklist

  • Read the contribution guide
  • Successfully build your changes locally
  • Include a title that clearly describes your changes and references relevant issues / backlog items
  • Include a summary of your changes
  • Include tests for your changes where possible

@coveralls

coveralls commented Jun 15, 2026

Copy link
Copy Markdown

Coverage Report for CI Build 29020820228

Warning

No base build found for commit 48e6999 on mainline.
Coverage changes can't be calculated without a base build.
If a base build is processing, this comment will update automatically when it completes.

Coverage: 95.44%

Details

  • Patch coverage: 18 of 18 lines across 1 file are fully covered (100%).

Uncovered Changes

No uncovered changes found.

Coverage Regressions

Requires a base build to compare against. How to fix this →


Coverage Stats

Coverage Status
Relevant Lines: 1289
Covered Lines: 1244
Line Coverage: 96.51%
Relevant Branches: 246
Covered Branches: 221
Branch Coverage: 89.84%
Branches in Coverage %: Yes
Coverage Strength: 1750.47 hits per line

💛 - Coveralls

@ggyurchev
ggyurchev merged commit ed0bda5 into mainline Jul 9, 2026
5 checks passed
@ggyurchev
ggyurchev deleted the reports-phase-two branch July 9, 2026 14:22
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.

6 participants