newfold-labs / wp-module-ecommerce

Next-generation eCommerce Experience for WordPress sites at Newfold Digital.
GNU General Public License v2.0
9 stars 3 forks source link

Update isHostgator check to isBluehost since the content is FOR bluehost #132

Closed circlecube closed 1 year ago

circlecube commented 1 year ago

isHostgator is being used to determine whether to show bluehost links. With expanding the e-commerce module to other plugins (like crazy-domains), this should probably be an affirmative isBluehost check to ensure that the brand contains bluehost in order to show bluehost links.

https://github.com/newfold-labs/wp-module-ecommerce/blob/3385c278c5185189fe966fde2ffde3c40ddebad3/src/configs/YITHPlugins.config.js#L31 https://github.com/newfold-labs/wp-module-ecommerce/blob/3385c278c5185189fe966fde2ffde3c40ddebad3/src/configs/ProductsAndServices.config.js#L29

As it is now, this will display bluehost links in any plugin except hostgator, while we'd prefer it to only display these links in bluehost.

_

Would be nice to have a page that works for each brand, but that ends up being redundant and a lot of content to maintain. Ideally, we have brand-agnostic landing pages for these plugins at yith or elsewhere.