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

STRF-5521 PDP Product Reviews Link Not Clickable #1498

Merged
merged 1 commit into from
May 17, 2019

Conversation

bc-williamkwon
Copy link
Contributor

What?

Previously when config.json had show_product_reviews_tab as true the reviews link when clicked would do nothing. This fix should make it so that instead the reviews tab is opened and window is directed to it.

Tickets / Documentation

https://jira.bigcommerce.com/browse/STRF-5521

@bigbot
Copy link

bigbot commented May 15, 2019

Autotagging @bigcommerce/storefront-team @davidchin

@bc-williamkwon bc-williamkwon changed the title Update to fix product reviews link STRF-5521 PDP Product Reviews Link Not Clickable May 15, 2019
@junedkazi
Copy link
Contributor

Will it also support the use case when some one tries to hit a url with the anchor in it ? Example http://example.com/productpage/#product-reviews

@bc-williamkwon
Copy link
Contributor Author

Yes, it does. It moves the page down to the reviews just as if it was show_product_reviews_tab:false

@mattolson
Copy link
Contributor

@bc-williamkwon Can you update the changelog as well?

@bc-williamkwon
Copy link
Contributor Author

updated

@mattolson
Copy link
Contributor

@bc-williamkwon Please squash your commits

Update changelog.md for pr 1498
Made fix to product reviews link
@bc-williamkwon
Copy link
Contributor Author

bc-williamkwon commented May 17, 2019

Rebased and squashed

@mattolson mattolson merged commit 803072b into bigcommerce:master May 17, 2019
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

5 participants