Skip to content

Feature: Implementation Tastytrade Brokerage#8809

Merged
Martin-Molinero merged 2 commits into
QuantConnect:masterfrom
Romazes:feature-brokerage-tastytrade
Jun 4, 2025
Merged

Feature: Implementation Tastytrade Brokerage#8809
Martin-Molinero merged 2 commits into
QuantConnect:masterfrom
Romazes:feature-brokerage-tastytrade

Conversation

@Romazes

@Romazes Romazes commented Jun 2, 2025

Copy link
Copy Markdown
Collaborator

Description

Related PRs

Related Issue

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>

@Romazes Romazes requested a review from Martin-Molinero June 2, 2025 12:37
@Romazes Romazes self-assigned this Jun 2, 2025
@Romazes Romazes added the feature label Jun 2, 2025
test:feat: TastytradeFeeModel
refactor: curl brackets in namespaces to keep project style

@Martin-Molinero Martin-Molinero left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@Martin-Molinero Martin-Molinero merged commit d9bdbab into QuantConnect:master Jun 4, 2025
7 of 8 checks passed
@Romazes Romazes deleted the feature-brokerage-tastytrade branch July 14, 2025 16:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants