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

Change price presentation by removing from prefix if needed #77

Merged
merged 2 commits into from
Feb 5, 2018

Conversation

radyslavkrechet
Copy link
Contributor

No description provided.

imagesArray.append(image)
}
let imagesNodes = item.images.edges.map({ $0.node })
var imagesArray = [Image]()
Copy link
Contributor

Choose a reason for hiding this comment

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

Change to var imagesArray: [Image] = []

images = imagesArray

let variantsNodes = item.variants.edges.map({ $0.node })
var variantsArray = [ProductVariant]()
Copy link
Contributor

Choose a reason for hiding this comment

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

See above

}

let optionsNodes = item.options
var optionsArray = [ProductOption]()
Copy link
Contributor

Choose a reason for hiding this comment

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

See above

available = item.availableForSale
image = Image(with: item.image) ?? Image(with: productImage)
let selectedOptions = item.selectedOptions
var selectedOptionsArray = [VariantOption]()
Copy link
Contributor

Choose a reason for hiding this comment

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

See above

@radyslavkrechet radyslavkrechet merged commit 350f93f into develop Feb 5, 2018
@radyslavkrechet radyslavkrechet deleted the feature/price_presentation branch February 5, 2018 11:41
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.

None yet

3 participants