-
Notifications
You must be signed in to change notification settings - Fork 30
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
Tbolt/4182 Fix issue with renewing state admin #4262
Conversation
Codecov Report
@@ Coverage Diff @@
## main #4262 +/- ##
==========================================
+ Coverage 86.30% 86.32% +0.01%
==========================================
Files 260 260
Lines 5931 5932 +1
Branches 1240 1240
==========================================
+ Hits 5119 5121 +2
+ Misses 752 751 -1
Partials 60 60
Continue to review full report at Codecov.
|
This deploy was cleaned up. |
Looks good to me!
|
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.
Works as expected. Maybe this is obvious, but a consideration is that the record a (FFY 2022) must be matched before adding a new certification for record b (FFY 2023). If you don't it will just list both without the ability the match record b. I don't see this as a bug and I wasn't able to identify any real issues with the way it work now.
Just to clarify, as a user you just need to make sure that the status of the previous user says matched before you recertify, but it is a nonissue. |
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.
This works as advertised. Thank you.
Resolves #4182
Description
Updates the state admin matching process to allow matching on already existing matches.
Significant changes or possible side effects
State admins who have been previously approved should now be eligible "matches." Unsure if any side effects are introduced from this.
Automated test cases written
Test added to verify that state admin role will return a match
Steps to manually verify this change
This pull request is ready to review when
This pull request can be merged when