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

Adding Equal Access Scanner to Dev #982

Merged
merged 46 commits into from
Jan 31, 2025
Merged

Adding Equal Access Scanner to Dev #982

merged 46 commits into from
Jan 31, 2025

Conversation

dmols
Copy link
Contributor

@dmols dmols commented Jan 17, 2025

This PR would bring the equal access changes to the dev branch. This branch also has the php upgrade changes, which explains why there's a large number of files changed. To avoid any possible conflicts, I suggest we merge the php 8.4 branch to dev first and then pull those changes to this branch, prior to merging this PR to dev.

To test this branch, simply change the scanner chosen in the .env file by altering the following line.

ACCESSIBILITY_CHECKER: __________

In dev the option is limited to phpally but here you can type equalaccess_lambda and restart the containers for the scanner to switch over.

panbed and others added 30 commits August 21, 2024 15:19
… class that calls each scanner depending on the .env
Updating equal access branch with lambda functions
…nd error with style_color_misuse needs to be checked
displaying the message from equal access for ufixitreview

changing template for getDomDocument (needs some fixing since getting parent shows placeholder)
Merge dev into equal-access
@dmols dmols requested review from mbusch3 and evannaderi January 17, 2025 22:17
@dmols dmols self-assigned this Jan 17, 2025
@dmols dmols changed the base branch from dev to dev-pr-testing January 31, 2025 20:57
@dmols dmols merged commit 3564afa into dev-pr-testing Jan 31, 2025
4 checks passed
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.

3 participants