Skip to content

Conversation

@joe-allen-89
Copy link
Contributor

Removed the check for aria-expanded on both menuResources and pageResources due to recent change to core drawer (adaptlearning/adapt-contrib-core#663)

#137

Fix

@joe-allen-89 joe-allen-89 requested a review from oliverfoster May 29, 2025 14:59
Copy link
Contributor

Choose a reason for hiding this comment

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

👀

Copy link
Contributor

@kirsty-hames kirsty-hames left a comment

Choose a reason for hiding this comment

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

👀

@oliverfoster oliverfoster merged commit 997fc8d into master Jun 2, 2025
@oliverfoster oliverfoster deleted the issue/137 branch June 2, 2025 09:37
@github-project-automation github-project-automation bot moved this from New to Recently Released in adapt_framework: The TODO Board Jun 2, 2025
github-actions bot pushed a commit that referenced this pull request Jun 2, 2025
## [5.7.2](v5.7.1...v5.7.2) (2025-06-02)

### Fix

* e2e test updated to not check for aria-expanded (fixes #137) (#138) ([997fc8d](997fc8d)), closes [#137](#137) [#138](#138)
@github-actions
Copy link

github-actions bot commented Jun 2, 2025

🎉 This PR is included in version 5.7.2 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update resource e2e tests

6 participants