-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Estimated delivery metadata #72
Conversation
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.
Awesome! just a couple small things.
@@ -116,6 +126,16 @@ internal final class RewardCell: UITableViewCell, ValueCell { | |||
|> UILabel.lens.textColor .~ .ksr_text_navy_500 | |||
|> UILabel.lens.numberOfLines .~ 0 | |||
|
|||
_ = self.estimatedDeliveryLabel | |||
|> UILabel.lens.text %~ { _ in | |||
localizedString(key: "Estimated_delivery", defaultValue: "Estimated delivery:") } |
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.
can swap this with the Strings
version of this string.
Library/Strings.swift
Outdated
@@ -8330,7 +8330,7 @@ catch your eye?" | |||
/** | |||
"%{backers_count} backers total, %{percent_funded} funded, %{time_left} to go" | |||
|
|||
- **es**: "De momento tiene %{backers_count} patrocinadores, financiado al %{percent_funded}, tiempo restante: %{time_left}" | |||
- **es**: "De momento tiene %{backers_count} patrocinadorefs, financiado al %{percent_funded}, tiempo restante: %{time_left}" |
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.
this looks like an accident. you should revert this change. this file is generated automatically.
|| userIsBacking(reward: reward, inProject: project) | ||
.map { project, reward in | ||
reward.remaining != 0 | ||
|| userIsBacking(reward: reward, inProject: project) |
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 think you could set this spacing back to what it was?
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.
Looks good @cdolm92! Made two small comments :)
@@ -15,6 +15,10 @@ internal final class BackingViewController: UIViewController { | |||
@IBOutlet fileprivate weak var backerSequenceLabel: UILabel! | |||
@IBOutlet fileprivate weak var backerShippingAmountLabel: UILabel! | |||
@IBOutlet fileprivate weak var backerShippingDescriptionLabel: UILabel! | |||
@IBOutlet fileprivate weak var estimatedDeliveryDateLabel: UILabel! | |||
@IBOutlet fileprivate weak var estimatedDeliveryLabel: UILabel! | |||
@IBOutlet fileprivate weak var estimatedDeliverySeperatorView: UIView! |
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.
Spotted a couple misspellings of separator.
self.vm.inputs.viewDidLoad() | ||
|
||
self.estimatedDeliveryDateLabelText.assertValues(["July 2016"], | ||
"Emits estimated delivery date") |
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 think the comment should be in line with the first argument:
self.estimatedDeliveryDateLabelText.assertValues(["July 2016"],
"Emits estimated delivery date")
@kickstarter/native-squad g2g? |
Gonna merge this is cause realize the strings updated and so we'll get screenshot failures in other branches. |
WHAT
Estimated delivery metadata now appears in Reward Cells and Backing Screen.
Reward Cell
Backing Screen