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

Layered Navigation Filters not displaying on top-level category #1378

Closed
duffner opened this issue Apr 12, 2019 · 5 comments
Closed

Layered Navigation Filters not displaying on top-level category #1378

duffner opened this issue Apr 12, 2019 · 5 comments
Assignees
Labels

Comments

@duffner
Copy link

duffner commented Apr 12, 2019

Preconditions

Magento Version : 2.3.0

ElasticSuite Version : 2.7.6

Environment : Production

Third party modules :
ElasticSearch

Steps to reproduce

  1. Have Child of Root Level Category
  2. Look at Layered Navigation Options

Expected result

  1. I should be able to select what attributes to display on the category

Actual result

screencapture-dev-gibuys-admin_fbxw54-catalog-category-edit-key-3adee78e93abae0d6782f2330d373f75e2fa48e3d04d482d9864703b9ec73e75-id-7-2019-04-12-10_16_41

Do you have any ideas about what may be causing this issue?

@romainruaud
Copy link
Collaborator

I'm nearly sure it's due to a mistake of mine : https://github.com/Smile-SA/elasticsuite/blob/2.7.x/src/module-elasticsuite-catalog/Plugin/Ui/Category/Form/DataProviderPlugin.php#L121

Should be >= 2 and not >2.

I guess you have the issue on first level of categories only ?

Regards

@duffner
Copy link
Author

duffner commented Apr 15, 2019

@romainruad, yes it’s only with that level and your reference makes sense! I was just hoping this wasn’t expected behavior. Thanks for looking. Do you know what version the fix will be available?

@romainruaud
Copy link
Collaborator

next minor version since I pushed the PR #1379 that fix this issue.

Sorry for the inconvenience it caused.

Regards

@romainruaud
Copy link
Collaborator

PR merged so I close the issue

@duffner
Copy link
Author

duffner commented Apr 15, 2019

@romainruaud, let me know once this is available via composer and I'll test for you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants