search

uclibs / scholar_uc_legacy

Source code for Scholar@UC up to version 3.x. Replaced by ucrate
Other
5 stars 1 forks source link

Use Eastern time zone when sending embargo notifcations #1942

Closed hortongn closed 5 years ago

hortongn commented 6 years ago

The embargo notification rake task is calculating the current time using UTC instead of Eastern time zone. So when the task runs late at night it thinks it's already the next day.

This just adds a Time.zone = 'EST' so that Time.zone.today returns the current time for our time zone.

I've tested this on scholar-qa. It works as expected.

coveralls commented 6 years ago

Coverage Status

Coverage remained the same at 95.952% when pulling 52920866c39a3f89eb5b40781af9a6553e1f1aa3 on fix-embargo-notify-time into d0ad2ca7031a2cedef7cb99c7e140e3ba1a03918 on develop.