Skip to content

fix :filter stock vs non-stock items when Item Visibility changes #1359

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

fix :filter stock vs non-stock items when Item Visibility changes #1359

wants to merge 3 commits into from

Conversation

@Gadha2311
Copy link
Contributor

No description provided.

@Gadha2311 Gadha2311 force-pushed the new-filter-itemVisibility branch from ab3b60b to 5a8fe5f Compare October 13, 2025 10:52
akshayitzme
akshayitzme reviewed Oct 19, 2025
View reviewed changes
akshayitzme
akshayitzme reviewed Oct 19, 2025
View reviewed changes
akshayitzme
akshayitzme requested changes Oct 19, 2025
View reviewed changes
Copy link
Collaborator

@akshayitzme akshayitzme left a comment

Choose a reason for hiding this comment

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

Please take a look at the comments above. Ping me if you need more clarity.

Let's use Check fields wherever there is only binary values. That will be easier in development, readability and debugging too.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants

@Gadha2311 @akshayitzme