Skip to content

portfolio-trading-accounts upsert#584

Merged
Aboruhen merged 7 commits intomasterfrom
investment-service/phase-2-data-models-ingestion-trading-account
Feb 25, 2026
Merged

portfolio-trading-accounts upsert#584
Aboruhen merged 7 commits intomasterfrom
investment-service/phase-2-data-models-ingestion-trading-account

Conversation

@apawan
Copy link
Contributor

@apawan apawan commented Feb 20, 2026

Description

Investment:

  • Implement Portfolio Trading Account Creation

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.

@apawan apawan requested a review from a team as a code owner February 20, 2026 07:07
Copy link
Collaborator

@Aboruhen Aboruhen left a comment

Choose a reason for hiding this comment

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

LGTM

@Aboruhen Aboruhen changed the base branch from investment-service/phase-2-data-models-ingestion to master February 24, 2026 10:20
Copy link
Collaborator

@Aboruhen Aboruhen left a comment

Choose a reason for hiding this comment

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

Update based on the latest version

@apawan apawan closed this Feb 24, 2026
@apawan apawan reopened this Feb 24, 2026
@apawan apawan force-pushed the investment-service/phase-2-data-models-ingestion-trading-account branch from a7c4648 to aedb524 Compare February 24, 2026 13:23
@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed

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

See analysis details on SonarQube Cloud

@Aboruhen Aboruhen merged commit 6880719 into master Feb 25, 2026
5 of 6 checks passed
@Aboruhen Aboruhen deleted the investment-service/phase-2-data-models-ingestion-trading-account branch February 25, 2026 14:03
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.

2 participants