pwa-studio icon indicating copy to clipboard operation
pwa-studio copied to clipboard

[bug]: Filter delimiter is likely to fail under some circumstances

Open Jordaneisenburger opened this issue 2 years ago • 4 comments

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:

  1. Add an attribute to the filters with a comma in the label
  2. 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

Jordaneisenburger avatar Apr 01 '22 10:04 Jordaneisenburger

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


m2-assistant[bot] avatar Apr 01 '22 10:04 m2-assistant[bot]

Please let me know what delimiter we should use and I can create a PR

@magento I am working on this

Jordaneisenburger avatar Apr 01 '22 10:04 Jordaneisenburger

@magento export issue to JIRA project PWA as Bug

anthoula avatar Apr 04 '22 15:04 anthoula

:white_check_mark: Jira issue https://jira.corp.magento.com/browse/PWA-2732 is successfully created for this GitHub issue.

github-jira-sync-bot avatar Apr 04 '22 15:04 github-jira-sync-bot

This issue is reproducible and hence confirming the issue.

glo80771 avatar Jul 03 '23 07:07 glo80771

@adobe export issue to JIRA project PWA as Bug

glo42707 avatar Jul 06 '23 11:07 glo42707

:x: Cannot export the issue. This GitHub issue is already linked to Jira issue(s): https://jira.corp.adobe.com/browse/PWA-2732

github-jira-sync-bot avatar Jul 06 '23 11:07 github-jira-sync-bot

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. image

glo42707 avatar Jul 19 '23 06:07 glo42707

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.

glo42707 avatar Aug 16 '23 17:08 glo42707