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

v0.9.0 #66

Merged
merged 11 commits into from
Nov 4, 2024
Merged

v0.9.0 #66

merged 11 commits into from
Nov 4, 2024

Conversation

samjtro
Copy link
Member

@samjtro samjtro commented Oct 21, 2024 •
edited
Loading

#67

@samjtro
major:
63570f8 
- working on multi-leg orders
- removed redundant json struct instructions, changed some struct names to reflect naming convention of Schwab TraderAPI
- README updates, but i will probably reverse those at a later date
@samjtro samjtro self-assigned this Oct 21, 2024
@samjtro samjtro changed the title WIP: v0.9.0 v0.9.0 Oct 21, 2024
@samjtro samjtro added the wip label Oct 21, 2024
@samjtro samjtro marked this pull request as draft October 21, 2024 18:52
@samjtro
major:
2502992 
ref: #67, v0.9.0
desc:
- refactor utils.go to reflect removal of previous oauth implementation in favor of x/oauth2, vis-a-vis go-schwab/oauth2ns
- check -> isErrNil
@samjtro
major:
67452aa 
ref: #67, v0.9.0
- changed Initiate implementation to reflect go-schwab/oauth2ns changes
- refactored some misc functions
@samjtro
minor:
1853b83 
ref: #67, v0.9.0
desc:
- added findAllEnvFiles(), pass to Initiate() to allow any .env file name convention to work
@samjtro
major:
b1b86ac 
ref: #67, v0.9.0
desc:
- changed from using the fatal isErrNil function to returning the err if non-nil, allowing for better error management & enabling live trading
@samjtro
minor:
3d9d0ff 
desc:
- bump go deps
@samjtro
major:
f64c618 
ref: #67, v0.9.0
desc:
- added CONTRIBUTING.md, made more changes to README.md
@samjtro
major:
a9f3aad 
ref: #67, v0.9.0
desc:
- remove DB struct, in favor of native oauth2.Token struct, so that we don't have to do line-by-line assignment in Initiate
- random README.md updates
@samjtro
major:
eb1dc6a 
ref: #67, v0.9.0
desc:
- finally got x/oauth2 working
- moved token storage into cwd/.json
- refactored utils.go, removed unused funcs
- edited README, gitignore to reflect changes
github-advanced-security[bot]
github-advanced-security bot found potential problems Nov 4, 2024
View reviewed changes
utils.go Fixed Show fixed Hide fixed
@samjtro
minor:
3322c93 
ref: #67, v0.9.0
desc:
- correcting ci error
@samjtro
minor:
2c780f3 
ref: #67, v0.9.0
desc:
- README update
@samjtro samjtro marked this pull request as ready for review November 4, 2024 01:40
@samjtro samjtro merged commit f1a3128 into main Nov 4, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@samjtro samjtro

Labels
release wip
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@samjtro