Candidate test run page #461
Conversation
evmiguel
force-pushed
the
candidate-test-run-page
branch
from
f025730
to
bf83eb3
Compare
evmiguel
force-pushed
the
candidate-test-run-page
branch
from
118cc8e
to
b9a70a3
Compare
| recommendedStatusTargetDate | ||
| vendorReviewStatus | ||
| issues { | ||
| author |
There was a problem hiding this comment.
The mockup also includes a hyperlink on the text which says something like x people have left feedback for this test. My guess is that should be to the issue(s) (although I'm not sure how to account for multiple links visually, a pop up or so of applicable links? But this may be something to discuss internally).
But note that link is available in the query to get access to those GitHub issue links
There was a problem hiding this comment.
@s3ththompson @isaacdurazo how should we account for the multiple links in the case of x people have left feedback for this test?
There was a problem hiding this comment.
Great question! Sorry it is not clear in the mockups. Would it be possible to keep it as a single link that takes you to Github and display only the issues for that particular test once there?
There was a problem hiding this comment.
It is possible. We'd just have to map a link with the right query to the component.
Co-authored-by: Howard Edwards <howarde.edwards@gmail.com>
evmiguel
force-pushed
the
candidate-test-run-page
branch
from
27a6919
to
31e049e
Compare
…backListItem.css Co-authored-by: Howard Edwards <howarde.edwards@gmail.com>
Co-authored-by: Howard Edwards <howarde.edwards@gmail.com>
…mbers were getting duplicated across the columns for each AT)
* Start to test plan management page * Add new Disclosure Component * Update Test Plan Management page; update DisclosureComponent * Add bulk update status resolver for TestPlanReport * Split Summary Status Row into its own component to maintain focus and row update advantages; updating error message when bulk updating test plan report statuses to make it easier to find what report in the group prevents the update * Update tests * Update tests * add additional test plan report status (RECOMMENDED); change FINALIZED -> CANDIDATE * Change width at which navigation menu collapses due to more navigation options * Make updates to reports related pages to account for Recommended Reports being shown * Adjust vendorReviewStatus being changed in updateStatusResolver based on PR feedback * Update content copy for Test Management and Reports page * Update vendor review status being inferred for updateStatusResolver.js * Remove unnecessary comment
…uncement (#472) * Using disclosure component and adding accessbility fixes * Address console warning * Update announcement when navigating through tests on Candidate Run Page Co-authored-by: Howard Edwards <howarde.edwards@gmail.com>
This PR implements the Candidate Test Run Page and feedback modals as specified in this mockup and this design.
yarnto install dependenciesInstructions for dev server:
yarn sequelize db:migratehttp://localhost:3000/?fakeRole=vendorhttp://localhost:3000/candidate-testsand selecting one of the test plans