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

Swift3 #671

Merged
merged 51 commits into from
Sep 26, 2016
Merged

Swift3 #671

merged 51 commits into from
Sep 26, 2016

Conversation

mtnbarreto
Copy link
Member

No description provided.

Denis Martin and others added 30 commits August 12, 2016 18:50
Fixed Unit Test (2 errors in HiddenRowsTests.swift which I cannot figure out)
…nableDeselection = false and user deselects a row)

Added enableDeselection customisation parameter to SelectorViewController
… size of the app. This only happens if the app uses `preferredFont:forTextStyle`. So no need to worry if you were using other fonts.
Operator fixes

Wip

Build errors fixed

Fixed KVO

Warning fixes

Additional warning fixes

Update Core.swift

Fixes with kvo for wrong casts

Fixed conversion to NSNumber in KVO

Removed development team
Fix predicate evaluation when values include Optional and or nil
Remove PostalAddressRow from project
mats-claassen and others added 6 commits September 16, 2016 14:05
@mtnbarreto mtnbarreto force-pushed the Swift3 branch 6 times, most recently from c44c588 to 7d68bad Compare September 21, 2016 23:05
Martin Barreto and others added 3 commits September 21, 2016 20:19
@mtnbarreto mtnbarreto added this to the 2.0.0 milestone Sep 22, 2016

Pull requests associated with this milestone can be found in this [filter](https://github.com/xmartlabs/Eureka/issues?utf8=%E2%9C%93&q=milestone%3A2.0.0%20).

We have made tons of changes in Eureka API to follow the new Swift API design guidelines.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tons of changes to the Eureka API

@@ -0,0 +1,53 @@
# Eureka 2.0 Migration Guide

Eureka 2.0.0 includes complete Swift 3 Compatibility and adopts the new [API Design Guidelines](https://swift.org/documentation/api-design-guidelines/). It also includes new validations build-in feature.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It also includes validations as a build-in features

# Eureka 2.0 Migration Guide

Eureka 2.0.0 includes complete Swift 3 Compatibility and adopts the new [API Design Guidelines](https://swift.org/documentation/api-design-guidelines/). It also includes new validations build-in feature.
Bring support to swift 3 involves some API updates to follow apple Swift API best practices, we have also changed and deprecated some API.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Apple's
deprecated some API's

Removed APIs:

* `PostalAddressRow` and `ImageRow` was deleted.
* row has been deleted. You can find it and many other custom rows at EurekaCommunity [organization account](https://github.com/eurekaCommunity).
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove row has been deleted.

Removed APIs:

* `PostalAddressRow` and `ImageRow` was deleted.
* row has been deleted. You can find it and many other custom rows at EurekaCommunity [organization account](https://github.com/eurekaCommunity).
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can find them and many other custom rows at EurekaCommunity's


`highlightCell` and `unhighlightCell` callbacks were deleted, now we should use `row.isHighlighted` from cell update to check from highlighted status and make UI modification according its value.

In case you want to do something when highligth state switches its value you can set up `onCellHighlightChanged` callback.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

highlight

button.setImage(UIImage(named: "checkedDay"), forState: .Selected)
button.setImage(UIImage(named: "uncheckedDay"), forState: .Normal)
button.setImage(UIImage(named: "checkedDay"), for: .selected)
button.setImage(UIImage(named: "uncheckedDay"), for: UIControlState())
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should be .normal

@@ -35,6 +35,7 @@
<string>UIInterfaceOrientationPortrait</string>
<string>UIInterfaceOrientationLandscapeLeft</string>
<string>UIInterfaceOrientationLandscapeRight</string>
<string>UIInterfaceOrientationPortraitUpsideDown</string>
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This line shouldn't have been added.

* row has been deleted. You can find it and many other custom rows at EurekaCommunity [organization account](https://github.com/eurekaCommunity).

`highlightCell` and `unhighlightCell` callbacks were deleted, now we should use `row.isHighlighted` from cell update to check from highlighted status and make UI modification according its value.
* `PostalAddressRow` and `ImageRow` was deleted. You can find them and many other custom rows at EurekaCommunity [organization account](https://github.com/eurekaCommunity).
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PostalAddressRow and ImageRow were deleted

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Where can I find these rows in EurekaCommunity?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

They are currently in progress. We will soon add them there. I believe the ImageRow has also been added to the Examples project.

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, I just noticed that while looking for examples to make my custom rows working again. Swift 3 really is a pain in the neck!

* validatesOnChangeAfterBlurred - Validates whenever the row value changes after it resigns first responder for the first time.
* validatesOnDemand - We should manually validate the row or form by invoking `validate()` method.

If you want to validate the entire form (all the rows) you can invoke Form `validate()` method.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you can invoke form.validate() is more clear

Martin Barreto added 2 commits September 26, 2016 17:46
Changing access level of keyboardWillShow and keyboardWillHide methods
@mtnbarreto mtnbarreto merged commit 3efe00e into master Sep 26, 2016
@mtnbarreto mtnbarreto deleted the Swift3 branch October 3, 2016 21:55
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.

10 participants