-
Notifications
You must be signed in to change notification settings - Fork 80
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
Repost alert #410
Comments
This is a separate discussion in #127. |
TODO:
|
Can an employer indicate that they have hired for that role? If the role was filled, was it filled via Hasjob? If not, what else worked? How could Hasjob have worked better for them? I think we should capture this feedback as well. |
Managing celery is a pain. We can use cronjob for this. Can write a manage.py command and call it every minute. The jobs will require a flag to show that the repost reminder mail has been sent. Any idea how to implement that?
|
Why does it have to be called every minute? Once a day should suffice?
Why is it necessary to store this notification? If there's an error, there will be an exception raised so we'll know something went wrong? |
Hasjob already has a periodic cronjob that is called every 5 minutes. Look in |
Status must be kept. The periodic job is not guaranteed to run if there is server downtime. On every run, we query for items (a) older than a certain date (30 days) and (b) without the notification flag set. |
If it's necessary to store the notification, I suggest adding a flag called What's also important to track:
I suggest adding a |
@shreyas-satish what is |
|
If you have a job post |
Ok. Got it. Can we call it |
Found them. Thanks. |
|
@shreyas-satish yeah. |
@iambibhas Please close your other PRs before proceeding with this. This one is not a trivial issue. |
Hasjob introduced reposting in #212. Hasjob should go one step further and send an email alert to job posters whose job post recently expired. This would help encourage them to repost their job if the position is still available.
Caveat: Should job posts that were newly posted be shown prior to job posts that were reposted?
The text was updated successfully, but these errors were encountered: