-
Notifications
You must be signed in to change notification settings - Fork 25
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
API needs to be objective C compatible #141
Comments
I'm going to create an Obj-C ExampleApp this week which should address any of these quirks. |
I have pushed a basic example app for Objective-C that covers the calls above. They were already compatible with Objective-C but you just need to add parameter names in the right place. See here: https://github.com/tradingticket/TradeItIosTicketSDK2/blob/develop/ExampleAppObjC/ViewController.m Make sure you import the Objective-C header as well to get access to the Environments:
I also pulled out a |
My point was you add the parameter with your convention so others will just use xcode autofill and start working, without guessing what should I write there. |
Cosmetic, not a big deal. |
When you pass a block as an argument, you are defining the block so you will always have to name the parameters to your block. If you want XCode to add a placeholder when autocompleting block arguments, you'll need to file a Radar ticket with Apple: https://bugreport.apple.com. |
For example, the API in TradeItOrder
Give this error while writing ObjC code:-
Also, for this API
Please make sure NOT to set a default value for the parameter, otherwise, I am getting this error:-
The text was updated successfully, but these errors were encountered: