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

feat: added text overrides on CartItemDetail #450

Conversation

HarisSpahijaPon
Copy link
Contributor

Part of: #409
Impact: minor
Type: feature

Component

CartItemDetail now has overrides when i18next will be implemented (see #409)

Breaking changes

No breaking chances, all added fields are optional

Testing

  1. Add a new prop quantityText to <CartItemDetail />
  2. Add the value "test" to quantityText
  3. Label should display "test" when test is given as a value to the prop. When the prop quantityText is not added, it should default to the values given in default props

@HarisSpahijaPon HarisSpahijaPon changed the title feat: added text overrides on CartItem feat: added text overrides on CartItemDetail Nov 21, 2019
@HarisSpahijaPon HarisSpahijaPon force-pushed the feat-harisspahijapon-add-overrides-cartitemdetail branch from 6da0576 to 1222834 Compare November 21, 2019 09:09
/**
* Product title of chosen item.
*/
title: PropTypes.string
};

static defaultProps: {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

has to be "=" instead of :

Copy link
Contributor

@aldeed aldeed left a comment

Choose a reason for hiding this comment

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

Just the one issue you noted

…ithub.com:PonDigitalSolutions/reaction-component-library into feat-harisspahijapon-add-overrides-cartitemdetail
@rc-publisher
Copy link
Collaborator

🚀 Preview deployed

Built with commit 02017c5

https://deploy-preview-450--stoic-hodgkin-c0179e.netlify.com

@HarisSpahijaPon
Copy link
Contributor Author

done

@spencern spencern merged commit bbe80b0 into reactioncommerce:master Dec 9, 2019
@rc-publisher
Copy link
Collaborator

🎉 This PR is included in version 0.69.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@rc-publisher rc-publisher added the released Applied automatically by semantic-release label Dec 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
released Applied automatically by semantic-release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants