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

theme: filterable dynamic options. #9570

Open
wants to merge 12 commits into
base: master
Choose a base branch
from

Conversation

suraj-webkul
Copy link
Collaborator

No description provided.

@ashishkumar-webkul
Copy link
Collaborator

@suraj-webkul

There are some conflicts in this PR. Please fix the PR conflicts.

@ashishkumar-webkul ashishkumar-webkul added the Conflicts When conflicts present in pull request. label Mar 26, 2024
@suraj-webkul suraj-webkul force-pushed the theme-filter-options branch 2 times, most recently from 82724ad to 4e1c928 Compare April 2, 2024 12:17
@suraj-webkul suraj-webkul removed the Conflicts When conflicts present in pull request. label Apr 2, 2024
@suraj-webkul
Copy link
Collaborator Author

Resolved conflicts.

@ashishkumar-webkul ashishkumar-webkul added the QA Passed When QA pass the pull request. label May 6, 2024
@ashishkumar-webkul ashishkumar-webkul self-assigned this May 6, 2024
@ashishkumar-webkul ashishkumar-webkul removed the QA Passed When QA pass the pull request. label May 6, 2024
devansh-webkul
devansh-webkul previously approved these changes May 6, 2024
@ashishkumar-webkul ashishkumar-webkul added the In Testing When PR in testing. label May 7, 2024
@ashishkumar-webkul ashishkumar-webkul added QA Passed When QA pass the pull request. and removed In Testing When PR in testing. labels May 7, 2024
@ashishkumar-webkul
Copy link
Collaborator

Checked and Found that the filters are working fine, please check the video for reference.

Video

Filters.added.mp4

@ashishkumar-webkul ashishkumar-webkul added Change Request if need any changes on PR and removed QA Passed When QA pass the pull request. labels May 7, 2024
@ashishkumar-webkul
Copy link
Collaborator

ashishkumar-webkul commented May 7, 2024

Need to fix the issue of static content view, page is not appearing properly. Please fix the issue.

Video

Static.mp4

@devansh-webkul
Copy link
Member

Conflicts.

@suraj-webkul suraj-webkul added the Fix Proposed A issue that has a PR related to it that provides a possible resolution but stil needs to be merged. label May 14, 2024
@ashishkumar-webkul ashishkumar-webkul added In Testing When PR in testing. QA Passed When QA pass the pull request. and removed Fix Proposed A issue that has a PR related to it that provides a possible resolution but stil needs to be merged. In Testing When PR in testing. Change Request if need any changes on PR labels May 14, 2024
@ashishkumar-webkul
Copy link
Collaborator

Checked and found that the issue related to the static content pages in theme section and also the product crousal concern are added with filters, hence the issue has been fixed and working fine. Please check the video for reference.

Video

Static.mp4

Copy link
Collaborator

@ashishkumar-webkul ashishkumar-webkul left a comment

Choose a reason for hiding this comment

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

checked and found that he static content issue fixed and also product crousal is added with the attribute options.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
QA Passed When QA pass the pull request.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants