Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Bug#pwa 3025 #4053

Merged
merged 4 commits into from
Mar 23, 2023
Merged

Bug#pwa 3025 #4053

merged 4 commits into from
Mar 23, 2023

Conversation

RaghavendraTirumalasetti
Copy link
Contributor

Description

On a mobile device or desktop, when clicking on the text of a specific filter, it doesn't select the filter. For it to work properly, you have to specifically press on the checkbox.

To reproduce

Steps to reproduce the behavior:

Go to http://develop.pwa-venia.com/default/venia-tops.html?page=1
Expand any filter group
Click on the text label (not the checkbox itself) to apply a filter
Witness the problem: the filter doesn't get selected (it used to)
Expected behavior

When pressing or clicking on the text label beside the filter checkbox, the filter should be applied exactly as if I have explicitly clicked on the filter checkbox. The full line should be responsive to the selection (not only the checkbox part)

Related Issue

Closes #https://jira.corp.adobe.com/browse/PWA-3025.

Acceptance

Verification Stakeholders

Specification

Verification Steps

Test scenario(s) for direct fix/feature

Test scenario(s) for any existing impacted features/areas

Test scenario(s) for any Magento Backend Supported Configurations

Is Browser/Device testing needed?

Any ad-hoc/edge case scenarios that need to be considered?

Screenshots / Screen Captures (if appropriate)

Breaking Changes (if any)

Checklist

  • I have added tests to cover my changes, if necessary.
  • I have added translations for new strings, if necessary.
  • I have updated the documentation accordingly, if necessary.

@pwa-studio-bot
Copy link
Collaborator

pwa-studio-bot commented Mar 3, 2023

Fails
🚫 A version label is required. A maintainer must add one.
Messages
📖

Associated JIRA tickets: PWA-3025.

📖 DangerCI Failures related to missing labels/description/linked issues/etc will persist until the next push or next pr-test build run (assuming they are fixed).
📖

Access a deployed version of this PR here. Make sure to wait for the "pwa-pull-request-deploy" job to complete.

Generated by 🚫 dangerJS against 35b0b33

@glo42707
Copy link
Contributor

run cypress

@pwa-test-bot
Copy link

pwa-test-bot bot commented Mar 16, 2023

Successfully started codebuild job for cypress

@pwa-studio-bot
Copy link
Collaborator

Fails
🚫

node failed.

🚫

Cypress tests in the following files did not pass 😔. All tests must pass before this PR can be merged

  • user should be able to add and remove promotions from cart:
    • AssertionError: Timed out retrying after 30000ms: Expected not to exist in the DOM, but it was continuously found.
  • user should be able to update their address book:
    • CypressError: Timed out retrying after 60000ms: `cy.wait()` timed out waiting `60000ms` for the 1st request to the route: `gqlSignInAfterCreateMutation`. No request ever occurred.

      https://on.cypress.io/wait

Log

ERROR ON TASK: cypressTests


Error:  Danger had errors running. See message(s) above for more details.
danger-results://tmp/danger-results.json

Generated by 🚫 dangerJS against 35b0b33

@glo82145 glo82145 merged commit 480a4e9 into magento:develop Mar 23, 2023
@RaghavendraTirumalasetti RaghavendraTirumalasetti added the version: Minor This changeset includes functionality added in a backwards compatible manner. label Mar 28, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
pkg:venia-ui Progress: done version: Minor This changeset includes functionality added in a backwards compatible manner.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants