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

Fix/ae 2031 cypress tests fix #98

Merged
merged 62 commits into from
Feb 14, 2024
Merged

Conversation

solita-juhohaa
Copy link
Contributor

No description provided.

@solita-juhohaa solita-juhohaa force-pushed the fix/AE-2031-cypress-tests-fix branch 11 times, most recently from 03ddf53 to 846c3af Compare February 8, 2024 08:25
@solita-juhohaa solita-juhohaa force-pushed the fix/AE-2031-cypress-tests-fix branch 19 times, most recently from 6d0ec59 to 18d727e Compare February 9, 2024 17:57
e2e-tests/cypress/README.md Show resolved Hide resolved
etp-front/webpack.config.js Show resolved Hide resolved
etp-core/etp-db/Dockerfile Outdated Show resolved Hide resolved
@@ -8,4 +9,4 @@
}
</style>

<h2>{text}</h2>
<h2 {name} data-cy={name}>{text}</h2>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Of data-cy it is quite obvious that it is related to Cypress, but is that name attribute also intentional?

Remove invalid `name` attribute from headers and change the `name` prop into `dataCy`
@solita-juhohaa solita-juhohaa merged commit b5cb3bd into main Feb 14, 2024
19 checks passed
@solita-juhohaa solita-juhohaa deleted the fix/AE-2031-cypress-tests-fix branch February 14, 2024 15:17
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