sending emails to eligible students only and setting proper deadline. (#143)
This commit is contained in:
parent
290639c032
commit
9529f45062
|
@ -42,7 +42,7 @@ class Student(models.Model):
|
|||
@property
|
||||
def _history_user(self):
|
||||
return self.changed_by
|
||||
|
||||
|
||||
@_history_user.setter
|
||||
def _history_user(self, value):
|
||||
if isinstance(value, User):
|
||||
|
@ -51,7 +51,6 @@ class Student(models.Model):
|
|||
self.changed_by = None
|
||||
|
||||
|
||||
|
||||
class Admin(models.Model):
|
||||
id = models.CharField(blank=False, max_length=15, primary_key=True)
|
||||
name = models.CharField(blank=False, max_length=JNF_TEXT_MAX_CHARACTER_COUNT)
|
||||
|
@ -61,7 +60,7 @@ class Admin(models.Model):
|
|||
@property
|
||||
def _history_user(self):
|
||||
return self.changed_by
|
||||
|
||||
|
||||
@_history_user.setter
|
||||
def _history_user(self, value):
|
||||
if isinstance(value, User):
|
||||
|
@ -71,7 +70,8 @@ class Admin(models.Model):
|
|||
|
||||
|
||||
def two_day_after_today():
|
||||
return timezone.now() + timezone.timedelta(days=2)
|
||||
# round off to nearest day
|
||||
return timezone.now().replace(hour=0, minute=0, second=0, microsecond=0) + timezone.timedelta(days=2)
|
||||
|
||||
|
||||
class Placement(models.Model):
|
||||
|
@ -145,7 +145,8 @@ class Placement(models.Model):
|
|||
created_at = models.DateTimeField(blank=False, default=None, null=True)
|
||||
updated_at = models.DateTimeField(blank=False, default=None, null=True)
|
||||
changed_by = models.ForeignKey(User, on_delete=models.CASCADE, blank=True, null=True)
|
||||
history = HistoricalRecords(user_model=User)
|
||||
history = HistoricalRecords(user_model=User)
|
||||
|
||||
def format(self):
|
||||
if self.company_name is not None:
|
||||
self.company_name = self.company_name.strip()[:JNF_SMALLTEXT_MAX_CHARACTER_COUNT]
|
||||
|
@ -187,12 +188,13 @@ class Placement(models.Model):
|
|||
if self.other_requirements is not None:
|
||||
self.other_requirements = self.other_requirements.strip()[:JNF_TEXTAREA_MAX_CHARACTER_COUNT]
|
||||
if self.additional_info is not None:
|
||||
self.additional_info = [info.strip()[:JNF_TEXTMEDIUM_MAX_CHARACTER_COUNT] for info in list(self.additional_info)]
|
||||
self.additional_info = [info.strip()[:JNF_TEXTMEDIUM_MAX_CHARACTER_COUNT] for info in
|
||||
list(self.additional_info)]
|
||||
|
||||
@property
|
||||
def _history_user(self):
|
||||
return self.changed_by
|
||||
|
||||
|
||||
@_history_user.setter
|
||||
def _history_user(self, value):
|
||||
if isinstance(value, User):
|
||||
|
@ -235,14 +237,14 @@ class PlacementApplication(models.Model):
|
|||
@property
|
||||
def _history_user(self):
|
||||
return self.changed_by
|
||||
|
||||
|
||||
@_history_user.setter
|
||||
def _history_user(self, value):
|
||||
if isinstance(value, User):
|
||||
self.changed_by = value
|
||||
else:
|
||||
self.changed_by = None
|
||||
|
||||
|
||||
class Meta:
|
||||
verbose_name_plural = "Placement Applications"
|
||||
unique_together = ('placement_id', 'student_id')
|
||||
|
@ -267,7 +269,7 @@ class PrePlacementOffer(models.Model):
|
|||
@property
|
||||
def _history_user(self):
|
||||
return self.changed_by
|
||||
|
||||
|
||||
@_history_user.setter
|
||||
def _history_user(self, value):
|
||||
if isinstance(value, User):
|
||||
|
|
|
@ -270,9 +270,9 @@ def opening_description_table_html(opening):
|
|||
# check typing of opening
|
||||
if isinstance(opening, Placement):
|
||||
details = model_to_dict(opening, fields=[field.name for field in Placement._meta.fields],
|
||||
exclude=EXCLUDE_IN_PDF)
|
||||
exclude=EXCLUDE_IN_PDF)
|
||||
# check typing of opening is query dict
|
||||
else: #if isinstance(opening, QueryDict):
|
||||
else: # if isinstance(opening, QueryDict):
|
||||
details = opening
|
||||
keys = list(details.keys())
|
||||
newdetails = {}
|
||||
|
@ -318,44 +318,48 @@ def send_opening_notifications(placement_id):
|
|||
placement = get_object_or_404(Placement, id=placement_id)
|
||||
students = Student.objects.all()
|
||||
for student in students.iterator():
|
||||
if PlacementApplicationConditions(student, placement)[0]:
|
||||
try:
|
||||
student_user = get_object_or_404(User, id=student.id)
|
||||
subject = NOTIFY_STUDENTS_OPENING_TEMPLATE_SUBJECT.format(company_name=placement.company_name)
|
||||
data = {
|
||||
"company_name": placement.company_name,
|
||||
"opening_type": 'Placement',
|
||||
"designation": placement.designation,
|
||||
"deadline": placement.deadline_datetime.strftime("%A, %-d %B %Y, %-I:%M %p"),
|
||||
"link": PLACEMENT_OPENING_URL.format(id=placement.id)
|
||||
}
|
||||
sendEmail(student_user.email, subject, data, NOTIFY_STUDENTS_OPENING_TEMPLATE)
|
||||
except Http404:
|
||||
logger.warning('Utils - send_opening_notifications: user not found : ' + student.id)
|
||||
except Exception as e:
|
||||
logger.warning('Utils - send_opening_notifications: For Loop' + str(e))
|
||||
|
||||
if student.branch in placement.allowed_branch:
|
||||
if student.degree == 'bTech' or placement.rs_eligible is True:
|
||||
if PlacementApplicationConditions(student, placement)[0]:
|
||||
try:
|
||||
student_user = get_object_or_404(User, id=student.id)
|
||||
subject = NOTIFY_STUDENTS_OPENING_TEMPLATE_SUBJECT.format(
|
||||
company_name=placement.company_name)
|
||||
data = {
|
||||
"company_name": placement.company_name,
|
||||
"opening_type": 'Placement',
|
||||
"designation": placement.designation,
|
||||
"deadline": placement.deadline_datetime.strftime("%A, %-d %B %Y, %-I:%M %p"),
|
||||
"link": PLACEMENT_OPENING_URL.format(id=placement.id)
|
||||
}
|
||||
sendEmail(student_user.email, subject, data, NOTIFY_STUDENTS_OPENING_TEMPLATE)
|
||||
except Http404:
|
||||
logger.warning('Utils - send_opening_notifications: user not found : ' + student.id)
|
||||
except Exception as e:
|
||||
logger.warning('Utils - send_opening_notifications: For Loop' + str(e))
|
||||
|
||||
except:
|
||||
logger.warning('Utils - send_opening_notifications: ' + str(sys.exc_info()))
|
||||
return False
|
||||
|
||||
|
||||
def exception_email(opening):
|
||||
opening = opening.dict()
|
||||
data = {
|
||||
"designation": opening["designation"],
|
||||
"opening_type": PLACEMENT,
|
||||
"company_name": opening["company_name"],
|
||||
}
|
||||
"designation": opening["designation"],
|
||||
"opening_type": PLACEMENT,
|
||||
"company_name": opening["company_name"],
|
||||
}
|
||||
pdfhtml = opening_description_table_html(opening)
|
||||
name = opening["company_name"]+'_jnf_response.pdf'
|
||||
name = opening["company_name"] + '_jnf_response.pdf'
|
||||
attachment_jnf_respone = {
|
||||
"name": name,
|
||||
"html": pdfhtml,
|
||||
}
|
||||
|
||||
}
|
||||
|
||||
sendEmail(CDC_MAIl_ADDRESS, COMPANY_OPENING_ERROR_TEMPLATE.format(company_name=opening["company_name"]), data,
|
||||
COMPANY_OPENING_SUBMITTED_TEMPLATE, attachment_jnf_respone)
|
||||
COMPANY_OPENING_SUBMITTED_TEMPLATE, attachment_jnf_respone)
|
||||
|
||||
|
||||
def store_all_files(request):
|
||||
files = request.FILES
|
||||
|
@ -377,4 +381,4 @@ def store_all_files(request):
|
|||
# description pdf
|
||||
for file in files.getlist(DESCRIPTION_PDF):
|
||||
file_location = STORAGE_DESTINATION_COMPANY_ATTACHMENTS + "temp" + '/'
|
||||
saveFile(file, file_location)
|
||||
saveFile(file, file_location)
|
||||
|
|
Loading…
Reference in New Issue