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

Add iOpen and iClose methods #28

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

Add iOpen and iClose methods #28

wants to merge 2 commits into from

Conversation

lazou
Copy link
Owner

@lazou lazou commented Apr 11, 2023

This PR will add the following two methods:

Based on a request in vdemydiuk#278

@lazou
Copy link
Owner Author

lazou commented Apr 11, 2023

@zemuss Please check if this is working for you. At least in the test client it worked on my end.

@zemuss
Copy link

zemuss commented Apr 19, 2023

I need to convert to this client for MT5 and test...

@zemuss
Copy link

zemuss commented Apr 19, 2023

@lazou I dont see it listed in the MtAPi5Client I pulled latest code.

@lazou
Copy link
Owner Author

lazou commented Apr 19, 2023

Did you used this branch for pulling the code? In the testclient ui both methods are available.

@lazou
Copy link
Owner Author

lazou commented Apr 19, 2023

Maybe some hints for using my fork. This fork is targeting .Net core. You have to use mtapi5.dll and mtapiservicenetcore.dll on your Client side. I recommend the portable "installation" into metatrader (see readme).

@zemuss
Copy link

zemuss commented Apr 19, 2023

NVM - I figured out how to get the branch.. I will test it tonight..

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants