-
Notifications
You must be signed in to change notification settings - Fork 86
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
removing check for expiration on jobs #311
Conversation
Signed-off-by: vsoch <vsochat@stanford.edu>
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.
Thanks @vsoch. It looks good to me. It does just what it is intended to do!
Look at the difference! https://887-120906408-gh.circle-artifacts.com/0/usrse.github.io/jobs/index.html
I kept feeling like we were overlooking a problem that the expiration date solved. I think it's for jobs like this: https://cu.taleo.net/careersection/2/jobdetail.ftl?job=18969&lang=en&src=LinkedIn. It isn't going to expire...potentially ever.
I'd rather have multiple "closed" job postings than remove one that is still open, so how about we leave the expiration dates in, but as standard practice put them far in the future? Perhaps many months. Treat them as a back up in case jobs are removed but URLs don't break.
So I'm suggesting we don't merge, and go back and edit the expiration dates of all jobs to 6+ months, and then let the existing checks take care of things.
Thoughts?
I personally like this strategy better to just leave jobs while the url is valid. It’s much simpler than trying to manage each alone with an expiration date. |
When you say "this strategy" which one do you mean? I've read it three times and can see it going either way. Should we (a) remove expiration dates entirely and only check urls, or (b) set expiration dates 6+ months away and keep the removal script when it hits this date, or (c) something else? I like (b) as it will keep things up for a while as long as the url is valid. And if the url is valid but the position closes the expiration date will eventually remove it without manual intervention. |
Sorry wasn't clear! I was referring to the current pull request, so I like (a) removing expiration dates entirely, which is almost the same as putting a far away expiration date, but requires much less work. |
OK! Got it. I didn't want to assume! Honestly, (a) was my original thought too. But now my concern is with what we do with urls like these: Some are obviously closed (would the url check get the first?), but the others are far more subtle. And this one still looks to have active jobs, but it's at the higher level so the URL will likely never expire: |
yeah, there could be some trailing links. I'm not on the usrse committee so I think it's up to you :) |
I don't want to play that card :). And you're the defacto website committee chair. Here's another one that will likely persist for ages, but won't get caught by the URL checker: http://carver.cs.ua.edu/RSE.htm. If you click on the link it's gone. If no one else chimes in, I say we leave it as is with expiration dates, and I'll go through and update dates on active postings. |
Sounds good to me! I'll close the PR here then. |
Per suggestion of @cosden in #309, this PR will remove the job expiration date, and show all jobs that have not yet 404'd as links.
Signed-off-by: vsoch vsochat@stanford.edu
cc @usrse-maintainers