-
Notifications
You must be signed in to change notification settings - Fork 44
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
[726] added error messages to recruitment dates in offer creation #1041
base: master
Are you sure you want to change the base?
[726] added error messages to recruitment dates in offer creation #1041
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1041 +/- ##
==========================================
- Coverage 91.52% 90.89% -0.63%
==========================================
Files 119 119
Lines 2901 2922 +21
Branches 31 41 +10
==========================================
+ Hits 2655 2656 +1
- Misses 242 252 +10
- Partials 4 14 +10
Continue to review full report at Codecov.
|
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 part of validations should also be run on backend here: https://github.com/CodeForPoznan/volontulo/blob/master/backend/apps/volontulo/serializers.py#L175
@@ -17,3 +17,7 @@ label { | |||
text-align: left; | |||
} | |||
} | |||
|
|||
.form-text{ | |||
font-size: .875rem |
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.
should end with semicolon
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.
added
Backend issue here #1094 |
Description:
Form show a message that date of recriutment can't be set after action is finished. Offer with wrong dates cannot be saved in database.