-
Notifications
You must be signed in to change notification settings - Fork 24
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
No recipes support #19
Comments
Hey, Send us pull request or link to repo/commit and we'll see how to include it into wrapper. |
Hi, It is working fine, but i writen it to be compatible with upstream without need to readd new versions every time, btw, what I really not like with current implementation, that on every onapp upgrade my backend brokes, because of this case: 3.2, etc things, In my part of code in Recipes.php I tried to implement possible solution for this. |
also please check #16 it fixes serious problem with IP implementation. |
I started work on implementation due to the need for running recipes on vm's. A pull request will follow once we have our implementation working without issues in development: https://github.com/OnApp/OnApp-PHP-Wrapper-External/compare/3.5.0...Luceos:3.5.0?expand=1 |
There is one issue; the http code once a recipe has been scheduled is 207; the OnApp class (line 959+) does not see this as a valid response. |
@prado1991 Any comment on closing not implemented features ? |
No recipes support in API wrapper, I made one myself, but not sure that it could be pulled to your one.
The text was updated successfully, but these errors were encountered: