-
Notifications
You must be signed in to change notification settings - Fork 2
Application review #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
Conversation
Start adding review system endpoints Remove reviewer model Add routes and logic to get application to review Remove unused db query in applicant service Add ReviewService tests
Add review mapping for model properties
Adds support for unverified auth level
Codecov Report
@@ Coverage Diff @@
## master #35 +/- ##
==========================================
+ Coverage 62.64% 68.69% +6.05%
==========================================
Files 36 44 +8
Lines 803 936 +133
Branches 89 96 +7
==========================================
+ Hits 503 643 +140
+ Misses 292 282 -10
- Partials 8 11 +3
Continue to review full report at Codecov.
|
Add service method, get number of reviews created by a specific auth id
Codecov Report
@@ Coverage Diff @@
## master #35 +/- ##
==========================================
+ Coverage 62.64% 67.85% +5.21%
==========================================
Files 36 44 +8
Lines 803 952 +149
Branches 89 97 +8
==========================================
+ Hits 503 646 +143
- Misses 292 295 +3
- Partials 8 11 +3
Continue to review full report at Codecov.
|
Resolves #7
ApplicationMapped
decorator for review mappingmaxLength
property in long application questions