Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Отрефакторить ReportTables #21

Closed
vananiev opened this issue Dec 10, 2021 · 1 comment · Fixed by #27
Closed

Отрефакторить ReportTables #21

vananiev opened this issue Dec 10, 2021 · 1 comment · Fixed by #27
Assignees
@vananiev
Labels
improvement Imrovement of existing feature minor Priority
Milestone
2022.2

Comments

@vananiev
Copy link
Contributor

vananiev commented Dec 10, 2021 •
edited
Loading

Заменить

ReportTable<SecurityTransaction> getSecurityTransactionTable();
ReportTable<DerivativeTransaction> getDerivativeTransactionTable();
ReportTable<ForeignExchangeTransaction> getForeignExchangeTransactionTable();

одним методом

ReportTable<AbstractTransaction> getTransactionTable();

И

ReportTable<SecurityEventCashFlow> getCouponAmortizationRedemptionTable();
ReportTable<SecurityEventCashFlow> getDividendTable();
ReportTable<SecurityEventCashFlow> getDerivativeCashFlowTable();

методом

ReportTable<SecurityEventCashFlow> getSecurityCashFlowTable();
@vananiev
Copy link
Contributor Author

vananiev commented Jan 25, 2022 •
edited
Loading

Единый метод getSecurityCashFlowTable() требуется для spacious-team/investbook#404

@vananiev vananiev added improvement Imrovement of existing feature minor Priority labels Jan 25, 2022
@vananiev vananiev added this to the 2022.2 milestone Jan 25, 2022
@vananiev vananiev self-assigned this Jan 25, 2022
This was referenced Jan 25, 2022
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@vananiev vananiev

Labels
improvement Imrovement of existing feature minor Priority
Projects
None yet
Milestone
2022.2
Development

Successfully merging a pull request may close this issue.

Изменить интефейс ReportTables spacious-team/broker-report-parser-api
1 participant
@vananiev