-
Notifications
You must be signed in to change notification settings - Fork 8
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] Add claim verification UI #10502
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #10502 +/- ##
============================================
+ Coverage 31.57% 38.45% +6.87%
- Complexity 0 1421 +1421
============================================
Files 777 1002 +225
Lines 25124 30763 +5639
Branches 6570 6589 +19
============================================
+ Hits 7933 11829 +3896
- Misses 17015 18911 +1896
+ Partials 176 23 -153
Flags with carried forward coverage won't be shown. Click here to find out more. β View full report in Codecov by Sentry. |
Are we sure about this, @petertgiles requested it to be like so in the first place #10434 (comment) |
π @Jerryescandon Said we do not need to restrict it, just prefer they did not. |
This comment was marked as resolved.
This comment was marked as resolved.
Yeah, that makes sense but I also found some other issues with the ROD seeders so I feel like if we start toying with them here it could expand the scope too far π |
In that case, please make an issue combining them |
Issue created: |
Thanks, I also meant to combine the other problems you found into that issue. If they are small tweaks that is |
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.
Seems to work fine, just a few things to bring to your attention
...oolCandidates/ViewPoolCandidatePage/components/ClaimVerification/ClaimVerificationDialog.tsx
Outdated
Show resolved
Hide resolved
.../web/src/pages/PoolCandidates/ViewPoolCandidatePage/components/ClaimVerification/messages.ts
Outdated
Show resolved
Hide resolved
.../web/src/pages/PoolCandidates/ViewPoolCandidatePage/components/ClaimVerification/messages.ts
Outdated
Show resolved
Hide resolved
...web/src/pages/PoolCandidates/ViewPoolCandidatePage/components/ClaimVerification/ClaimRow.tsx
Outdated
Show resolved
Hide resolved
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.
Great work as usual. Nothing else seems amiss to me.
π
π€ Resolves #10418
π Introduction
Adds the UI for admins to verify claims made by candidates.
π΅οΈ Details
There were some changes to validation since they were too strict. While currently we won't be setting an expiry date for rejected claims (and hide the field) it should not be prohibited so those rules and the test for it have been removed.
π§ͺ Testing
pnpm run dev
[email protected]
πΈ Screenshot