Skip to content
This repository has been archived by the owner on Dec 17, 2021. It is now read-only.

Interim improvements to the current a11y scan process #108

Open
gbinal opened this issue Feb 3, 2017 · 5 comments
Open

Interim improvements to the current a11y scan process #108

gbinal opened this issue Feb 3, 2017 · 5 comments

Comments

@gbinal
Copy link
Member

gbinal commented Feb 3, 2017

Ideas here:

@gbinal
Copy link
Member Author

gbinal commented Feb 3, 2017

Inspect phase

  1. On Hold (not a blocker): Switch from using inspect.py to using pshtt.py.
  2. Done: Deprecate use of inspect or pshtt and instead just refer to an https or dap data file from a recent scan.

a11y phase

  1. Done: Make it so that the a11y scan doesn't require the inspect scan cache results. (issue)
  2. Done: Debug the ignore list and why it is and isn't working. (issue)
  3. Address sites that fail clumsily (causing the scan to have to be restarted). (issue)
  4. In Progress: Address sites that fail gracefully (the scan continues and just no errors are detected).

processing stage

  1. In Progress: Ensure that domains.json and agencies.json factor in sites that have no errors in a11y.csv. (issue)
  2. In Progress: Ensure that all of the math is good. (issue)
  3. Make semantic changes. (issue)

@gbinal
Copy link
Member Author

gbinal commented Feb 3, 2017

To elaborate on 2) above, I currently use inspect.py to:

  • Determine which domains are live.
  • Determine which domains are redirecting.

Hypothesis: I could instead pull the results here or the DAP results file, since the latter also goes ahead and cuts the domain list down to just executive branch domains.

Resulting idea: change the a11y script to work with the formatting of the DAP results file so that I don't need to change the headers.

Update: I'm checking but ^^ might not be necessary.
Update2: It looks like it is. When I run the script to generate the 3 json files in step 6a of the actual workflow, I have to delete the branch column in order to generate a proper agencies.json. I am guessing that this is b/c the script is looking for Agency in the 3rd column.

@gbinal
Copy link
Member Author

gbinal commented Apr 6, 2017

Still outstanding:

  • include all of the domains that error out in the a11y.csv (only some were included).
  • address the canonical domain link

@konklone
Copy link
Contributor

konklone commented May 8, 2017

Looks like moving from the inspect scanner to pshtt is also still outstanding. The change should be relatively minor.

@konklone
Copy link
Contributor

@gbinal - Just to confirm, is this still outstanding? If so, do you want to keep it open?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants