Skip to content

Feat/investment product images tar 797#610

Open
Aboruhen wants to merge 11 commits into
masterfrom
feat/investment-product-images-TAR-797
Open

Feat/investment product images tar 797#610
Aboruhen wants to merge 11 commits into
masterfrom
feat/investment-product-images-TAR-797

Conversation

@Aboruhen
Copy link
Copy Markdown
Collaborator

@Aboruhen Aboruhen commented Jun 3, 2026

Description

Add implementation to ingest investment product portfolio image icons

Checklist

  • I made sure, I read CONTRIBUTING.md to put right branch prefix as per my need.
  • I made sure to update CHANGELOG.md.
  • I made sure to update Stream Wiki(only valid in case of new stream module or architecture changes).
  • My changes are adequately tested.
  • I made sure all the SonarCloud Quality Gate are passed.

Roman Kniazevych and others added 9 commits June 2, 2026 11:56
- Bump investment-service-api.version to 1.6.0 in investment-core/pom.xml
- Fix InvestmentPortfolioService: update InvestorModelPortfolio (7-arg) and
  PortfolioProduct (10-arg) constructors; add two trailing nulls to
  listPortfolioProducts call for new 1.6.0 parameters
- Fix test fixtures: 7-arg InvestorModelPortfolio in AllocationServiceTest;
  13-arg listPortfolioProducts stubs across 5 occurrences in ServiceTest
- All 429 unit tests pass (mvn clean install -pl stream-investment/investment-core -am)

Co-authored-by: Cursor <cursoragent@cursor.com>
…d one

define all Product portfolio properties
…d one

define new product setup insert & update
…d one

ingest product portfolio image icons
…d one

improve code definition; improve logging info.
…d one

upgrade investment-service version to 1.6.2
@Aboruhen Aboruhen requested a review from a team as a code owner June 3, 2026 09:54
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Jun 3, 2026

Quality Gate Failed Quality Gate failed

Failed conditions
49.6% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant