antonio-hickey / antonio-hickey/tradestation-rs

Expose more `accounting::accounts` structs & enums in the `accounting` module.

Open
#84 0 comments 0 reactions 0 assignees View on GitHub
Accounting Difficulty: Easy Enhancement Good First Issue Help Wanted Priority: Low Status: Not Started
Dominant language
Rust
Stars
14
Forks
8
PR merge metrics
No merged PRs in 30d

Description

One obvious one I just ran into is the account type enum which requires me to fully resolve with:
```rust
use tradestation::accounting::accounts::AccountType;
```
Really we should at least cut out the `accounts` path requirement, so it's just:
```rust
use tradestation::accounting::AccountType;
```

There's likely other similar scenarios in this module? Please cover all which make sense if there's more.

https://github.com/antonio-hickey/tradestation-rs/blob/9b60f13218a698830e1a7159d801693735e6f5b3/src/accounting/mod.rs#L327

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.