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

pity we didn't put some testid attribute to this our extra element against which we could check here... #883

Open
jiridanek opened this issue Feb 4, 2025 · 1 comment · Fixed by #910
Labels
good first issue Good for newcomers

Comments

@jiridanek
Copy link
Member

          pity we didn't put some `testid` attribute to this our extra element against which we could check here...

Originally posted by @jstourac in #874 (comment)

@github-project-automation github-project-automation bot moved this to 📋 Backlog in ODH IDE Planning Feb 4, 2025
@jiridanek jiridanek added the good first issue Good for newcomers label Feb 4, 2025
@jiridanek jiridanek linked a pull request Feb 20, 2025 that will close this issue
3 tasks
jiridanek pushed a commit that referenced this issue Feb 20, 2025
)

* feat: add data-testid attribute for loading spinner

* feat: change assert for data-testid attribute

* fix: incomplete attribute

* chore: keep long lines
@github-project-automation github-project-automation bot moved this from 📋 Backlog to ✅Done in ODH IDE Planning Feb 20, 2025
@jiridanek
Copy link
Member Author

@jiridanek jiridanek reopened this Feb 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

1 participant