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 CartItem #449

Conversation

HarisSpahijaPon
Copy link
Contributor

@HarisSpahijaPon HarisSpahijaPon commented Nov 21, 2019

Part of: #409
Impact: minor
Type: feature

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

Breaking changes
No breaking chances, all added fields are optional

Testing
Add a new prop messageText or totalText to <CartItem />
Add the value "test" to either messageText or totalText
Button or title should display "test" when test is given as a value to the prop. When the prop messageText or totalText is not added, it should default to the values given in default props

@HarisSpahijaPon HarisSpahijaPon force-pushed the feat-harisspahijapon-add-overrides-cartitem branch from 248e10a to 7889739 Compare November 21, 2019 08:56
@rc-publisher
Copy link
Collaborator

🚀 Preview deployed

Built with commit 2d74021

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

@rc-publisher
Copy link
Collaborator

rc-publisher commented Nov 21, 2019

🚀 Preview deployed

Built with commit fab85cb

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

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.

One eslint issue to resolve

Signed-off-by: Haris Spahija <[email protected]>
@HarisSpahijaPon HarisSpahijaPon force-pushed the feat-harisspahijapon-add-overrides-cartitem branch from a5c24e6 to fab85cb Compare December 9, 2019 14:12
@spencern spencern merged commit 9eb3fdc 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