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

Wrong filling type when closing orders in MT5 #664

Closed
kenorb opened this issue Dec 19, 2022 · 1 comment
Closed

Wrong filling type when closing orders in MT5 #664

kenorb opened this issue Dec 19, 2022 · 1 comment
Labels
bug high-pri High priority
Milestone

Comments

@kenorb
Copy link
Member

kenorb commented Dec 19, 2022

Refs: EA31337/EA31337#349

@kenorb kenorb added the bug label Dec 19, 2022
@kenorb kenorb added this to the v2.013 milestone Dec 19, 2022
@kenorb kenorb added the high-pri High priority label May 20, 2023
@kenorb
Copy link
Member Author

kenorb commented May 21, 2023

Fixed in v2.012.2.

@kenorb kenorb closed this as completed May 21, 2023
kenorb added a commit that referenced this issue Jun 17, 2023
* origin/v3.001-dev:
  Disables TradeSignalManager.test.cpp test due to errors
  Generic ArraySize() now takes reference to constant.
  Trade: Improves performance issues
  Order: Adds missing order filling (fixes GH-664)
  Should fix problem "'array' - constant variable cannot be passed as reference" in Array.mqh.
  Made Dict and DictObject to have public Serialize() and SerializeStub() methods.
kenorb added a commit that referenced this issue Apr 27, 2024
* tag 'v2.012.2':
  Order: Adds missing order filling (fixes GH-664)
kenorb added a commit that referenced this issue Apr 27, 2024
* v3.002-dev:
  Fixes Std.h(54,9) : warning 30: macro redefinition
  StrategyAdd: Pass EA ptr to Init()
  Trade: Improves performance issues
  Order: Adds missing order filling (fixes GH-664)
  Fixes pointer syntax for C++
  EA/Strategy/Trade: AddTask: Fixes code syntax to avoid issues with invalid pointers
  EA/Strategy/Trade: Adds AddTaskObject()
  Trade: Disables filling modes for MQL4
  Indicator: Fixes timeframe assignment in class constructor
  Removes Stats class
  Removes Ticker class
  Removes BasicTrade class
  Removes SVG class
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug high-pri High priority
Projects
None yet
Development

No branches or pull requests

1 participant