Skip to content

Comments

[MODORDERS-1403]. Enhance the holding-detail endpoint to always return all linked PO lines#1241

Merged
BKadirkhodjaev merged 16 commits intomasterfrom
MODORDERS-1403
Feb 17, 2026
Merged

[MODORDERS-1403]. Enhance the holding-detail endpoint to always return all linked PO lines#1241
BKadirkhodjaev merged 16 commits intomasterfrom
MODORDERS-1403

Conversation

@BKadirkhodjaev
Copy link
Contributor

@BKadirkhodjaev BKadirkhodjaev commented Feb 13, 2026

Purpose

Approach

  • Make POL, piece and item retrieval fully independent and work in parallel
  • Add traversal and aggregation by all affiliated tenants

@BKadirkhodjaev BKadirkhodjaev marked this pull request as ready for review February 16, 2026 13:57
@BKadirkhodjaev BKadirkhodjaev requested a review from a team as a code owner February 16, 2026 13:57
.map(v -> aggregator);
}

protected Future<List<String>> getUserTenantsIfNeeded(RequestContext requestContext) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We have similar method getUserTenantsIfNeeded in PieceStorageService, probably they can be combined for example in ConsortiumConfigurationService

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can improve like this

Image

.map(v -> aggregator);
}

protected Future<List<String>> getUserTenantsIfNeeded(RequestContext requestContext) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can improve like this

Image

@sonarqubecloud
Copy link

@BKadirkhodjaev BKadirkhodjaev merged commit df4b6b4 into master Feb 17, 2026
6 checks passed
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.

2 participants