Hiding banner in WC if landing page = shop page; A few adjustments to the popup maker module code comments - Githubissues

strangerstudios / sitewide-sales

Run Black Friday, Cyber Monday, or other flash sales on your WordPress-powered eCommerce or membership site with WooCommerce, Easy Digital Downloads, and Paid Memberships Pro.

https://sitewidesales.com

GNU General Public License v3.0

2 stars 6 forks source link

Hiding banner in WC if landing page = shop page; A few adjustments to the popup maker module code comments #97

Closed kimcoleman closed 2 years ago

kimcoleman commented 2 years ago

All Submissions:

[x] Have you followed the Contributing guideline?
[x] Does your code follow the WordPress' coding standards?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Changes proposed in this Pull Request:

A couple inline code comments updates for Popup maker module.
New filter to hide banner on landing page if it is set to the "shop" page in WooCommerce.

Other information:

[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] Have you successfully run tests with your changes locally?

Changelog entry

BUG FIX/ENHANCEMENT: Now hiding banner if landing page is set to the WooCommerce "Shop" page.

Enter a summary of all changes on this Pull Request. This will appear in the changelog if accepted.