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

[Feature ALS-1732] e2e for responsiveness #99

Merged
merged 85 commits into from
Nov 15, 2023

Conversation

wadhawh
Copy link
Collaborator

@wadhawh wadhawh commented Nov 14, 2023

Added data test ids for responsive e2e tests.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

its-aazizi and others added 28 commits September 1, 2023 04:55
This PR has responsive cases as well as desktop cases for the E2E tests, the responsive cases have been skipped for now. This PR adds missing data-testid to various responsive elements. Once we deploy this PR to prod, I'll update the same PR to not skip responsive E2E tests.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
…responsiveness_main

[Feature ALS-1732] e2e for responsiveness
@vinguthh vinguthh merged commit e947349 into main Nov 15, 2023
3 checks passed
@wadhawh wadhawh deleted the feature_ALS-1732_e2e-for-responsiveness branch January 8, 2024 12:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants