New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change ResolvedTrade to refer to PortfolioItemInfo #1724

Merged
merged 1 commit into from Jun 8, 2018

Conversation

Projects
None yet
2 participants
@jodastephen
Member

jodastephen commented Jun 8, 2018

Allows implementations to be positions

Change ResolvedTrade to refer to PortfolioItemInfo
Allows implementations to be positions

@jodastephen jodastephen requested a review from Mistuus Jun 8, 2018

@Mistuus

Mistuus approved these changes Jun 8, 2018

@jodastephen jodastephen merged commit 41c4b28 into master Jun 8, 2018

15 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
security/snyk - modules/basics/pom.xml No dependency changes
Details
security/snyk - modules/calc/pom.xml No dependency changes
Details
security/snyk - modules/collect/pom.xml No dependency changes
Details
security/snyk - modules/data/pom.xml No dependency changes
Details
security/snyk - modules/loader/pom.xml No dependency changes
Details
security/snyk - modules/market/pom.xml No dependency changes
Details
security/snyk - modules/math/pom.xml No dependency changes
Details
security/snyk - modules/measure/pom.xml No dependency changes
Details
security/snyk - modules/pom.xml No dependency changes
Details
security/snyk - modules/pricer/pom.xml No dependency changes
Details
security/snyk - modules/product/pom.xml No dependency changes
Details
security/snyk - modules/report/pom.xml No dependency changes
Details
security/snyk - pom.xml No dependency changes
Details

@jodastephen jodastephen deleted the topic/portfolio-item-info branch Jun 8, 2018

@jodastephen jodastephen added this to the v2.0 milestone Jun 12, 2018

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