accessibility-insights-web icon indicating copy to clipboard operation
accessibility-insights-web copied to clipboard

Quick Assess HTML report groups requirements incorrectly

Open madalynrose opened this issue 1 year ago • 8 comments

Describe the bug

The generated HTML report for Quick Assess currently uses the same summary model as Assessment. The assessment summary model groups requirements by Assessment. This means that the test details here display the Assessment title, not requirement titles. It also means that if any requirements are part of the same Assessment, they appear together and that the order of requirements is not respected in this list.

Expected behavior

Test details list requirements by name and show one requirement per title. It should instead use the Quick Assess summary model, as used on the Overview page for Quick Assess.

Screenshots

quick assess report with a red circle around "Headings", which has two circles indicating requirements

Are you willing to submit a PR?

Yes

Did you search for similar existing issues?

N/A

Additional context

N/A

madalynrose avatar Sep 26 '23 17:09 madalynrose

@madalynrose, I ma interested in this repo, please assign this to me.

thesharifi avatar Oct 03 '23 04:10 thesharifi

Hello there!

I'm interested in working on this issue. Could you please assign it to me?

shanmukhasaisai avatar Oct 10 '23 13:10 shanmukhasaisai

@thesharifi we've assigned the bug to you. If you're no longer interested, let us know and we'll assign it to the other user who showed interest.

JGibson2019 avatar Nov 13 '23 19:11 JGibson2019

Implementation note: Please don't change the report package in any way, since it's used by other packages that we create. The optimal path forward may be to leverage the code from get-quick-assess-summary-model.tx from AssessmentsProviderImpl so that the results are properly grouped when running from the QuickAssess. Once the results are correctly grouped, that should be automatically reflected in the report package.

DaveTryon avatar Nov 13 '23 21:11 DaveTryon

@thesharifi, any update on this? We might want to assign to someone else if it's not going to happen anytime soon

DaveTryon avatar Jan 08 '24 19:01 DaveTryon

@shanmukhasaisai, are you still interested in working on this? The first person to volunteer seems to have lost interest

DaveTryon avatar Jan 29 '24 20:01 DaveTryon

Yes, iam interested to work

shanmukhasaisai avatar Jan 30 '24 01:01 shanmukhasaisai

@shanmukhasaisai, Sorry for the delay in getting back to you. If you're still available, please take this and run with it. Thanks!

DaveTryon avatar Mar 18 '24 17:03 DaveTryon