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.

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

Implement CharlesSchwab API (new Lean Plugin) #1

Open
wants to merge 106 commits into
base: master
Choose a base branch
from

Conversation

Romazes
Copy link

@Romazes Romazes commented Oct 24, 2024

Description

Related Issue

Related PRs:

Motivation and Context

Requires Documentation Change

How Has This Been Tested?

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • Refactor (non-breaking change which improves implementation)
  • Performance (non-breaking change which improves performance. Please add associated performance test and results)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Non-functional change (xml comments/documentation/etc)

Checklist:

  • My code follows the code style of this project.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.
  • My branch follows the naming convention bug-<issue#>-<description> or feature-<issue#>-<description>

feat: GetAuthorizationUrl
feat: Required property in CharlesSchwabAccessToken
feat: new model CharlesSchwabErrorResponse
test:feat: GetAuthorizationUrl
remove: CharlesSchwabAccountMetaData
test:feat: GetAccount with Mock
feat: starting impl Initialize
@Romazes Romazes added enhancement New feature or request good first issue Good for newcomers labels Oct 24, 2024
@Romazes Romazes self-assigned this Oct 24, 2024
refactor: CharlesSchwab Order Request
test:feat: validate Placing TrailingStopOrder
refactor: RequestAsync methods()
feat: Handle Cancel or Invalid stream order responses
feat: create enum for MessageType
refactor: ConversionRate and MarketPrice in GetAccountHoldings
feat: Handle Invalid Order Status when PlacedNewOrder
feat: Handle Exception in CancelOrder
feat: Handle Filled,Reject,Cancel OrderEvent
remove: Catch exception in Api PlaceOrder
feat: OrderFillCompleted in MessageType
feat: new AccountActivity entities
test:feat: Split test on Equity and Option
remove: support Future Symbol in SymbolMapper
test:refactor: trading test cases
feat: Custom Convert of Data Content
test:feat: DataQueueHandler
feat: new ErrorResponse in Api
test:refactor: new initializer of Brokerage in TestSetup
feat: SetJob
refactor: add missed xml description
test:feat: Update Order Equity\Option
feat: new MessageTypes
fix: IsOrderOpen extension
fix: handle order exception in ApiClient
refactor: skip duplicate order event
refactor: make error exception more developer friendly
feat: support Place IndexOption
test:feat: IndexOption (Index doesn't support)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant