-
Notifications
You must be signed in to change notification settings - Fork 49
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
Accessibility Issues Provider #564
Conversation
…ty-report-integration
…ty-report-integration
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please provide some documentation for this features as well - How to use it from AET user perspective (i.e. report) etc.
Please also update the CHANGELOG.md file with info about this PR
…egration' into feature/accessibility-report-integration
|
||
URL is as follows: | ||
|
||
`<domain>/api/accessibility/report?company=<company>&project=<project>&suite=<suite>&correlationId=<correlationId>&type=<type>&extension=<type>` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'type' should be replaced by 'verbosity'
Where: | ||
- `domain`: domain on which AETs are set up (eg. `localhost`) | ||
- `company`, `project`, `suite`, `correlationId`: for these the report will be generated, if available | ||
- `verbosity`: verbosity level of the report, comma separated values: `NOTICE`, `WARNING`, `ERROR` (case-insensitive; e.g.: `notice,error`) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
WARNING
is not working (it's 'WARN' instead) so documentation should be adjusted or implementation changed
Implementation of #557
Description
Motivation and Context
Types of changes
Checklist:
I hereby agree to the terms of the AET Contributor License Agreement.