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

gspc-hidden-product-handler.php: Added snippet to remove url for Hidden Catalog Products on Cart. #888

Merged

Conversation

saifsultanc
Copy link
Contributor

@saifsultanc saifsultanc commented Sep 12, 2024

Context

⛑️ Ticket(s): https://secure.helpscout.net/conversation/2695302163/70820

Summary

When a Woo product has a catalog visibility of hidden, the cart item still shows its product link. This snippet removes it.

BEFORE:
Screenshot 2024-09-11 at 6 22 00 PM

AFTER:
Screenshot 2024-09-11 at 6 22 27 PM

…it button for Hidden Catalog Products on Cart.
@saifsultanc saifsultanc changed the title gspc-hidden-product-handler.php: Added snippet to remove url and edit button for Hidden Catalog Products on Cart. gspc-hidden-product-handler.php: Added snippet to remove url for Hidden Catalog Products on Cart. Sep 12, 2024
Copy link
Contributor

@claygriffiths claygriffiths left a comment

Choose a reason for hiding this comment

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

First, rename the file to gspc-unlink-hidden-products-in-cart.php.

Then, S&M with

`gspc-unlink-hidden-products-in-cart.php`: Added snippet to remove links in the cart to products with a hidden visibility.

…nks in the cart to products with a hidden visibility.
Copy link

Warnings
⚠️ When ready, don't forget to request reviews on this pull request from your fellow wizards.

Generated by 🚫 dangerJS against a60bf4b

@saifsultanc saifsultanc merged commit 24e0397 into master Sep 18, 2024
3 checks passed
@saifsultanc saifsultanc deleted the saif/add/70820-gspc-remove-url-edit-hidden-products branch September 18, 2024 06:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants