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

Add product page title #175

Merged
merged 4 commits into from
Jan 10, 2020

Conversation

trojanh
Copy link
Contributor

@trojanh trojanh commented Jan 7, 2020

Resolves N.A.
Impact: minor
Type: bugfix

Issue

Product and Variants page does not show page titles.

Solution

Add page title for products page using Helmet.

Signed-off-by: trojanh <rajan.tiwari@kiprosh.com>
@willopez willopez self-requested a review January 9, 2020 00:49
Copy link
Member

@willopez willopez left a comment

Choose a reason for hiding this comment

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

Please sign your commit that updates the page title and remove the commit that updates how prices are updated, as that is part another PR.

Signed-off-by: trojanh <rajan.tiwari@kiprosh.com>
@trojanh
Copy link
Contributor Author

trojanh commented Jan 10, 2020

@willopez fixed the linting errors and signed the commits, thanks for pointing.

@trojanh trojanh force-pushed the add-product-page-title branch 2 times, most recently from 2397efd to 5e9a308 Compare January 10, 2020 13:50
Signed-off-by: trojanh <rajan.tiwari@kiprosh.com>
…nto add-product-page-title

Signed-off-by: trojanh <rajan.tiwari@kiprosh.com>
@willopez willopez merged commit 00ffb27 into reactioncommerce:trunk Jan 10, 2020
@kieckhafer kieckhafer mentioned this pull request Feb 6, 2020
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.

2 participants