-
Notifications
You must be signed in to change notification settings - Fork 35
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
Feature/reports table view #2691
Feature/reports table view #2691
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## staging #2691 +/- ##
===========================================
- Coverage 85.81% 85.34% -0.47%
===========================================
Files 228 227 -1
Lines 7394 7451 +57
Branches 2360 2393 +33
===========================================
+ Hits 6345 6359 +14
- Misses 962 1002 +40
- Partials 87 90 +3 ☔ View full report in Codecov by Sentry. |
Passing run #2006 ↗︎
Details:
Review all test suite changes for PR #2691 ↗︎ |
@kepae @datherton09 |
@clari182 Clara, this is wonderful! Thank you so much! One quick thing I was wondering about with sorting the issue reports by date is that I'm not entirely sure if it's actually sorting them in either ascending/descending order? (The ordering seems semi-haphazard? Or maybe I'm missing something!) |
@datherton09 date filter and sort fixed! Ready for another round of testing |
@clari182 Really nice!! Thank you so much, Clara! :) |
Sorry, didn't finish my review here before merge conflicts came up! Will review shortly. |
a76b741
into
responsible-ai-collaborative:staging
Test URL: https://pr-2691--staging-aiid.netlify.app/
Changes:
Issue Reports
andReports
in the table view page/apps/reports
page, now everything is in the table view pageapps/incidents