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
[TIMOB-13416] iOS:ListView editing support #4397
Conversation
- (NSDictionary *)itemAtIndex:(NSUInteger)index; | ||
- (void) deleteItemAtIndex:(NSUInteger)index; | ||
- (void) setItemAtIndex:(NSDictionary*)item withIndex:(NSUInteger)index; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not fully sure about the method name. Is the first argument an index, a dictionary with item and index? If it's just an item, why is index mentioned there? Also, it says set, not insert.
Comments left, but otherwise looks good. |
Code looks good from my side. went ahead and did FR .. looks all fine too me. Will wait till you can address blains comment to merge this in. |
Ready For Review |
Code Reviewed and APPROVED |
Functionally Tested and APPROVED |
[TIMOB-13416] iOS:ListView editing support
Fixes TIMOB-13416
Test using KS PR 105