pwa-studio
pwa-studio copied to clipboard
[bug]: Filter delimiter is likely to fail under some circumstances
Describe the bug
When you have a filter label that looks like "Audio, Video & Multimedia" the default delimiter "," is not going to work and it will send a part of the label to the backend instead of the attribute code.
To reproduce
Steps to reproduce the behavior:
- Add an attribute to the filters with a comma in the label
- Go to category page and click that new filter and note that it's not sending the attribute code in the filter variable to the backend
Expected behavior
It just works like other filters
Possible solutions
The easiest solution is to change the DELIMITER inside https://github.com/magento/pwa-studio/blob/551d9680b49236b76e9db69e3156cca5d99b0853/packages/peregrine/lib/talons/FilterModal/helpers.js#L1 to something else like ",," or "/" something that won't be used in a filter label
Debug Report
Please complete the following device information:
- Device [e.g. iPhone6, PC, Mac, Pixel3]: Windows
- Browser [e.g. Chrome, Safari]: Chrome
- Browser Version [e.g. 22]:
- Magento Version [e.g Adobe Commerce 2.4]: 2.4.x
Please let us know what packages this bug is in regards to:
- [ ]
venia-concept
- [ ]
venia-ui
- [ ]
pwa-buildpack
- [x]
peregrine
- [ ]
pwa-devdocs
- [ ]
upward-js
- [ ]
upward-spec
- [ ]
create-pwa
Hi @Jordaneisenburger. Thank you for your report. To speed up processing of this issue, make sure that you provided sufficient information.
Add a comment to assign the issue: @magento I am working on this
- Join Magento Community Engineering Slack and ask your questions in #github channel.
Please let me know what delimiter we should use and I can create a PR
@magento I am working on this
@magento export issue to JIRA project PWA as Bug
:white_check_mark: Jira issue https://jira.corp.magento.com/browse/PWA-2732 is successfully created for this GitHub issue.
This issue is reproducible and hence confirming the issue.
@adobe export issue to JIRA project PWA as Bug
:x: Cannot export the issue. This GitHub issue is already linked to Jira issue(s): https://jira.corp.adobe.com/browse/PWA-2732
hey,
@Jordaneisenburger, after retesting by team, we are not able to reproduce it even as per given steps followed.
kindly let us know if we are missing anything to reproduce same. since Filter is working fine for custome attributes.
closing this ticket since no reply from Reporter on Github Ticket from past 2 weeks as per defined process. please feel free to reopen in case still persisit.